labofsahil/github-event-dataset-2012 huggingface dataset:
Unfortunately the git protocol has no support for TLS or X.509 client certificate verification, AFAIK. Defaulting to HTTPS, and allowing users to trust Github's cert/CA ensures that users are not vulnerable to MITM. This way, if a MITM occurs, the user will get a prompt that A) the certificate they're being served is not valid for GitHub.com and B) that the CA has changed to an untrusted party. If a user continues at this point, it's their responsibility. Composer has done everything it can to protect them. By defaulting to git://, a MITM could occur silently.

If we could promote and enforce the use of signed tags, this problem would go away entirely.
why adding extra braces here ?
Looks like a merge conflict with this commit https://github.com/diaspora/diaspora/commit/95d5c9ca681e9990a2caf10448b922339b9d694a
Write a function that reads ow_name_t  from eeprom and store that name (that char part) in *ow_sensors. This function will be called whenever a sensor change (discovered/removed/lost) occurs.
You are right. :)
I'll work on it.
What's the justification for returning a list here?  It doesn't seem intuitive to me
then you have `tuple.first` and  `tuple.second` here
style: newline before {
Ok shall I fix the default.html?
These are in addition to those give in the `modules` option?
In the notes on https://id.etherpad.mozilla.org/primary-transition-ux-swarm, it mentions there should be a screen displayed for the `transition_to_primary` state, yet here it has the user entering their password.
"mobile devices" - meaning typical phones, but not tablets?

We have a lot of different min-width. It might be good to standardize the 2 or three layouts we support and document them at the top of the file, then use these same values in the media queries. 620 and 505 seem like less popular breaking points, but I haven't done a lot of production media queries.

These are just knits, r+
There was originally a test for `completeTransition`, did that get renamed?
fixed, this made tons of sense. I'm bad at names haha
class doesn't need to start for the first "div" because the classes are repeated. if its id its a different story, id is used for specific block of codes.
what about `isCaseInsensitive` ?
Should we also support `include`, which is compatible with the Puppet DSL, as an alias for this?
This seems to be very similar in purpose to the code in `puppet/util/methodhelper.rb` implementing `set_options`; perhaps we can move this there and use methodhelper here?

If you do that, please sweep the rest of the code for other open-coded versions of the method and unify, also, those to the methodhelper version.
You could change `BodyResp` to `RD2` and pass it to calls below to save some space and repetition.
I think at some point we should break this up into classes, but its ok for now.
2 spaces… please follow code style
OOoh this is just for that, not the applicant dashboard. I was confused. :)
Both fixed, thanks!
Should this class name be renamed to CamelCase?
I'm wondering if this check should be configurable. It's not uncommon to define custom media types such as `application/vnd.my.app+json` -- which would fail this check, but still be something you'd want to automatically decode. The configuration could be present in a protected class member.
VehicleStateChangeEvent?
I moved the UpdateHealthMessage to setHealth in VanillaPlayer.
why not

    permissions do
      { GDS::SSO::Config.default_scope => ["signin"] }
    end
I would import it as HTMLParser_ and create the new class named HTMLParser (and not PelicanHTMLParser)
This is a BC break, but i suppose the probability somebody implementing it is very low (instead of using the abstract base classes).

Can you add a statement to UPGRADE.md file nevertheless?
Originally blank strings were being accepted here, and I assume that's not a valid case. Maybe strlen($mapping['fieldName']) === 0 is better? I'm not really sure what the valid cases are to be honest.
Should we actually support access? What if it is some malicious use of access? :)
With inner joins (true for visibility also). We can't use them in criterion handlers without modifying search gateway.

As for visibility, legacy fetches include ezcontentobject_tree table in FROM clause, which is not the case here.
Please use repoman to check for problems. DESCRIPTION has a limit of max 120 chars. Please drop PN from DESCRIPTION.
Should update the dictionary returned by the parent, instead of returning a new one.
import order!
Missing license doc.
maybe s/take/use/ ?
Ok. What's the difference between org.jboss.bom and org.jboss.spec? Is the latter intended to be used as well for importing?
The line doesn't do anything. Are you still working on calling macros with keyword args?
Figured out that the problem was in the code in search.html; not the test. I'll back this commit out and send in the pull request
I mean this whole way of creating groups. Could it be on the Group model? or is it phonebooks specific?
For other descriptions you have removed the final dot, here you add it. We should be consistent even if it's a very little nit.
should write play(5) sets currenttime to 5, or, play(n) sets currenttime to n.
yep - I should get some sleep!
Ok then.. I see the test suite for mocks also does that, although I don't see how it could fail due to the assertion.
Just preference. Underscores look strange to me when in a link, as links are usually underlined, but I'm happy to change it.
Just updated the pull request with better path normalization. Also, both path styles can now be used in `require` like with Node. See 2e4e6cdb2be204740ca341da787aaac00d347721.
I dont' think there's any reason to put your instance methods in a submodule InstanceMethods and then include them. This is something you see in the wild in rubyland that has no purpose whatsoever. Just include your instance methods directly in the module Searcher, and then they'll be automatically included when Searcher is included, because that's how ruby modules work. You follow me?

