labofsahil/github-event-dataset-2012 huggingface dataset:
I think `map: String` would have being simpler, will do the same too.
I can't find a test for this.
Its to override the base style.
It's probably with looking into. We still need to add a bit of extra functionality to this (e.g. we need to separate consumer side errors and provider side errors and notify the appropriate parties). I agree that maybe having a structured fallback to default values is probably a good idea. However, I don't think we specifically want to use a common method used by other apps as the determination of who to mail will be based on a very transfer-app-centric view of who is there and who is here.

I don't think this is a show-stopper for the pull request, though. Can I add a ticket for this and move on?
@joshcooper Great, thank you!
That feels like "is Ruby still Ruby"; are you sure it adds value?
I dont think this is a good idea.
k, thanks :)
$match === null
import Navigation instead of use FQDN here
Yes, meybe yes.

But, if you want to reuse parsed params from given route, output params must be same as input params.
Without this you can´t reuse them, because the controller segment  is prepend with the module.

I add new commit, which allow change controller segment to given segment.
Is this check needed? I mean hash() is in the core since PHP 5.2 , so detectHash() always will set type to hash
Not shure that there is any conventions. But I think it is better to use upper case for mysql/dql keywords.
Unreleated?
We might just want a ProjectileShootEvent, a projectile might be wanted to be shot from a point or something not an entity.
Oops, I guess this above line needs to go.
You still missed this one
Yeah, we should assume `exception.ex` is defining the fields in a proper order and just use it in here.
then you need to add a dependency to [Flask-Babel](http://pypi.python.org/pypi/Flask-Babel/0.8) in ``setup.py``
Can you use the absolute path (ie using `__file__`) here, to allow running the example from any folder?
Argument and its type and such
This is not cleaner, if your terminal is not wide enough you never see the second line and can be confused into thinking this is just an empty line.
I prefer for these line splits to have the new line to be more on the left,. so this would even become
 compilers = "-DCMAKE_C_FLAGS='%s' -DCMAKE_C_COMPILER='%s' " % \ 
                       (os.getenv('CFLAGS'), os.getenv('CC'))
Copyright header should be more extensive?
single line docstring?
Can you move this code down and use `$resourceOwner->handles()` instead? That will make this PR work for the new oauth1.0a support too.
part d - line 93-102
Also, you can avoid save signals by updating data on a `QuerySet`, which can still be as efficient a selector. So you could do that in the signal to avoid the double-save/argument-that-controls-point-updating thing.

See: https://docs.djangoproject.com/en/dev/ref/models/querysets/#update
Why is this wrapped in a string?
This is because sometimes requesting a start time of 10, would end up being 9.xx.

We basically give it a one second margin for error in the test.

Pretty harmless because we really just care if it is not 0.
You can provide a log by using a function ref and update it with the source's log when it is initialized..
Why does this need to be set here, and as an instance variable? Why not set it in the `step` block as a local variable when you need it?
It's only redundant if you make the assumption that Facebook will always return a next URL when it also returns a limit, offset and count. But that's fine I'll just go with this assumption and simplify the conditional.
No need for the new line, titles already have padding.
Missing processes when the user is in a group
ERBスタイルのデリミタが好みではない場合、UnderscoreのtemplateSettingsにて、コードの置き換えを異なるシンボルを使って行うよう変更することができます。
interpolateは、一致したものを補完する正規表現式を定義してください（デフォルトは`<%=　%>`用）。	
escapeは、HTMLエスケープした後の値を挿入する正規表現式を定義してください（デフォルトは`<%-　%>`用）。	
evaluateは、結果文字列に挿入することなく評価される正規表現式を定義してください（デフォルトは`<%　%>`用）。
上の3つについて、それぞれ定義、省略することができます。
この例では、[Mustache.js](http://github.com/janl/mustache.js#readme)スタイルのテンプレートを真似ています。
кто-то плохо слушает на семинарах.
if originalNodesOrder was not yet init, why do we have to init here? i think you could just if (null !==...)
@peterbe I'm messing with this so that I will work with django 1.4. The issue is that in the current master version the `.css` files are assumed to be in `settings.MEDIA_URL`. This patch is basically saying if the user has set a `settings.STATIC_URL` find the `.css` files there and not at the `settings.MEDIA_URL`
hey dave, please note that i did not write the test, i'm just trying to make it pass.

favorites are in fact a function of the logged in user, not a profile / cookie.
Did you run JSHint on this? I thought it was all about the "use braces all the time" thing.
Require is overkill for this.
Absolutely not!...... Jk. ;p I'll put it in in a few. 

Sent from my iPhone

On Apr 6, 2012, at 3:02 AM, Ryan McGrath<reply@reply.github.com> wrote:

>> @@ -175,6 +175,7 @@ def __init__(self, twitter_token=None, twitter_secret=None, oauth_token=None, oa
>>         self.access_token_url = 'http://twitter.com/oauth/access_token'
>>         self.authorize_url = 'http://twitter.com/oauth/authorize'
>>         self.authenticate_url = 'http://twitter.com/oauth/authenticate'
>> +        self.api_url = 'http://api.twitter.com/1/'
> 
> Mmm, so this is just about good to go in; the thing I wanna point out here is that Twython, by default, allowed people to specify `version=x` in their function calls and it'd be merged in. With this modification it's defaulted/stuck to `/1/`; can you modify this to still allow it on a function by function basis, defaulting to `1` if no `version` is supplied?
> 
> If this goes in then we're good I'd say.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ryanmcgrath/twython/pull/72/files#r654084
Wouldn't you only get 1 column if you're doing a `SELECT FIRST 1 *` ?
This is quite an expensive operation, so we should be careful with it.
Agreed. As we're removing firstName and lastName as well, I suggest the following

`RestAPI.User.createUser(restCtx, userId, password, displayName, {'visibility': , 'locale': , 'timezone': }, callback) {`

What do you think?
If the owner of `threads` changed, should the cleanup be done by `ScalaDebugTarget` now?
Ch. 6 iii
(The text here comes from Ch. 12 in the current UG.)
I actually found out the hard way that the Maven Repo was somehow marked 1.0.6 so the reason for jumping to "7" was to avoid future users pulling from Maven and getting errors in their projects. For example, I use Play! and sbt and my project wouldn't start until I changed to 1.0.6. So logically in the Maven versions to keep things consistent I'd jump it to 7.
maybe calling only the nested routers implementing MatcherInterface here ?
Why you use csv and not a native array?
For the sake of consistency, could we either make a separate 'Edit in Form' column for phenotype attempts, or make the mi attempt grid also do edit links like the above?
ik zal er even mee spelen. en wat sneller maken.
Yeah this is a ServiceLocator code smell, but I didn't want to make SettingsViewModel depend on SpellCheckExtension in the ctor, as the goal is to dynamically load extensions.
*literal и *subject_literal - это указатели (pointers) на объекты типа NSString.
по уму нельзя сравнивать указатели, надо пользоваться методом объекта isEqualToString

См.: http://stackoverflow.com/questions/3703554/understanding-nsstring-comparison-in-objective-c
Нужно переопределить сеттер и геттер, иначе об инкапсуляции можно забыть.
Что, можно будет поменять 150%?
This should be:

Random random;
"Maamärk" or "Piiripost" are better, in my opinion "Maamärk" without space is correct.
I think there are some words missing after "if the install location does".
I would leave this to be done in the user config honestly, mostly because you want it at the end of the chain in almost all cases.
There are a couple of tests that "Checkout" a bare repository outside of the specific BranchFixture tests. These include `CanCorrectlyCountCommitsWhenSwitchingToAnotherBranch` and `CanEnumerateFromDetachedHead`. Both of these look like they should be updated to reference a standard repository (vs just updating the current HEAD reference)
.... There already is a copy arg set to False in Epochs.crop


On Thu, Dec 6, 2012 at 4:14 PM, Alexandre Gramfort <notifications@github.com
> wrote:

> In mne/fiff/raw.py:
>
> > @@ -594,6 +595,55 @@ def resample(self, sfreq, npad=100, window='boxcar',
> >          self.last_samp = self.first_samp + self._data.shape[1] - 1
> >          self.info['sfreq'] = sfreq
> >
> > +    def crop(self, tmin=0.0, tmax=None):
>
> I agree for a copy. But then we should not call it crop as Epochs.crop()
> works inplace... Or just a param to crop.
>
> crop(self, tmin=0.0, tmax=None, copy=True)
>
> and add the copy arg to Epochs.crop(copy=False) (False to avoid breaking
> the API)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/218/files#r2335825>.
>
>
should be:

```
    logger.info('    ' * indent + 'end } %d' % block)
```

+ not ,
On this line I get this error:
```
IOError: [Errno 2] No such file or directory: '/home/larsoner/.local/lib/python2.7/site-packages/mne-0.4.git-py2.7.egg/mne/layouts/Vectorview-all.lout'
```
Add @should annotations and use the behavior test generator plugin,  see https://wiki.openmrs.org/display/docs/Generate+Test+Case+Plugin
Yes, comment. Sorry :P
the TranslationExtension class is not used.
they "might" be show.
Well, AFAIR the main objection was that it is yet another less-than half-baked Date&Time abstraction, but that’s really not what it is. It is just a duration, not more, not less. Going with nanoseconds everywhere would be confusing to all those people which expect that `Long` to be taken as milliseconds instead (since there is enough precedent for this mental connection).

So, if all the Date&Time-scented parts were taken out—which I think is only the ability to parse String(“1.5 seconds”)—would that still be objectionable?

Another view, compare:

    await.result(f, 1.5 seconds)
    await.result(f, 1500000000L)
В свитче кейсы у нас не отбиваются (и в следующей функции тоже).
Можно просто break делать, и так screening пустое же.
```haml
= render :partial => 'pic', :locals => { :a => 'xxx', :b => 1}
```
```haml
%div{ a => b }
```
```html
<div xxx="1"></a>
```
測試一下需不需要clearfix
Entendido, pensé que no eran parte del action.
There seems to be duplication between this method and the ones in the other xxxSearch classes  - could it be shared?
add tests to installer
@drewtempelmeyer if `prepare()` does not return an `HttpResponse` then `dispatch` will go on to call the appropriate `handler`. When I suggested that "this be changed", perhaps it was unclear that I meant L357 of dispatch.
I have the feeling that this assignment can be merged with the block above it, as if a restart is required, then we will `finish()` this activity in `mapRestart()` and never need the newly assigned `mapController` anyway since no user interaction will take place.

So can't we just use this one?

```java
        if (restartRequired) {
            mapRestart();
        } else if (mapView != null) {
            mapView.setMapSource();
            mapController = mapView.getMapController();
        }
```

I think we should go even further and remove `mapController` altogether as it looks like its sole purpose is to cache the value of the `getMapController()` accessor. It should be enough to cache this value only locally if we are going to use it several times in the same method, and this will remove any risk of divergence between `mapView` and `mapController`.
typo: ensures
i dont really like the fact that this "fake" stuff is coming into our code here. I rather everything to handle fakes/tests be in the tests.
space. wanna go. go to space. space. spaaaaaaaaace.
Do we also want to fake things out if it can't find an entry for 'remote_implementation'?
I'd rather you had a separate log message for the last chunk, to make it clearer.
Sorry, just realised you had two comments above - not just one.  Yes, we expose an enum as a part of the API, and we implement the behaviour described by the options in the enum using a TopologyAwareAddress (if exists), otherwise we default to the behaviour in DistributedTaskExecutionPolicy.ALL.
Ok, I understand. Do you recommend to check if the entry already exists and
if it is to do nothing or to update it ?
Le 4 déc. 2012 03:29, "Manik Surtani" <notifications@github.com> a écrit :

> In
> cachestore/mongodb/src/main/java/org/infinispan/loaders/mongodb/MongoDBCacheStore.java:
>
> > +
> > +   @Override
> > +   public void init(CacheLoaderConfig config, Cache<?, ?> cache, StreamingMarshaller m) throws CacheLoaderException {
> > +      super.init(config, cache, m);
> > +      this.cfg = (MongoDBCacheStoreConfig) config;
> > +   }
> > +
> > +   @Override
> > +   protected void purgeInternal() throws CacheLoaderException {
> > +      BasicDBObject searchObject = new BasicDBObject();
> > +      searchObject.put(TIMESTAMP_FIELD, new BasicDBObject("$gt", 0).append("$lt", System.currentTimeMillis()));
> > +      this.collection.remove(searchObject);
> > +   }
> > +
> > +   @Override
> > +   public synchronized void store(InternalCacheEntry entry) throws CacheLoaderException {
>
> store() should definitely not be synchronized. Any concurrency control
> really should be in MongoDB itself.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/infinispan/infinispan/pull/1473/files#r2301829>.
>
>
Should be changed to use TestCacheManagerFactory
I thought I had tried that and got a NoSuchElementException, but I was probably using the () operator. I'll change it.
Lets change the chart style to a string and not an enum.  If we kept it this way, we'd have to edit the code every time we want to put in a new graph.
Doesn't this need to be fixed with the updated code?  Since it is no longer counts+1, right?
The first thing that comes to mind is using 'to: phase' and 'to: 1-phase'; it should be fine with a binding, as long as you're not binding to the same property you're animating.

Not a problem, though.
yeah that looks reasonable :)
The example now uses demo.opengeo.org so this change is not necessary anymore.
Add a size!
Cool, that can be a new pull req.
IdempotentRequest will retry forever until it gets *a* response, but that response might well be an OperationResult that indicates failure. In that case, AgentTagsManager actually yields to its caller with the raw response instead of a list of tags!

Thus, we should at least detect that case here -- see whether tags is an operation result or a list of tags. You can look into AgentTagsManager to confirm this (and to add spec coverage to ExecutableSequenceProxy  to make sure it handles this case, since it's rather important not to put mysterious gunk into InstanceState).

If AgentTagsManager returns us an error response, I vote that we log the error, but continue with the converge. No sense being pedantic (and we'll be auditing the tags anyway, right?!)

Now, for the issue of what to do when the idempotent request takes forever: I vote that we let it take forever, and block the converge until it succeeds. At first this might seem dumb, but given that a tag query is only going to a local mapper, if the mapper isn't healthy enough to service our query, then it might not be healthy enough to process our audit requests and such once converge begins.
@bp323 Doing this on the back-end is indeed the way to go forward. If it's really not possible, can't we at least combine this with another POST?
What's the intention of removing SS_HTTPResponse as the return value of respond()? This changes things considerably.
You can remove this block of comments above
O man, we really don't want to do `eval` there.
The current `Lookup` class has a variable called `@lookup_tables = {}` I think every table (or dictionary) should be a sperate instance. You can then ask it to `#store` a new entry and to `#lookup` an ID by value. The glue logic needed, to assign lookup tables to attributes and definitions should not be part of that `Dictionary` instance. You can den define a simple "DictionarySerializer` interface to store those instances on disk, in memory or wherever you want.
`$body` property is missing.
The render method btw is a lot like copy pasta but the devil is in the details. Zend response and symfony response are different. The NEXT phase is to search for the common stuff and abstract it in the class, but it is the next phase.
Whoops, I forgot to remove the argument. It doesn't really make sense to have it in the function, I put it in the loop instead.
ENABLED and DISABLED are already defined on L46 and L47 respectively, although the DISABLED on L47 has a typo ('DISBALED').
space needed in the test
Are you sure ``new \Zend\Log\Writer\FirePhp\FirePhpBridge(new \FirePHP())`` has the correct type as "object<Zend\Log\Writer\FirePhp\FirePhpBridge>" is not a sub-type of "object<Zend\Log\Writer\FirePhp\FirePhpInterface>"?
Are you sure ``$value`` has the correct type as ``*`` is not a sub-type of ``string``?
Are you sure the method ``serialize`` can be called on ``$this->adapter`` (of type ``object<Zend\Serializer\Adapter\MsgPack>|null``)? I found this a couple of more times on lines: 60, 69, 78, 87.
Existsing? :musical_note:

-> Existing
VERY important change from double quotes from single quotes...do I even review my changes?
It was my first thought also. 
I printed their content in `visitAny` method, but I had problem with order of visited nodes. For example in `SumOp`, the order is `operator`, `leftTerm`, `rightTerm` and I didn't want solve these cases. But maybe there is some way, about which I didnt know?
"Moustache" -> "Mustache", though it does have a certain Old World charm your way.
Not sure. I use eclipse, so I just set project-specific settings based what's used in that project. HTML is typically indented by two spaces though because it tends to get nested quickly, so you might just want to set that as your default for HTML if possible.
исходный дамп должен быть пустой
Apparently this commit creates problems due to the use of test config.
Do you think we could remove that ?

Regards
Pascal
See http://developer.joomla.org/manual/chap-Joomla_Platform_Manual-Testing.html#idp6957800 for information on working with protected and private variables and methods.  New inspectors should not have methods to work with them as that is now provided by ReflectionHelper.
I may be missing something but it seems you are passing $oauth into almost every single method.  It would seem to make sense to put that into one of your base classes so that it doesn't have to be passed every time.  Similar to the way that JGithubObject stores the credentials and then automatically uses them when sending requests.
Nope. They will never be executed. Good catch.
Can we just set this to "example_project.settings.local"?
We're discussing the queue; there isn't a wait for the discussion to take place.
ah haha yeah this one maybe not, I have weird rules for those things. If it's inline I try to use `//` but if it's a function I'll use the jsdoc style
In anonymous functions, have spaces around `()`.
You can also wrap dict in a similar way we do it in Python.
haha maybe it would be faster to do TODO :D. 
So I should just leave 'piplmesh/images/logo.png' and add {{ STATIC_URL }} in js?
Indentation! Inside of `if` you should indent once.

Use only *Results for:* as a string. :-)
Maybe someday :D - this could be a new issue :D....

First, remove unique_with='sign'
Then objects(date=current_date).update(....)...

OK?
See how we defined choices [here](https://github.com/wlanslovenija/PiplMesh/blob/master/piplmesh/account/fields.py).
cannot really test because celery is not working on windows
Use of ?T for the ER case was just about me debugging. I wanted to use the same mechanism as everything else for returning the errors.  When the fitting returns {error, Reason} that's what will shutdown the pipe, unless it's better to do it this way.
Hmm.. Tricky... Perhaps this may be of help: http://stackoverflow.com/questions/677373/generate-random-values-in-c-sharp
Wie wäre es wenn wir #to_s auf dem Commit verwernden, und dort die Logik platzieren? Die benötigten Informationen befinden sich alle auf dem Commit...
There was no $match[1], but fixing this didn't fix the problem.
looks like name is currenlty one of 'user/company/inbox' - what's the intention of this? 

If we're in rails land - can benefit from active support -e.g.:

http://api.rubyonrails.org/classes/ActiveSupport/Inflector.html#method-i-classify
http://api.rubyonrails.org/classes/ActiveSupport/Inflector.html#method-i-constantize
What version of hadoop-lzo does this pull in? Is this the twitter version? I don't believe that's in the maven central repo yet, so this probably pulls in the wrong version. I think it needs the in-git-repo version for now.
What about thrift-test?
I plan to use this class in the dao package, in my read-write branch, so it would have to be public, but I guess for now it can stay visible only to other classes of the package.

Regarding CoreEntity, it was on purpose and consistent with MusicBrainz convention.
Indeed you have a lot of entities in MB: mediums, track, recording, release, but only some are considered as core ones. Those have a UUID, a comment, a name. E.g. if you compare Track vs Recording: a track has only an id, but no UUID, no comment. Recording has both.

Please let me know what you think. I think CoreEntity fits better, but I'll change it to AbstractEntity if you have a strong opinion about it.
`inst` is an instance of https://github.com/rubygems/rubygems/blob/1.8/lib/rubygems/installer.rb#L27,
it gets `options` here https://github.com/rubygems/rubygems/blob/1.8/lib/rubygems/installer.rb#L94
and they are parsed here https://github.com/rubygems/rubygems/blob/1.8/lib/rubygems/installer.rb#L412

comparing `process_options` implementations:
https://github.com/rubygems/rubygems/blob/master/lib/rubygems/installer.rb#L550
https://github.com/rubygems/rubygems/blob/1.8/lib/rubygems/installer.rb#L412 (1.5- 1.8 are similar)
https://github.com/rubygems/rubygems/blob/1.4/lib/rubygems/installer.rb#L96 (1.3-1.4 does it in initialize)

`@options` + `attr_reader :options` are not available bellow 1.5
`@env_shebang` is available in every version ... this code could be simplified to just:

    ruby_name = Gem::ConfigMap[:ruby_install_name] if inst.instance_variable_get(:@env_shebang)

unless there is something that prevents us from doing this ... but I do not think so.

even the implementation would change in master this code is here to backport it to older versions of rubygems, it would not  be required when in **rubygems 2.0**
I think the word "factory" would fit better here. Minor issue.
I don't know what this did. The problem with it being too long ago!

I've removed it now, new commits
I see that it makes the missing checks easier
Dict keys put convert into strings.
I could go either way on this one too. I don't think there are going to be very many instances of there being multiple transaction ids resulting from multiple relatestos. Just logging one transaction is more simple, but seeing as you had already done the work, I don't think there is enough reason to revert.

This is one of those things where either way you had done it, I don't think there is enough reason to advocate spending more time on it. Especially on such a fringe case.
can't this just be file:${nhinc.properties.dir}/gateway.properties
Blank.gif
It would, if I had been aware of that string method :)

I'll update.

On 14 May 2012, at 09:30, Mike Jarvis wrote:

>> +
>> +    Does some checking for sensible inputs, unlike the functions it calls (ReadAsciiInputCat()
>> +    and ReadFitsInputCat()).
>> +    """
>> +    # First check for sensible inputs
>> +    if cat_file_name == None:
>> +        raise IOError("No filename given!")
>> +
>> +    # Raise an apologetic exception for FITS input-wanting users
>> +    if filetype == "FITS":
>> +        raise NotImplementedError("FITS catalog inputs not yet implemented, sorry!")
>> +
>> +    # Then read in from the ASCII-type catalogs
>> +    if filetype == "ASCII":
>> +        input_cat = ReadAsciiInputCat(cat_file_name=cat_file_name, comments=comments)
>> +    # Return catalog to the user
> 
> Wouldn't it be clearer to just write:
> ```
> if filetype.upper() is "FITS":
>    ...
> elif filtype.upper() is "ASCII":
>    ...
> else:
>    ...
> ```
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/146/files#r817380

--
Barnaby Rowe

Jet Propulsion Laboratory
California Institute of Technology
MS 169-237
4800 Oak Grove Drive
Pasadena CA 91109
United States of America

Department of Physics & Astronomy
University College London
Gower Street
London WC1E 6BT
United Kingdom
OK, I'll make this change and let you know when it's pushed.
Hmmm, minor but could we just omit the "weakly" here in that last statement?  I'm not sure that lambda^(-6/5) is so weak, it's stronger than 1/ lambda after all!  The FWHM dependence is weak at lambda^(1/5), I'll grant that, but not so much r0.
I think that a user might wonder "ratio with respect to what?"  Could make this explicit, similar to how lam_over_D is explicit about what ratio is being made?
Why is this appearing?
unused import ProblemSet
Not very REST-ful URL names.  I guess there's no way to differentiate in the URL file by method?  So the GET would go to one view, and the POST to another?  I guess you could have both go to the same endpoint, and then differentiate in there by method -- why not?
Please give a and b more descriptive names.. ie subject and content
alternatives (that i can think of) are more cumbersome ... open to suggestions but agree as a separate issue (probably as part of a wholescale API review after some time)
Done, sort of. This has been rewritten int the JerseyClient fiter. It throws exceptions if anything goes wrong.
Signature of Widget#noResponderFor is a mistake perhaps. I will fixes it.
these -8s are a bit inscrutable, especially for what should be a nice example script. Maybe at least store this off in a nicely named variable and use it in the three places it occurs.
Not sure how the command went missing here. To be added.
Not a problem. Feel free to tell me to take a hike. I'm just taking a chance on you doing housekeeping while you're at it.

I'll always err on the side of merging in a change that makes things *better*, even if it's not "perfect".
lets leave as is, created #18 for this
Should be a constant
Is this really LGPL and your code or somebody elses and we need to credit the original LICENSE?
This whole block just got indented 4 spaces and wrapped in a try/except.
Why add a blank test?
add an else statement here which searches through var/contents, to cover the "just in case"s, such as when someone is being stripped
Which two lines? The part where I define the fire protection proc and the part where I make sure it only works for simulated turfs?
Maybe those methods could be generated, but I guess it's OK so.
I was being lazy, just fixed it.
I will at a link to the pinout file in the hardware/Production/DiscoveryF4 directory.
not 100% sure if every response is Faraday::Response...
Sure, sounds good. Is it easy to get access to the settings for upstream modules? Something I haven't looked into yet.
Good test coverage
Don't call it __g__ViewSourceUtils. `g` mean global, and it is not.
oops, that wasn't intended to be a tag.... oh well...
Maybe create select with XML and JSON options near api key input
Done, but out of curiosity why?  Is there a technical reason or is it readability?
a unit test=Unitテストという用語でいいかな。ユニットテストよりはUnitテストがいいような気がするので賛成。単体テストは意味が違う気がする。
用語集に追加よろしく！
If this is changed, it will also need changing in SwappableMetadata as well, which overrides clone in the same way.  We can always use a different method name if using Clonable here is wrong.   The intention was to use the copy constructor for all copy operations so that it's all done in a single place, and to permit copying of the coremetadata without any knowledge of the derived type.  It's used in ReaderWrapper, but currently via the reader interface (which I'd like to change in the future).
On Thu, Sep 13, 2012 at 03:04:27AM -0700, Andreas Mueller wrote:
>    If possible, I'd like to avoid the word "train" in sklearn. We don't
>    usually use it and I think it is a bit confusion to have both, "train" and
>    "fit" methods.

+1 for avoiding the *verb* 'train'. I am fine with the *noun* 'train', as
with 'train' versus 'test' set.
`self.l1_ratio = 1. - rho`

?
tab versus spaces ?
The idnentation issue apperas clearly in github view, at least
Is it not possible ot use the "os.name" System property? http://www.leepoint.net/notes-java/io/30properties_and_preferences/40sysprops/10sysprop.html
works for me.. that matches the wording used in lots of places already.
What happens if length is > 2?  the code doesn't seem to take that into account.
The commit comment talks about setting WU descriptions to blank, but this is changing the behaviour of WU debug values

With this code it's no longer possible to set a debug value to an empty string - not sure whether I think that is desirable or safe.
Repushed to fix this
does addResouce take ownership/destroy 'resource' created above?
I'm not sure you want to support dictionary options.  For datasets they were used for HOLe type support, and weird input formats.  E.g.,

    DATASET(record, COUNT(SELF.somefield)) items;

They can always be added later.
Just found the references in the file at the end.  Is this duplicating information, or is there a difference?
This check should come before the io->size() above
Minor quibble, while you're changing it, you might as well give a explicit error if sn>n, rather than say 'duplicate' when it isn't.
Here's how I would do it so that I can use the urlencode_params() method in the end:

```python
possible_params = [
     ...
]
url = ''
params = {}
for param in possible_params:
    if param not in kwargs:
      continue
    value = kwargs[param]
    if isinstance(value, (tuple, list)):
      value = '+'.join(value)
    params[param] = value
    url += '/' + param + '/%(' param + ')s'
    self.urlencode_params(params)
    
    response = self.fetch(url % params)
```

That way you get the advantage of letting `urlencode_params()` worry about all serialization such as dates or unicode strings.
Why rescue here? You've verified bytesize can be called, so, no NoMethodError on this one. Are you guarding against another exception?
Why not just include the one from `Spree::Core::UrlHelpers`?
This code suppress the NPE, I dont know why...
typo: preforms instead of performs
Wildcard?
While I understand your point about maintaining no  gaps in the numeric array keys by using array_splice, I've also never read anything to indicate that unsetting, and thus introducing a hole in the array keys, somehow changes the array from "numeric" to "associative".

> Unsetting a value from inside a numeric array changes it to an associative - that's by PHP design.

Where did you read this? I've never head that PHP strictly distinguishes between numeric and associative arrays like you are describing. The way I understand it is that in Zend Framework, we're using this IsAssocArray::test() method to check if it meets _our_ qualifications of an "associative" (non-numeric) array, and I was simply pointing out that the behavior of that test is altered by this change.

If this new definition, as you describe it, is what we want to consider an associative array, then great. I was just pointing out the change because this will change which arrays we consider associative. It's my opinion that `array('foo', 'bar')` and `array(0 => 'foo', 2 => 'bar')` are both "numeric" (not associative) arrays for all intents and purposes and by that logic, the original test was correct.
Obviously, it's gonna have to start with `django-`. The rest? I dunno.

Doing something like `django-whiskerboard-mongo` seems appropriate, but long.
Hi,
Sorry, which pull request ? The one for cocos2d-html5 or cocos2d-html5-tests? Thanks.
See note at https://jira.media.berkeley.edu/jira/browse/CLC-675
Looks better. Done in 0ba40556c94a748fd3067ec19d38e4e32f6770e0
http://dev.metasploit.com/redmine/projects/framework/wiki/DeveloperGuide section 2.5 Logging and from http://dev.metasploit.com/documents/api/:

     # File lib/rex/logging/log_dispatcher.rb, line 144
144: def ilog(msg, src = 'core', level = 0, from = caller)
145:     $dispatcher.log(LOG_INFO, src, level, msg, from)
146: end

Also 

$ grep -r --include=*.rb ilog\( * | wc -l
      14

So this is for logging and I now know how it works. But, I want real time feedback about wether or not the proxy connection was successfully established. If print_good is not in scope and $stdout cannot be used, what are my options?
Please describe in what situation a user might want to run each recipe.
Oh, if IPython automatically falls back to `__repr__`, then don't worry about. (Man, the IPython team really thought through this system.)

Yeah, a warning would definitely be better than printing every time. I think either case (warning vs failing silently) can be confusing for users. Whatever you decide is best is fine with me.
Yes, i do. 

I think that 10 seconds are slightly long. 
However, since "iframe" was required until the preservation place selection dialog was displayed, this value was used for safety.
SAme for samplers below.
@kring we'll need to work on this when integrating with imagery.  Most likely the material remains unchanged, but the imagery shader decides when to use it based on the specular and diffuse maps.
Good point.  I think the changes are minor and simple enough that I would probably still change it; however, I won't complain if you decide not to.
No, my mind translated the original copy as going into  a separate member variable because I thought we kept two copies.  I merged it into one like to make the code clearer.
I am confused, the scope of this var self = this is EventHandler, and still the context ``this'' is where EvenHandler is called? Wow, it does not make sense to me at all. If this is something that JavaScript enforces, then I've no clue.

If this is how JS works, then the self could return a set of getters in the self hash, rather than being empty. e.g., it could return the view:..., and the options:....
I did understand what the method does. The documentation did not match it. 

You said "I feel like you only read the first line of the method description, and posted you comment based on that. It is not a good way to review code, man." My answer to that is "that is not the case," but are you suggesting that I should not read or not pay too much attention to the methods' description? then, if you are, then  why are u documenting this method?  This documentation is misleading then. A better documentation could be: ``A helper method that allows us to bind an array of events to a single event handler. Note: This method assumes the name of event is the same as the name of the event handler. Violating this assumption will cause an error.'' This documentation is more descriptive.

If you like your bindAll()... then I would suggest you to have a EventHandlers.js script where the bindAll method is placed. The fact that you are putting a restriction on the how we name handlers, which it does not cover the cases where the name of handlers is different the name of events, I somewhat disagree with having this bindAll in the EventHandler mixing.
Note that when `rm` is executed without a directory check it is harmless, but we do get the spurrious error message:

> error encountered when deleting harness: Error: Command failed: rm: /tmp/driver/harness/ios/harness: No such file or directory
I tried this, and it works offline both in Dartium and with dart2js. Cool!
Personally, I'd put parenthesis around "canvas == document.webkitPointerLockElement". At first glance, I thought you were doing a compound assignment. I do not feel strongly about this comment.
Aqui tem um problema de responsabilidade. As classes de domínio não devem lidar com entrada/saída. Era melhor passar esses valores por parâmetro.
Fair enough. Restored.
Instead of putting a default value inside the block, `#fetch` takes a seconds parameter (see line 57 below for an example).
> Also, is there any way from the index to see if it is marked for deletion? 

I see no documented property on the index that would be helpful here.

> Is 1 second realistic here?

Probably not. I'll change to 5 second max.
ok, I was just following step 2 to set up a project (as described in your post https://groups.google.com/d/msg/codehaus-mojo-gwt-maven-plugin-users/vrLAcqp5oAg/0oL-89D5m2gJ) with "mvn package -Pdraft"
That's why I did that change.

Perhaps, a simple README could clarified the expected workflow.
consolidate asserts, print `text(val)` if assert fails
It makes code look way cleaner IMHO, especially when you end up with having to indent stuff from chained function calls, because then each argument lines up nicely with the function that it goes with, making the code easier to read than if it's using 4x indentation everywhere.

I'm certainly not going to fault you for preferring a different indentation style, but I'm also not going to agree that we should try and use the indentation style that you're suggesting everywhere.
std.typecons is not the appropriate place for this. "Typecons" stands in for "type constructors", i.e. it creates new types. This function creates new values of already-existing types.
Shipit!
```
       _~
    _~ )_)_~
    )_))_))_)
    _!__!__!_
    \______t/
  ~~~~~~~~~~~~~
```
per @pmuir, target product is WFK
Could you fix this too ?
Also `final`.
allow failures? O_o
est-ce que l'api est a l'envers ?
the paragraph should include the next one too .. otherwise it seems like this intro paragraph covers the entire functionality.
Completed is better than History
Agreed that the comment is bad - will fix.

As for the ram_covered, part... There is a possibility that they may want to change how many RAM entitlements are required to provide unlimited RAM. This was an attempt at proofing for that... Had to decide on how RAM was counted when multiple entitlements are counted in that case. I made an assumption that they would be summed. While currently it won't be used with 2 meaning unlimited, If that was changed to 4, we would need to know how much was covered.

Do you think that I should remove that supporting code for now, and deal with it IF/WHEN the time comes?
this sentence is not finished
The problem is that you can't do this:

    union { ... } __data;

So even if you did:

    union { ... }

You wouldn't completely conform to the C API since the union members can't be accessed through `__data`.
Ohh wait, i see what you saying, my bad. Perhaps we can introduce a boolean "channelChanged" and if its true, then re-resolve the channel and reset the boolean back to false, thus keeping the optimization without breaking changes. i think you'd agree
Its literally the same. . .
Hi, Gary!
Hardcoded version: may cause issue with Spring 3.0 (or the even 3.2) build
Yeah; I'm not sure that's the right solution either; I may be getting old, but I find it difficult to keep track of all these conditionals in my head.
why all `int`s are being changed to `long`?
You could just do ``import dtk.bicycle as bike`` or something similar, then use ``bike.front_contact()`` below. Doesn't really matter though.
This variable declaration should be moved up to above `attach()` for clarity.
underscore length must match with title length (the same occurs with the rest of headers in this PR)
With '0' priority, [ArraySubscriber](https://github.com/KnpLabs/knp-components/blob/master/src/Knp/Component/Pager/Event/Subscriber/Paginate/ArraySubscriber.php) catches this target. : (
Please remove these steps as you only need `git clone`, `git init` is not needed nor the mkdir!
typically i would be put on a separate line to increase readability
@apex13 Good catch, should be fixed
@mweimerskirch yes ok for me
What is the issue here ?
best to keep code comments in english :-)
why ``$(that)`` ? ``that`` is already the jQuery object so you are only adding useless overhead
How about this setup:

```js
$('.thread-container').on('fos_comment_thread_params', function(e, params) {
    params.sorter = 'date_desc';
});
```

```js
getThreadComments: function(identifier, permalink) {
    var params = {
        permalink: encodeURIComponent(permalink)
    }

    console.log(params); // {permalink: 'string'}

    FOS_COMMENT.thread_container.trigger('fos_comment_thread_params', params);

    console.log(params); // {permalink: 'string', sorter: 'date_asc'}

    FOS_COMMENT.get(
        'url',
        params,
        function() {}
    );
}
```
Did these options get lost?
Does this method still need to be in? I don't see it being used anywhere
I got it~~~
Dit is nodig om properties van een pagina in een andere tab dan de eerste te krijgen. Kan misschien beter opgelost worden door de pagina een interface te geven ipv te controleren of de methode bestaat.
While it is often the case I add these comments why too lightly, this is a work in progress and I need to review all these comments after all tests are passing, to make sure I'm not missing important cases.

I can add comments on the comments, but again this being wip it felt redundant.
Maybe keep the time out in case it is going to be fully acknowledge in future release of iOS?
hablame bien. -.-
you're bit-rotting yourself!  Also, that was a pretty cool typo on my part, eh?
Just a stylistic thing, but it'd probably be better to use armstrong.core.arm_content.mixins.PublicationMixin
Sure, I hope to give this a try before the meeting today.

One suggestion for things like this, though would be to create a specific "Issue" for this via
https://github.com/osu-capstone-nvidia-project-2013/test/issues/new
and then we can assign & track the work there.
Makes sense to me. It's opposite of what I tried to do, so I'm closing the pull request.
We don't need the console message here
Singleton instance could just be called instance
document.write - are we using this so that the widget isn't loaded with JS off?
> Extends jQuery Library
In this method are you assuming that when calling this you are in another extension?
i have no other choice so I am leaving it as it is. @jeffheifetz you ok with that or do you want me to remove the console.log
I rather this doesn't mention hackerleague but rather focusses on what an organiser should look for in a tool for submitting hacks.
It's six of one, a half dozen of another.

Another, slightly more elegant way of handling this is to use a memoization decorator so the function below becomes:

    @memoized 
    def get_query_set_params(self):
        # generate value and return
此处 "fold" 并非特指某函数的名称，而是对一类函数的泛指，是否也需要标记为代码呢？