I also wonder if there's a way to avoid this UrlFetcher business, still trying to understand what's going on there.
This seems reasonable, but just want to confirm that pagination won't be around any longer now that there are categories?
The old code attempted to import the C implementations on line 50. If that failed then it defined the functions with there Python implementation. Your changes prevent any of the C functions from being imported. Particularly c_websafe and uwebsafe_json which were used later on the file. I think that the previous try/catch construct needs to remain but without the import of uspace_compress and definition of spaceCompress inside the try block.
What would an abstract class help us with? We still need to be able to set the level somehow, and preferably without knowing anything about the underlying Zookeeper implementation. Perhaps I'm just not thinking clear, but could you expand on this idea?
Log.log?
Wait, endless loop why?
nit: indent looks a bit weird here and above (at least on github); maybe some residual tab/space confusion
anything that will break? i know a property in manager.py will always return "Unknown" from now on
Indeed... Can I change/fix this and correct this pull request? Or I'll need to start a new one?
resque-lock should really be using a set instead of separate keys to make it easy to get a list of locks and check if locks aren't being cleaned up.  I have a [commit to use redis sets](https://github.com/dylanahsmith/resque-lock/commit/aa98d2af6e0baf17a41890b8f3d7fa3b1fdb9894), but I will wait for #2 and #6 to be merged before opening a pull request to avoid conflicts.
Right, `g_strstrip()` doesn't allocate the memory but rather modifies the string in-place.  So yeah, the memory must not be freed (or it'd be freed twice and **boom**).  For the record, e05f1d6624db0deae08d4300630338910ba3c47e introduced the improper free.

BTW, you could even remove the temp_body variable and simply call `g_strstrip()` on `p_url`, or straight in the `g_strdup_printf()` call;
тесты лучше называть по тому, что они тестируют, а не по номерам.
аааа..... что тут происходит?
тот самый
А какими должны быть функции в заголовах очень хотелось бы узнать.
Any documentation is better than no documentation. I'm happy you documented everything here - thank you for that! :) Also, documenting what seems to someone as "obvious" is also necessary - it's not always obvious for everyone.
I would encourage keeping these docs. Then improve them if they turn out to be not good enough. But for now, they're better than most of the other docs in the code base.
You should also add the license here ( %doc LICENSE )
We don't need this  -- mongo doesn't need/provide/use it.
Feel free to add an appropriate copyright statement if desired.
If the only reason we really need these if for JDBC, then I'd move them into jdbc.clj.
Ah perfect resources! Thanks.. my bad..

On Mar 26, 2012, at 1:07 PM, Cees Moerman wrote:

>>           email_address:
>> -            tab_name: Email adres
>> +            tab_name: E-mailadres
> 
> I'm afraid I don't quite agree here. _e-mailadres_ is a compound, and according to official spelling rules compounds should hence be [written as a single word][1]. ([Van Dale][2] uses the middle dot '·' to separate syllables, compare [Onderwerp][3], and [Het Groene Boekje][4]). [This article][5] has some background on the issue.
> 
> [1]: http://woordenlijst.org/leidraad/6/2/ "Compounds, Official Word List of the Dutch language (Het Groene Boekje), Dutch Language Union"
> [2]: http://www.vandale.nl/opzoeken?pattern=Emailadres&lang=nn "E-mailadres, Van Dale Dictionary"
> [3]: http://www.vandale.nl/opzoeken?pattern=onderwerp&lang=nn "Onderwerp, Van Dale Dictionary"
> [4]: http://woordenlijst.org/zoek/?q=e-mailadres&w=w "E-mailadres, Official Word List of the Dutch language (Het Groene Boekje), Dutch Language Union"
> [5]: http://www.onzetaal.nl/taaladvies/advies/e-mailadres-emailadres "E-mailadres / emailadres, language advisory, Society for our Language / Genootschap Onze Taal"
> 
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/resolve/refinerycms/pull/1483/files#r602317
Looks like this isn't fixed yet.
Done. Also there is a permission check for this (only global admin can, unless configured to allow anyone)
I'm not entirely sure if this will work but:

If the actual commentId is returned in the GET requests than the user can use it to update/reply/delete a comment rather than using the created timestamp. That way the _getComment method can drop the first Cassandra read and just do a direct read against the Comments CF.
logged in user
Might be worth extending this to generate replies as well, as to encourage the use of replies in the tests.
visibility is not supposed to be renamed to privacy
! (i'm annoying huh)
Ack, thanks for catching this.
Oh. That's a reference to the old code. It could be removed, but I was worried that someone might look at the 6 lines in the new code and think "Hey, I can do that it 3 lines!", so this is an explanation of why that doesn't work.
why implementing the interface when you don't need the kernel ?
Got it:

```javascript


        update: function() {
            this.parent();
            this.move();
        },

        move: function() {
            if (ig.input.state('up')) {
                this.vel.y = -200;
            }
            else if (ig.input.state('down')) {
                this.vel.y = 200;
            }
            else if (ig.input.state('left')) {
                this.vel.x = -200;
            }
            else if (ig.input.state('right')) {
                this.vel.x = 200;
            }
            else {
                this.vel.y = 0;
                this.vel.x = 0;

            }
        }
```
ну тут разночтения из-за неточного задания 
Я и наверное не только я,  под словом заменить подразумевает операцию записи в массив (эдакий replace), что записывать тоже сказано - случайное, не где же не указано специально в задании что оно не должно совпасть со старым
Для корректного сравнения строк используется метом isEqualToString. != сравнит объекты-строки по указателю. В данном конкретном случае — != тоже будет работать, но это "случайность" и в будущем может быть источником больших проблем.
Ну, на вкус и цвет, как говорится... Ничего плохого в этом нет, а иногда - даже нужно, чтобы было написано именно так. Свойство - автоматически создаваемый интерфейс для какой-либо переменной, так что как раз сразу видно, что есть переменная, и у нее есть геттеры и сеттеры, с ней можно работать, не проверяя, есть ли соответствующая реализация. 

Вы должны понимать, что если в интерфейсе видите 

`@property NSMutableArray *DNA;`

это автоматически означает, что на самом деле написано что-то вроде этого: 

``` 
{
    NSMutableArray *_DNA; // название зависит от synthesize
}
@property NSMutableArray *DNA;
-(void)setDNA:(NSMutableArray *)DNA;
-(NSMutableArray *)DNA;
```

Да и в учебных материалах свойства учат именно так, используя совместно с переменными класса. 
--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
Отсутствует проверка percent на допустимый диапазон [0;100].
Ага, правильнее, и не деление по модулю, а, пожалуй, ограничение по значению. А то передадут 121, а выйдет 21%
Лучше подобный метод делать методом класса (class method)
OK, I followed an example I saw elsewhere in the code for this structure.  I'll update it so both messages are included in each conditional.
How about renaming this to `+currentOrDeferredScheduler`? That seems more like the behavior we're describing.
Same answer as above :metal:
I don't htink we need this class anymore.  Any concerns with removing it and references to it?
I'd prefer not doing that, because the net/https library is well known and calls them just `ca_file` and `ca_path`, so we ought to signal that these are what we're passing down.
Very quick :)

Nope, that file was deleted, but I guess I just did "Delete References". So it was still on disk but not in the project. It's gone now.
merge the three for loops in one loop.
Interfaces should begin with "I" in Wonder.
Wow. Does your provider not support the transceiver bind?
I definitely see your point the the ack_ids attr reader then.
You should remove the configuration-block if you have only 1 language (yaml).
这与上面描述的情况有同样的开销
写入请求日志
=>
写访问日志
吧，大伙都熟
This is now bound by how many items need to be interpolated, rather than the number of interpolations available.
JSONLint is implemented in Node - I prefer the pure ruby approach. I can't find another reasonable CLI tool to validate JSON. Suggestions welcome.

I'll update this to point to the web version.
Another thing: it should NULL-terminate the `argv_copy` array, i.e. `argv_copy[argc] = NULL`
This is definitely the kind of thing that needs to go in a coding standards document, and a script which validates these rules would be nice. Maybe during the creation of coding standards, we can discuss whether to keep things at 80, or bump up that limit to 100 or something else.
Okay, you are right. It's just a bit confusing since it only renders a row or div for the attribute, and I would understand the whole model displayed by CDetailView as the item. But will rename it to renderItem to be consistent.
GetItemFromId возвращает 0
Закрой плеер - ее и не будет
I just removed the reference, it's not needed on the MessageBox
Should this not be a boolean?
wrong brace alignment
please remove extra parenthesis
If you create new classes and methods using wizards or code completion, please clean them up afterwards.
What line number is here exactly? There's a NETWORK_MESSAGE printing the content length, for example.
why add this restriction? i'd like to be able to boot anywhere
How about:
```
checkbackports() {
    wget -q -O/dev/null http://backports.debian.org/debian-backports/dists/${1}-backports/Release
}
...
if checkbackports $RELEASE ; then
    ...
else
    echo "Backports for $RELEASE are not available"
fi
```
good idea. i like
Yeah after we talked i understand there are some issues with ovz here but not sure how to handle that yet.
u updated your colon?
Haha. Ic, sometimes less is more :-)

Sent from my iPhone

On 10 Oct 2012, at 22:25, Dan Berindei <notifications@github.com> wrote:

In core/src/main/java/org/infinispan/util/logging/Log.java:

>     void cacheViewPrepareFailure(@Cause Throwable e, CacheView newView, String cacheName, CacheView committedView);
>
>     @LogMessage(level = ERROR)
> -   @Message(value = "Custom interceptor %s has used @Inject, @Start or @Stop. These methods will not be processed.  Please extend org.infinispan.interceptors.base.BaseCustomInterceptor instead, and your custom interceptor will have access to a cache and cacheManager.  Override stop() and start() for lifecycle methods.", id = 173)
> +   @Message(value = "Custom interceptor %s has used @Inject, @Start or @Stop. These methods will not be processed. Please extend org.infinispan.interceptors.base.BaseCustomInterceptor instead, and your custom interceptor will have access to a cache and cacheManager. Override stop() and start() for lifecycle methods.", id = 173)

I'd rather we delete this completely, I'm seeing 1000s of instances of this
message during the test suite :)

—
Reply to this email directly or view it on
GitHub<https://github.com/infinispan/infinispan/pull/1372/files#r1811577>.
But why can't the cache be created by the nodes on demand when the task arrives, instead of having a specific command for it?
not wrong, but do we need it?
Hmm, you're right, but there are lots of other tests that use /tmp (I found 10 just in core).
I think the best approach would be for TestingUtil.tmpDirectory to not ask for a prefix at all, and instead put everything in java.io.tmpdir.
rawValues enables a HotRod server to be used by a RemoteCacheStore and a RemoteCacheManager at the same time (i.e. by not storing ICEs on the server).
hotRodWrapping enables rawValues plus it also enables a custom marshaller and entry wrapper to rewrap the ByteArrayKey / CacheValues so that they can be seen by the HotRod Server. It's just a configuration shortcut but easier than specifying all those params (which is error-prone)
Ok, fair point. It should go away anyway...
this condition is used twice and a bit hard to follow, perhaps move it in a variable named "stateTransferNeeded"?
We cannot unload the module at this point because:

- the module loader may not support unloading
- this module load action is not necessarily the one which initiated the original module load
This applies in reverse too.
Specifically, de-denting to align the starting single quotes--I hope that is what you mean, @pelson.
This was requested by stephend since it was no longer loading in the CI. I don't really have other details.
I would say the deafult openengsb formatter will create 120 chars per line for comments; not 80
it's a french translate
Yeah, I've marked this "minor" as in "not required for initial merge", I know its a decent chunk of work :)
Пустые лучше удалить
Just out of curiosity, if $bean_name is not set, I'm assuming the the sub panel won't either. Wondering if we should put something in the error log here to help people debug this fact.
I've moved it.
Should the type for parameter ``$container`` not be "null|AbstractContainer" instead of "AbstractContainer"?
``function lockExists()`` does not seem to conform to the naming convention (``^(?:is|has|should|may|supports)``).
Are you sure you want to override the parameter ``$values``? This often makes code less readable.
Add a `<!-- more -->` tag here so that the post doesn't appear entirely on the blog front page.
Are you happy with this behaviour?

I would have let the finally decision to the plugin.
I mean, if we are in the "send" method, we send.
In the reply, we reply and that's it.

You will use this project more than me anyway so you decide the default dehaviour :)
Why return `key` here?
What populates this?
Should this and following throws include the unsupported method name? (see line 1194)
Line 297 of this file should probably get the same treatment.

Or really, this only comes up when the input str is an empty string. Could just add this near the top:

if(len == 0) return str;

But yes, this should probably go upstream to the pystring repo. The crash happens on Windows when you try to access str[0] of an empty str.
compare this code block to the one below. How should each of them be highlighted?
Even w/ test :-)
How would we do that?
`kernel.root_dir` points to `app/`. I figured it'd be more practical if the image path was relative to the *project* root directory instead of the *kernel* root directory.
Experimentei remover esta migração (ficheiro) e refazer a migração. Funciona como é suposto, até porque o `ßticket_id` nunca é utilizado. :)
I'm proud of this one because I was unsure since nth-child isn't supported in IE 8 and below, so I looked at our Webtrends traffic and only 3% of our traffic this month was from IE < 9. The breakingness that would happen without it is greater than that. :D
All this bookable and openText calculation are used on other pages aren't they?
Shouldn't this logic be in signup.js?
If you call attach, you get a raw socket and have to do the rest yourself, including making sure it gets closed in the presence of errors. Java doesn't have with blocks or RAII to do deterministic cleanup in the presence of exceptions, so the standard idiom is to pass the necessary behavior in as an anonymous implementation of an interface.
Turn logic around: if not, raise.

Where do you get `self.user`?
That was a mistake, it belongs to the place where upper empty string is :)
They definitely don't use camel-case!
Why list? Every user will probably have only one settings, no?
I couldn't import views.py because views is importing resources. Can I move this constant to settings file or somewhere else?
There is a problem. Our API doesn't work with the newest tastypie. I installed it from the repository on github using this command ` -e git+https://github.com/toastdriven/django-tastypie.git#egg=django-tastypie `  and I'm getting an error: `QuerySet' object has no attribute 'query'`. Any idea what should I do?
No, there should be `"` around. This is how new `url` tags does it.
If you would really want to do it like this, it should probably be:

```
hugs = mongoengine.ListField(mongoengine.ReferenceField(account_models.User), default=lambda: [], required=False)
```

No?
Noo. Here you just pass number:

```
var format = ngettext("%(day)s day ago", "%(day)s days ago", days);
```
Copy+paste oversight. I'll fix that
this looks like it could change the functionality or break time parsing? wil `params['time']` return the `20` part of the year now? is still backwards compatible across versions 4 - 6?
Sorry - define an interface for what?  Retrieving the in-call state?
etwas überflüssig
Eben genau deshalb ;)
I don't have a specific example in hand, but it probably isn't long before users will want to be able to include additional data from taxa in their dataset. For example, data derived from reference genomes that match their OTU sequence exactly. In the case of shotgun metagenomes, this could mean including whatever information can be gleaned from the bits of genome adjacent to the phylogenetic marker on the contig. 

This is vague, and I was mostly curious if there were any plans for addressing this. Seems perfectly reasonable to postpone any effort until/unless users start asking for it, have some actual scientific reason to want to do it.
¿Puedes indentar todo este bloque 2 espacios a la derecha?
change attribution to "jQuery Fundamentals" and remove the status, editrequired, and source metadata, please
oops, `model.__name__` ?
Yea, I was aware of this one but wanted to profile it. It did show up as one of the top most expensive selectors, so I prefixed it. Thanks!

@p01, they can be quite bad in Opera since they aren't optimized at all.
querySelector("textarea") would be a bit more solid towards changes.
Is it possible for fetcher_ to be NULL here?
I don't believe so.  The LLR for the mean is monotonic between the mean and the maximum value of the data.  The algorithm should converge in all cases.  Nevertheless, I was thinking about deleting this as a whole since the gamma method works fine and even if it doesn't, brents is much faster too.
Oh, cool. Thanks.
I agree with everything you proposed. I don't see the problem if we override Google's CSS? In any way we want that also Google's results look like our web page? But yes, feel free to fix the CSS too.
Missing method description
...added header.
talked to Avi - we can restructure this if we ever get to the point where attributes are going to have fields that variants don't
For the always functions that fire regardless, thumbsFetched/psFetched seem like misnomers. If I were seeing these elsewhere in the code and they evaluated to true without there actually being any thumbs or exercises, I would wonder what happened. Maybe fetchThumbsAttempt = 'complete' or thumbAttemptComplete = true or something?
ok, changed to webcam,link
nothing against them -- just they haven't arisen.  might prefer a more customisable approach in any case.
As long as you don't use `--` in an expression they have identical behavior (minus the negligible and hopefully compiled away performance difference), and I'm of the persuasion that neither should be used in an expression (if you want `--a` then decrement on the line before, if you want `a--` then decrement on the line after). 

To me and most programmers I have encountered `a--` semantically means "decrement `a`," and that works fine. I won't be surprised if there is a shift towards `--a` as the canonical "decrement `a`," and I don't think that would be a bad thing.
I will rewrite int/*long*/ to GLint.
But, there will be a lot of similar problems. I haven't confidence to find all similar problem.
Ok, solved in the next commit.
Hannes Magnusson <notifications@github.com> wrote:

> >    mongo_cursor *cursor;
> >    mongo_collection *c =
> (mongo_collection*)zend_object_store_get_object(coll TSRMLS_CC);
> >    mongo_db *db = (mongo_db*)zend_object_store_get_object(c->parent
> TSRMLS_CC);
> >    int response, safe = 0, fsync = 0, timeout = -1;
> > -	mongo_link *link;
> > +	mongo_link *link = (mongo_link*)
> zend_object_store_get_object(c->link TSRMLS_CC);
> > +
> > +	timeout_p = zend_read_static_property(mongo_ce_Cursor, "timeout",
> strlen("timeout"), NOISY TSRMLS_CC);
> > +	convert_to_long(timeout_p);
> > +	timeout = Z_LVAL_P(timeout_p);
> > +
> > +	/* Read the default_* properties from the link */
> > +	if (link->servers->default_fire_and_forget != -1) {
> > +		safe = !link->servers->default_fire_and_forget;
> > +	}
> > +
> > +	/* Fetch all the options from the options array*/
> > +	if (options && !IS_SCALAR_P(options)) {
> > +		zval **safe_pp, **fsync_pp, **timeout_pp;
> 
> safe, safe_pp and safe_str seem to be obsolete now that w is working
> fine?
Obsolete, yes. But we can't simply remove it. We probably also should support fireAndForget in the options array and then fix the docs to our advantage.
Ahh, ok. I personally would prefer if this kind of stuff was not an explicit dependency of the project, but since a Guardfile is shipped with the project it is probably a good idea to make sure it works.
the second part should be removed as the file is now copied from SensioDistributionBundle on install
Oh yes, please do.
2 things

1 - why logger.info *and* print?

2 - why not print anything when qlen is 0? block_until_done prints an even less-interesting message.
Until homebrew has got a beagle formula, I'll have to remove this. But I can leave a comment so it could be added back later.

A beagle formula would be cool :-)
(Though the CUDA drivers would have to be installed manually in any case then)
А вот и нет, calendar меняется при распечатке, т е сдвигается сначала на первое число месяца/года, затем проходит через месяц, и тут мы его обновляем тк нас волнует текущее время
Должно быть System.exit(1)
Этот код присутствует в двух местах. Надо убрать copy-paste.
change the artifactId to something like jboss-as-helloworld-jdg to no colide with previous existing helloworld quickstart
bump to 0.12.7
What do you think about making this stricter? Maybe add an `is_rgb` function that checks the number of dimensions and also checks that the 3rd dimension has a size of 3 or 4.
yes dict learning also support dense data currently
In *the* non-metric version
I would add the ref to the smacof paper here too.
On 10/10/2012 08:07 PM, Lars Buitinck wrote:
>
> In sklearn/cluster/_k_means.pyx:
>
> > +
> > +    n_clusters: int
> > +        Number of desired clusters
> > +
> > +    distances: array-like, shape (n_samples)
> > +        Distance to closest cluster for each sample.
> > +
> > +    Returns
> > +    -------
> > +    centers: array, shape (n_clusters, n_features)
> > +        The resulting centers
> > +    """
> > +    cdef int n_samples, n_features
> > +    n_samples = X.shape[0]
> > +    n_features = X.shape[1]
> > +    cdef int i, j, c
>
> Hint for next time: you don't need to declare loop variables when 
> iterating over |xrange(N)| where |N| is declared as an integral C type.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/scikit-learn/scikit-learn/pull/1232/files#r1810567>.
>
I thought so too, but then cython -a told me otherwise (or maybe I did 
something else wrong)
Fixed in ndawe#16.
покажи структуру данных которая получается в итоге
It is automatically set, if not provided.

But post-install scripts can install additional crap, generate files, uncompress stuff, etc. So that we also need a way to provide this info manually.
In IntelliJ, you can easily modify a prior commit (even if it is already pushed in a remote repo) as long as you use gitbash to force push to your personal fork.

It is how my commits come out flawlessly now in formatting....I can make sure things work well in my personal fork then push to SpoutDev when I am satisfied.
Oh okay, the case should be where it's used in that case.
Should be mentionned that the 2nd param is ignored.  Although in this case as you'll be adding support in the part 2, it should be fine for now.
how about:

``` ruby
        tracking_numbers = REXML::XPath.match(document, "//FulfillmentShipmentPackage/member/TrackingNumber")
        if tracking_numbers.present?
          order_id = REXML::XPath.first(document, "//FulfillmentOrder/SellerFulfillmentOrderId").text.strip
          response[:tracking_numbers][order_id] = tracking_numbers.join(',')
        end
```
It could be moved to deftype - that atom is also defined there.
getLogs handles backward compatibility..

So you should should be able to common of this up.
That's what thor slaves do.
They call the IDaFileSrvHook (not part of 'dalienv')

They do:
> IDaFileSrvHook *daFileSrvHook = queryDaFileSrvHook();
> if (daFileSrvHook) // probably always installed
>     daFileSrvHook->addSubnetFilters(globals->queryPropTree("NAS"), NULL);

See thslavemain.cpp in this commit
Don't you need to add some quotes here as well?
Should I verify whether the signature param is not None, or should I add it as a parameter in the function definition? I.e.: 

``def get(self, signature, **kwargs):``
or
``if kwargs.get('signature') is None: raise TypeError``
?
I believe the save logic of this sorts of stuff should be done in the view and not the form. It was the wrong layer in Pinax before that we used because it was overridable by the user in some way. Now that we've moved to class-based views we can put the logic where it belongs.
addons we do not need anymore. Those are included by default in Mozmill 2.0.
`value &&` looks optional here
'by' => 'ad' makes more sense.
I think this is a case where I'd rather be less DRY.

`import cpp` just for CONFIG seems wasteful. Also we may want this to differ some day.
Do you think this is a little dangerous? Maybe something a little less likely to be accidentally be a real user value?
@christianv @yuhunglin  !! works nicely, thanks for the tip. 
I was not able to get this working in the HTML, nor does filter seem to take an arg to reverse its truthiness, so I'm forced to have two methods afaict.
Only getting every 30th response?  What's the reasoning here?
if !success

Or if not success
(Sorry if this ends up being a duplicate, I thought I posted but it doesn't show up)

I don't quite understand point 3; integer dtypes can span the entire range of values. I more or less agree with the rest of your points, but I think the user should decide what to do with float images that exceed [-1..1]; e.g. clip vs shrink intensity range. Also, isn't `np.clip` much slower than `np.max` and `np.min` for large arrays.

That said, I'm not opposed to add a parameter to skip the range check. Perhaps `clip_float` which should default to `False`. (That doesn't really describe the fact that it skips the check, though.) Also, I think that it should raise an error rather than a warning. Do you have any thoughts on this @stefanv?
Suggestion:

    Normalized moments (translation and scale invariant) up to 3rd
    order.
        nu_ji = mu_ji / m_00^[(i+j)/2 + 1]
    where `m_00` is the zeroth moment (i.e. intensity-weighted area).

I'm not certain that last statement is accurate.
nuked it.
@bagnell any comments on the new `Quaternion` functions?
What exactly does this file do?

Even though this is third-party code, we can't put it in the third-party directory because of web workers?
It is because of the change importScriptURI → mw.loader.load, which currently breaks scripts where the wgServer is not used.
We should validate if the parsing was successful
It could be?

    return corsConfig.isCorsSupportEnabled() && RequestUtils.extractMethod(request).equals(RequestMethod.OPTIONS) || delegate.hasRouteFor(request);
For consistency, I'd say:

The Activity object is not created...
Add paragraph break after introducing the concept of "wrapping" and before describing the wrap = true case. Here's my suggestion:

```
*   `horizontal`. Horizontal layouts have different behavior depending on whether wrapping is enabled. Wrapping is enabled
      by default (the `wrap` property is `true`).

    With wrapping enabled, children are laid out horizontally from left to right, _in rows_. If a child requires more horizontal
    space than exists in the current row, it is wrapped to a new row. The height of each row is equal to the maximum height of
    the children in that row.
```
This code will fail if the user uses "-r" twice. Perhaps you should loop over the args, and check whether they are present in a set containing ['-r', '-f', '-rf', '-fr'].
Pimp: `authString.drop(6).binaryBase64Decoded`
Please explain the reason for `clone`ing the schema object.
Ah, I see.  You're making sure the parser function `empty()` is loaded.
Added in e2729227266ae1c844c21f8df4205d11ec2f0726.

There is no lint yet. I'm working towards fixing style issues and I will add lint when there are less issues, because currently there are too many of them.
It should probably be surrounded by the ifdef too
Can you use `destroy` (from the `object` module) instead? `delete` will be deprecated eventually, so let's not add more of it.
I think we don't need to do this. When Walter completed his work on purity inference, there was talk that qualifier inference would be to follow. If we move forward that way, we'll need to undo all this. So let's wait and see what happens on the qualifier inference front.
Same about "Pred".
setVideoDeviceID  is the user facing API while setSelectedVideoDevice is really an internal method, so perhaps i should bring it into the .m file
Just the method, not the property. :)
faghat yekam css ba mokhalafat ezafe kardam va ye class 'pre'.
nemidoonam chera ama alan hodode 2 haftas ke shaba khabam nemibare.
rasi chera to talk nisin? :D
Pourquoi t'as enlevé la notice de copyright ?
Working on that now, will report back soon.
https://www.relishapp.com/rspec/rspec-expectations/v/2-11/docs/built-in-matchers/yield-matchers
Not failing just looked like it should be.
Please change this so that it returns a tupple w/ the state instead of a throw
It is not necessary, I just copied/pasted a configuration from the framework bundle, and found on the way that it might be useful for people who just want to define a single doctype.
Whatever, I'll remove it.
Ich würde den AccountsReceivable solange auf exporting lassen, bis du sicher bist, dass die Einträge in einer Accounting-Software erfolgreich importiert sind.... So dass man im Falle des Falles weiss, welche Einträge erneut exportiert werden müssten....
so if read==true the writeSkewInCommitTime and the value hasn't change (this is the case when read==true) then writeSkewCheck will happen for this entry, or do I miss something?
Changes here should adhere to the style guide
I suggest storing this setting in transurlvania's settings module. Then you can avoid doing the getattr stuff here.

If this behaviour here is the only thing that uses "TRANSURLVANIA_DEFAULT_LANG", I think it might need a more specific name. Something like TRANSURLVANIA_NO_PREFIX_LANG
Small thing, but I'd name the last parameter `step_data`, as `step_list` could be construed as a list of actual `CaseStep` objects. Since one parameter here is an actual model instance and the other is just raw data, the names should make that clearer.
I feel like it should be shortened to something like: print HL7TemplateUtils.evaluateHL7TemplateByName("Generic PV1", [encounter:encounter, encounterIndex:encounterIndex])
Right; I just prefer to have a positive test (something was called as expected) rather than just a negative test.

Merging
I see now - they are populated indirectly via the static names you changed previously - I was thinking reflection was being used.
Nit: The indenting in the new lines is different from the old lines. Would be good to keep consistent.
the issure Id(s) are in the description.
Change the setting from `target` to `server`. No space between `(` and `{`.
Hm. Looks like I accidentally removed the RuntimeException.
It would be nice to have a single version of this class that we can reuse, I've found myself copy/pasting most of this class around a few times too (although I often tweak it slightly each time).
This is really gross. Just pick a corner and alternate drawing a line, then an arc, then a line, then an arc. It should be 8 lines + a bit of header.
It is not a requirement but I have done so many translations the last weeks and I would say that it can be seen as "best practice".
Indent even if it's just one statement, ie, 
`if condition:
`    do_stuff_here_even_if_its_just_one_line
`
I have no clue why this is here ... it can be removed.  I think it was picked up when we were casting responses as Integer to test them, but it's not necessary anymore (and we would want to use the native Java Integer class anyways).
Sorry. I think there is a problem with GitHub's Windows Tool. The diff looks fine locally, but gets messed up as soon as I sync it.

Maybe you could do the change yourself. I changed lines 196 to 198:
            BigInteger chunkSize = maxId.subtract(minId).add(BigInteger.ONE).
                divide(BigInteger.valueOf(chunks)).add(BigInteger.ONE);
            chunks = maxId.subtract(minId).add(BigInteger.ONE).divide(chunkSize).add(BigInteger.ONE).intValue();

- When calculating the chunkSize the last brace was at the wrong place. You can compare it with cascading-dbmigrate, it's calculation is different.
- And chunks needs to be increased by one. Otherwise the maxId won't be read in some cases. This is also different in cascading-dbmigrate.
the strings "false" and "true" should not be valid inputs.
Might say, "Identical to {@link #createTitaniumFile(String[], boolean)}, except that the path is passed as a single string instead of an array of path components."
Should be "test, development, or production."
Because these are `unused` parameters for a non-public API, they can be safely removed.
both; the engineering knowledge base (ekb) is a technical component and 
a concept; a part of the ekb is the registry you're talking about

On 02/18/2012 05:16 PM, ChristophPrybila wrote:
>> +                    registry. The viewed sourcecode is now splitted (semantically) into
>> +                    modelobjects, each modelobject representing a single method within the
>> +                    sourcecode.</emphasis></para>
>> +</example>
>> +<para>Every tool offers different views on data but, as assumed, works on data with
>> +                related structure. Every tool organizes data it is working on in his own<emphasis
>> +                    role="bold">toolenviroment</emphasis>. To participate in crossprogram linking, a
>> +                tool is associated with an appropriate model. How modelobjects of ones tool´s
>> +                associated model are connected with the toolenviroment is in the concern of the tool
>> +                itself. Every tool must be able to open or retrieve modelobjects by their
>> +                identifiers.</para>
>> +<para>To enable distributed linking between semantically related datapoints, which may
>> +                also be viewed by different tools, a central component, called<emphasis role="bold"
>> +>registry</emphasis>, is needed. This registry manages the defined models, the
>> +                association of tools to models and the<emphasis role="bold">mapping</emphasis>
>> +                between modelobjects from different models. This mapping is achieved by<emphasis
> I dont quite get it.
> Are you talking about a component in openengsb?
> Or a technical term?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openengsb/openengsb-framework/pull/639/files#r464870
I'm not sure this chunk of css belongs to a file called "onemenu". It's related to the cms plugin, not this special template.
from here to line 106:

please, comment on what this regex does. Each separatelly.
ah, you are right
So in webworks whitelisting we allow uris with a port to match the same uri without a port. Originally this code would strip the port off and then start trying to find a match

This change was needed in the case where a user specifies a port in their authority and is using a subdomain
We can't easily change the trigger without changing other parts of the event framework as well. I

 think it would be better to have a publicly exposed function that goes from JSONtoBlob and to use that (if that is what we are converting).
Please use toHaveBeenCalledWith vs mostRecentCall
So would you want to update the quest here to have the Errors decorator class be help-inline?