labofsahil/github-event-dataset-2012 huggingface dataset:
Use the *_size attributes defined here:
https://github.com/enthought/enaml/blob/master/enaml/components/window.py

But also read the comments for the overridden methods here:
https://github.com/enthought/enaml/blob/master/enaml/components/main_window.py

MainWindow does not respect the *_size_default attributes.
I'm not sure what's best - not catching stdout/stderr is simple and means they're always visible, but it's awkward if output turns up after a subsequent prompt.

I think this hook is currently only used from the terminal frontend, but if it was ever called from the Qt console/notebook, stdout/stderr of the subprocess wouldn't automatically be forwarded. I don't think you need to worry about that at present, though.
Quotes are not required in this case because the value "_blank" contains no spaces, but I can add quotes if you want.
Should it also do the `metricName.length() <= 255` check here as well, rather than failing in `build()`?
maybe add 

\@see ExtensionClient#getExtensionByAlias
\@see ExtensionNamespaces# (extended status?)
そうなんです。
実装の方でも使っていて、OmuniAuthが使うのをやめた時はこけてほしいので書かないままがいいと思います。
便利なので他で使うなら勿論入れちゃっていいと思いますが。
If the blob could be cached and the later call to load the blob didn't repeat work, I don't feel too bad about an inefficiency of skipping a too large blob, particularly since it may avoid loading two copies of that blob.

I think git has something vaguely similar to this, but I'm not 100% certain. I'll stud the code more. Is there no way to get the size of the blob without unpacking all the data? Who designed this shit?! :trollface:
It's consistent. When `url()` is used the `/$` is added, and when `page()` is used it isn't because the `page` function adds it.
Fairly certain you don't need to mark this safe, so please don't. If you *do* have to use |safe, then this would be a bug in nuggets/safe_django_forms.
I would rather be explicit about what elements we want instead of `*`. Is it too many?
Just a nitpick, can you indent this as it was before?
Pretty sure you can instead do `os.path.join(*rparts)`.
This line (and the comment before it) are not needed. This was part of my original selection implementation, but is no longer required since the selection is a div rather than a span.
We cannot rely on these methods as they are private in CodeIgniter 3.0-dev, which is what 2.2/develop is using.

This is the sort of reason why changes need to go into the NEXT version, not the current version. 2.1 is just for tweaks and bugfixes, newness goes into 2.2/develop.
If it's hidden, why is in a div as an input? :)
Very bad change.
They probably wanted to preserve my original browser js. We are no longer using this so the .backup can be removed, I see no reason for a revert on this code if its the only ui implementation that supports the multiconnection feature.
прибрав через помилку Doctrine, але зараз чомусь не можу її відтворити.. поверну назад
yes, they would
remove one space here
`return $this->isRegex($str) ? $str : '/'.preg_quote($str, '/').'/';`
I removed it completely as it was not really useful anyway.
I thought I was going to change the file, and saw that I had not yet added my name :) Anyway, I have made many small feature additions in that file that should warrant the tag.

If possible, I'd like to first talk about the general idea of this patch though. If we agree on that, we can also talk about when @author tags are justified, and whether this one (or others) need to be removed.
missing empty line before the return statement
the parameters are in the wrong order (the expected comes first)
Why not as an external fixture ?
More copies.
maybe `if ([utc isMemberOfClass:[NSString class]]) {` is better
nib ro yadet rafte ezafe koni
Actually no for two reasons:
the FreemanChain class encodes 4-connected curves, this class works with 8-connected ones;
turnedCode() compute the code after a rotation of pi/2 only.
It would indeed be nice if the FreemanChain class could also encode 8-connected curves...

Le 25 sept. 2012 à 17:19, David Coeurjolly a écrit :

> In src/DGtal/geometry/curves/FrechetShortcut.h:
> 
> > +		  d=5;
> > +	    }
> > +	}
> > +      return d;
> > +      
> > +    }
> > +    
> > +      /**
> > +	 Rotate the chain code d to put it in the frame where the octant 'quad' is
> > +	 treated as the first octant.
> > +	 @param d a chain code
> > +	 @param quad the octant
> > +	 @return an int (the new chain code)
> > +      */
> > +      
> > +      static int rot(int d, int quad)
> Would it be a duplicate of FreemanChain::turnedCode() ?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 

------------------
Isabelle Sivignon - Chargée de recherche CNRS
GIPSA-lab UMR 5216 
11 rue des mathématiques
Grenoble Campus
BP 46
F- 38402 SAINT MARTIN D'HERES Cedex
Tel : (+33) [0]4 76 57 43 74
http://www.gipsa-lab.inpg.fr/~isabelle.sivignon
Thing is criteria is not "optional" it is required and this call has no optional parameters.
Add convenience methods withConsistency(ConsistencyLevel leve) to set both read and write consistency.
I suggest adding that src_crs is an instance of ```:class:`cartopy.crs.CRS` ```.
(no action required, perhaps raise another ticket)
It would be good if the repeated code was refactored into a function because it would highlight the fact that only the projection is different.
Does it occur other than with {{each}}? I almost always end up with {{collection}} instead.
This is not thread safe, as org.jboss.as.ee.component.BasicComponent#waitForComponentStart() can be called the synchronized lock will be released when it calls this.wait(), and multiple threads can create the instance. 

You should copy the code out of org.jboss.as.ejb3.component.singleton.SingletonComponent to get around this.
I believe that OSGi JPA enterprise is supposed to call PersistenceProvider.createEntityManagerFactory() instead of PersistenceProvider.createContainerEntityManagerFactory().  Can you clarify what is actually expected?  I'm looking at sections 127.3.3 (client bundles) and 127.3.4 (custom configured entity manager).  Section 127.2.3 talks about how PersistenceProvider.createContainerEntityManagerFactory() is for EE managed mode.
No tests for when focus shouldn't happen?
Also here, yes.
that style should be put in a CSS class or use the hidden attribute
I'm sure somebody skilled in shell would have something better than the `grep` to suggest but I'm not that somebody.
nit: add a line break after this line
This is wrong: when the API returns 0 result --> set cache value represent that result, not delete the cache.
Nit: please leave an empty line at the end of each file.
@timdream The function will be called again. AlarmList.init() will do that when 'localized' event callback.
i -> launchPath ?
?

Cannot understand you comment (typo?) nor the code here.
XPATH old friend. I'm wondering if we can avoid Xpath usage, I recall that evaluating Xpath expressions was not really efficient. but I prefer to hear here from @vingtetun opinion.
Is this check generic enough?
You should use 'value' instead of 'name' (you coud use both, but 'value' should contains data that you want to retrieve)
looks like you're using this kind of indentation on purpose… please don’t. ;-)
add a note about what enable means here (per our IRC conversation)
With the new plugin structure, would it make sense to move requirements like this into the plugin itself for greater modularity? i.e. since now RMagick is only used with the loltext plugin it could be required in `/plugins/loltext.rb` ?
We can leave this method as-is for now, but ideally we should maybe wrap this in a class (see the Command class).
On Thu, Oct 11, 2012 at 06:00:43AM -0700, Giuseppe Sucameli wrote:
> > @@ -460,13 +457,13 @@ QString QgsPythonUtilsImpl::homePythonPath()
> >    }
> >    else
> >    {
> > -    return '"' + settingsDir + "python\"";
> > +    return settingsDir + "python";
> 
> This change seems to break something.
> 
> Have a look at https://github.com/qgis/Quantum-GIS/blob/master/src/python/qgspythonutilsimpl.cpp#L459:
> both the if and else statements return a string containing python code,
> in your code the else statement returns a path as string instead.

Yes, but the callers add the quotes when needed, doesn't them ?
But then I guess the xpanduser part wouldn't want to be quoted.
The problem was with https://github.com/qgis/Quantum-GIS/blob/master/src/python/qgspythonutilsimpl.cpp#L86
resulting in a weird sys.path (containing + in it).
no need for require, since they are imported...
Думаю, лучше было-бы сказать "Невозможно создать таксон". Я понимаю, что в англ. варинате немного не так, но мне кажеться так будет правильней. @insside, что скажите?
Is there a reason that `removeOldestNotification` is public and `_displayNotification` and `_queueNotification` are private?
I wonder if this would be cleaner to just overload the add_image_tag function like so:
add_image_tag(TagLib::MPEG::File* f, string tag_name, string image_file_path, unsigned char image_type)

C++ can figure out what method to select based on the signature. 

This could also be true of all the *mp3 functions you just added. I will get this to compile in my build env and see what works.
````js
        changeImageSource: function(url, image) {
            WDN.log(url);

            var myFunction = function() {
                image.src = img.src;
            };
            var img = new Image();
            img.addEventListener('load', myFunction, false);
            img.src = url;

        },
````
Error: Too many arguments supplied for building AppScale. Building AppScale requires either 0 args for the full version, or 1 arg for AppScale Lite (support for one database).
Don't print this - log it!
do you think it would make sense to switch to Jetty 7 oder 8? well, maybe after the release.. (i remember there were some incompatibilities with milton back then?)
if no ChatComponent comes up... sure
Even though this is using `||=`, this will still reload the JSON for every request.  Might be worth moving the JSON parsing into a class-level method, and memoizing it in there.
Instead of a multi-option Param, it seems this functionality could be implemented much cleaner using classes.  Then if/when test modules need to override, they can define a subclass to do so.  Similar to my suggestion above regarding pre-process.
this will mutate array in place which is probably shitty. i'd do `Backbone.history.handlers.slice().reverse()`
Because the code changed :) Fixed.
I'd rather have this use the ProcessExecutor class as well, for consistency and because it works around so many quirks.
Why this change? That seems wrong.
Can you revert the RC=r change? It's not allowed by version_compare, and has a fairly high likeliness of conflicting with svn's revisions. Also RC is already shortened from release candidate, no need to cut it off to one char.
Yeah, I realized it was the lock. I guess it changed because the deps are branches/a git-subtree split.

So just for the record – I did this: I ran `./composer.phar update` after I installed a few weeks back. You're right, updating should not be necessary hence the lock file should not change.

Unless of course I added a dependency (, which I didn't do). But if I did, what would be the _correct_ workflow then?

Anyway, I just pushed the lock from master.
implemented with bab4b9875c34846f62c3321decaf74cc28536db3
Add base: concat
Phonon is part of Qt
у нас не просто мд. но ок, согласен, лучше text/html отдать
do we still need `twttr` now we killed it in articles?
Struct name would be Nt3gdStreamDevConf
Actually in other places we use a different loop, like here https://github.com/inliniac/suricata/blob/master/src/app-layer-smb.c#L1313

Coverity doesn't like that one either.
When concatenate multiple JS files appending file content like `(...)` to another file that does not ends with `;` will cause an issues as it will be interpreted as a call to previous expression. So I kind of refined module wrapper to workaround that (I used ! ternary operator before but people complained that it's not obvious so I changed to this now). That being said I can remove `;` prefix if you want as I don't expect users of this code to do such concatenations.
According to guidelines you need `\n` after `{` if function has more then one line.
I've added a comment, that's to help finding which code use deprecated stuff.
There should be tests for each of those cases as well.
hmm.   don't we *only* need to check whether the session is expired if a user is logged in?  not all of our apis require authentication.  one such case is the `authenticate` call.
I'd rename this to just `handle_project`
Should this be a `goo.gl` link?
Why, yes, I _do_ like Git.
Yes, but it's often used to convert to boolean value.
I agree, that cleanup is needed and I'm happy to work on that.

Only usage of portname I had found so far was this one. In related dialog corresponding input field uses chan_mode. So, what is correct term for it? Dialog item is named Port name and is connected to chan_mode.
This is not good -- it's an API change. I'm okay with adding a method, but not with _removing_ one.
you don't need the ternary operator here. Just write `return (boolean) (ord($byte) + 1) % 2)`
On the above, put the braces on separate lines, even if the method body remains empty.
Not true. It can go either way and is recognised in IDE's in both forms. [Docs](http://www.phpdoc.org/docs/latest/for-users/tags/var.html) won't help us here though
If the param does not occur in the signature, remove it from the docblock.
You may be able to combine the ones that are duplicates so the second argument is either '.spinner-up, .spinner-down' or maybe '.spinner-buttons button'.  Looks like this may only apply to the ones that call stopSpin
This is looking pretty good.  Looking forward to when it's working

One small point -- I wouldn't hard code the token into the into the initializer through the ENV hash.  This is assuming a particular environment.  In most other gems of this nature, the token is retrieved outside the gem and passed in through the initializer.  

Regarding the question about creating a remote object in the initializer...  When the connection object is created, it is just initialized -- no connection is made to the Zoho servers on creation.  The remote connection is only made once the call method is invoked.   
Kenton White

On Sep 19, 2012, at 8:59 PM, Stan <notifications@github.com> wrote:

> In lib/zohoho/lead.rb:
> 
> > @@ -0,0 +1,65 @@
> > +module Zohoho
> > +  require 'httparty'
> > +  require 'json'
> > +  require 'xmlsimple'
> > +  require 'date'
> > +  require 'open-uri'
> > +
> > +  
> > +  class Lead
> > +    include HTTParty
> > +    attr_accessor :id, :owner_id, :owner_name, :first_name, :last_name, :email, :source
> > +
> > +    def initialize(email = nil, info = {})
> > +      @conn = Zohoho::Connection.new 'CRM', ENV['TOKEN']
> > +      if email
> Not sure if it's a good idea to create a remote object in the initializer...?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Need a [StringLength(20)] here so that this doesn't become an nvarchar(max).
we should check if LOCAL_ADDR is set before returning it. Maybe something like  `if (empty($this->_env['SERVER_ADDR']) && !empty($this->_env['LOCAL_ADDR'])`
This doesn't conform with PEP8. Please go to the next line and indent :)
Oh, and I first tried the WHILE statement, as you suggested, but the program would go into a loop, maybe due to that SRAND initial statement? Once it found that the result is below 1.5, calling CONST FLOAT r = RandDouble() in the WHILE loop would not kick it out of the loop. Not sure why, but I solved it by going to the IF statement and the increment approach.
this could be `return (from && year < from) || (to && year > to)`
@gerald end date ata ung kelangan dito. ngayon ko lang napansin, namerge ko na. what do you think?
One thing I see a lot in our code is what to me seems like overuse of dict.get().  If you're dealing with an external API then it often makes sense to do "defensive coding" and ensure default values if some field is missing, but if you're creating an abstraction you have control over then it's better, from the perspective of you or someone else reading and maintaining the code at a later point, to explicitly require the indicators key to be present in the dictionary by doing section['indicators'].  I mean, when would it ever not be?  And, it's easier to read at a glance.
This shouldn't be called here, it should be it's own before filter.
whats changed here? line endings?
Not sure - people will look for examples of how to use @EJB... maybe we could add another property injected via CDI? Or an alternative page with same functionality but with @Inject?
Is this OK:  For JBoss AS 7 or JBoss Enterprise Application Platform 6 (Maven user settings configured)
This code opens all the footers of all the files for the given input. Which is needed to generate InputSplits for example. For this the client will contact many slaves in the cluster to read the data. In practice, some slaves are slow or not responding and the client will wait until the connection times out to connect to another slave and access a different replica. Using threads allows the whole thing to go much faster by reading other files while some of the threads are hung on unresponsive slaves.
No need for the space before `filtered`.
Docstrings are nice.
そうなんですか・・・！
user_specではとりあえずomniauthに適当なhashを入れています
I think alphabetization is good when things are unrelated.  In computing, it is generally accepted that width precedes height.  Making it height then width just for the sake of alphabetization muddies the waters set forth by years of precedent in computing, the english language, and humanity itself.
This comment would be better as:

   Initialize and run a Snaplet. The 'serveSnaplet' function parses command-line arguments, runs the given Snaplet initializer, and starts an HTTP server running the Snaplet's toplevel 'Handler'.
For sure. Pushing them tomorrow afternoon.
change "cochalos"  to other word
we used to have some sprites to limit the number of HTTP requests (basic front end optimization trick), it would be nice to keep on using sprites.
IMO, the previous algo was more readable then the current. There is maybe much more `if` but you don't need to understand the loop. What do you think others? ping @tmartin @clemherreman
Nice, unsorted to make a point about it sorting them :)
не понятно, что тут происходит и комментарий увы не разъясняет
I would make these camel case for all variables, I think I'm going to put them in that way just so things are easier to remember- more consistent.
To the best of my knowledge tehre isn't a join,few - did you mean ,LOOKUP?
Makes sense, just pointing out that it's a change. I'm happy to have this merged in now.
Fair. We can remove it then. That said, if it keeps people from committing .DS_Store files, I'd rather see this file than have people commit such files.
you don't need the single quotes. `:hidden?` works fine.
Probably because we're shelling out to bundler.
I'd prefer it if modules didn't all have their own internal set of util functions. There is an extension for underscore that offers string utilities like this. However, I'm not sure that we need this at all, as relying on a string ending with something seems brittle.
I think there was some interest by @nicolaasmatthijs to not always have this call cached. Can you confirm nico?
This should go above OAE. Global 3rd party tools should be required above OAE-specific modules.
The only ones I can think of are file uploads and uploading of preview images (which is only in there for testing the endpoints that the PP will use.)
However, splitting it out in a _RestRequest and _RestMultipartRequest won't result in less code. You'd have to split out the stuff like getting a cookiejar, setting the host header, handling the response, etc..

I tried making everything multipart but than we can't do "empty" POST requests as that is not allowed.

I agree that it's a bit heavy on the number of lines of code, but I personally feel that it's still maintainable.
I'd prefer it if this didn't declare an internal jQuery function and we just replaced the type checks with native underscore functions. That will compact this function quite a bit.
The schema for this doesn't feel quite right. I'd like to suggest the following alternative:

1) The principals CF continues to exists as is, but contains the internal principal id and no longer holds the password.
Not holding the password in the principals CF makes it possible for 2 internal login id to point to the same account. It even makes it possible for a user on tenant A and a user on tenant B to be the same person

2) A second column family "PrincipalLoginIds" is created


```
|                         loginId                         |                         password                         |                         internalId                         |
------------------------------------------------------------------------------------------------------------------------------------------------------
|             cam:internal:nm417             |                    <passwordhash>                 |                    u:cam:XYYzaBc                  |
|            cam:internal:simong             |                    <passwordhash>                 |                    u:cam:BBNaSzx                  |
|           cam:twitter:nicolaasm            |                    <passwordhash>                 |                    u:cam:XYYzaBc                  |
|               gt:internal:nm417               |                    <passwordhash>                 |                        u:gt:PLopSA                     |
```

When a login is done, we prepend the current tenant and the strategy to the login id. This will avoid conflicts and seems to be the most flexible solution.

A third CF could be made where we store the login ids associated to an internal user, but I don't think that's necessary yet.
Why looking for the word ``macro`` (or better ``makro``, read my next comment) isn't enough? (I don't see why checking ``EQUALS``)
This could be limited to ``NumberFormatException``.
а з дефолтним параметром для knp_paginator_bundle не вийшло?
Given that you already test no params at the end, I'll change this to test params support since it doesn't seem like any other case tests that.
I tend to agree with [Crockford's style guide](http://javascript.crockford.com/code.html) here:

> If a function literal is anonymous, there should be one space between the word function and the `(` (left parenthesis). If the space is omitted, then it can appear that the function's name is function, which is an incorrect reading.

Plus, I like the symmetry of the space before and after the `()`. Style counts, but I have been known to be a little anal about it (like, all the time).

That said, it's your code, so I'll follow your style like a good contributor. :)
This should be moved into the webui src as its part of the webserver code. But I think there are some more general simplifications here I'll address in a bit.
let's open a new issue to discuss this it's way too cluttered in here
Ok, so why there's a getDefaults() method?
`//`?
If we're going to split this one into two lines - put an extra tab on the second line, have the operator in it as well, and probably the `isset()` too:

    if (class_exists('CI_DB') && $return === FALSE && $query_builder === NULL
        && isset($this->CI->db) && is_object($this->CI->db))
Почему так много людей делают кучу while'ов, внутри которых генерят случайные индексы и дальше проверяют, не использовался ли он (индекс) уже. Неужели не чувствуете, что чем больше процентов надо будет менять, тем сложнее и сложнее будет сгенерировать - "попасть" в неиспользуемое число? Представьте, что 99 индексов (числа от 0 до 98) уже выпали, и осталось вот рандомом получить индекс 100... С большой вероятностью это далеко не с первого раза произойдёт ага? А если бы длина ДНК была 10^6 и надо бы было мутировать 99% сколько бы прогонов циклов там было примерно представляете?
Не знаю, меня самого мучает этот вопрос, не создает ли он случайно за каждый проход новый указатель. Поэтому спросил.
Мне кажется, что у Вас здесь параметр функции arc4random_uniform должен быть просто kLengthOfDNA. иначе результат от 0..98 при размере ДНК = 100.
а почему бы этот вот код с кейсом не вынести в отдельную функцию, а не писать одно и то же 2 раза в разных модулях?
Moved the name creation for modifier nodes into a separate function
```
var d1 = document.createElement('x-dropdown'); // onCreate()
var d2 = document.createElement('x-dropdown'); // onCreate()

document.body.appendChild(d1);  // onInsert()
document.body.appendChild(d2);  // onInsert()

```


If you want something to only be ran once then you can put it inside a closure at the top of your dropdown.js
It's not entirely clear (from the documentation) what it means to pass in `immediateScheduler` here, since we don't clarify that we send from the main run loop.
Is `self` necessary here?

Also, you probably want to use `build` in favor of `create`.
config := &tls.Config{
   Certificates: ...
   ...
}
@nulltoken Do these need to be ref parameters? If they're not, you don't have to enter the types in the lambda over in `Proxy.git_note_foreach`.
I assume this means we need to modify the packaging scripts to copy the layout files when making the egg.
indent + images- ?
ok it makes perfect sense. Maybe a note of the code could help. Feel
free to merge
raw._preloaded = True, my comment above was implemented in #146
language: 7 should be spelled out as 'seven' in this and the next line. When referenced as a term "7-layer", like above, it's acceptable to use the numeral.
This is an API change that isn't backwards compatible.  I might prefer adding an overloaded constructor.  That preserves backwards compatibility, and makes it so that users who aren't using authentication don't need to pass in a null auth object.
Seems superfluous
Actually, I find that `virtual` makes me think of C++ too much.
0 docs
Тем более не факт, что оно не в shared_ptr будет
А если один фиг - то зачем?
h1 to replace
Cuál es el propósito de este Stopwatch? Es decir, sé que se devuelven los valores por JSON, pero... para qué?
I like to keep the puts clean for tests, so in this case I might:

1. Rails.logger.debug (and then check the log/test.log)
2. Remove the puts after they serve their purpose
now should use the parent ::pause method
lib/vendor is in the source tree
@ajoslin Andy, where are show.ui and hide.ui called from? bootstrap? I'm a little confused.
With mpl v1.2 and later, we can configure the axes (or subplot) creation using the projection parameter.

If the value of projection parameters is an object with a "_as_mpl_axes" method, the method will be called, which should return a tuple of the Axes class to create and a dictionary of additional parameters, and these will be used to create an Axes or a Subplot.

For example, if the WCS object implements following method, 

```python
def _as_mpl_axes(self):
    return WCSAxes, dict(wcs=self)
```

We should be able to do

```python
plt.subplot(111, projection=wcs.WCS('image.fits')
```

And I think this approach is better than having our own wcs.subplot or wcs.axes function.
The parameter defaults to chocolatey already: 
  [string] $source = 'http://chocolatey.org/' 

Is this not sufficient?  Just let me know if this needs to be backed out.
Yes, make a copy then use the copy of the shared_ptr. That's how shared_ptr was designed to be used IIUC. :smiley: 

Compare the cost of this with the reader-writer lock. I don't think you're going to see much of a difference as both of them will be incrementing an atomic count anyway. :grinning:
Yea, it is not actually.  I was working with Gradle for a long time and was using idea plugin(as IDEA integration with gradle projects is not that great) for it and completly forgot that the IDEA can open Maven projects from the top-level pom. Apologies.
This might be due to the funky naming for Partition.  A Partition holds groups of data points, which themselves could each be described as separate "partitions",  In my terminology, a Partition is a way to separate a set of datapoints into any number of distinct points.     So any Partition comparison is comparing the set of data groups within each Partition.
Mostly placeholder, it just has gc-api and associated files.

Let's not pull it out, its part of the Nova project so it makes sense there.
I wouldn't think so, since it's the same ID we present in an index call, right?
must be broken to comment it out.
Why produce a duplicate of org.infinispan.test.AbstractCacheTest? You can depend on the test module of the core and avoid duplicating code...
These have to be disabled for now. I might not be able to fix whatever breaks this test today and do not want to keep these changes stashed either. That's why I'm including them.
Perhaps this is a good use for something other than `pray`, since we're asserting that the public API is being used correctly.  Not sure what the best thing is to do here, but `pray()` is a good start.
This isn't correct. `char` should not be considered convertible to `unichar` under any circumstances. This should be using `-initWithUTF8String:`.
As above, I think `options:` is more conventional than `withOptions:`.
Remove leading space on this line.
Thanks Eric. I have implemented your suggestion.
same-line comments should have at least 2 spaces before the ``#`` caracter
+1

On Tue, Aug 7, 2012 at 4:31 PM, François Thillen
<notifications@github.com>wrote:

> In testproject/Protocols/IProtocol.cs:
>
> > +        /// </summary>
> > +        int SocketNumber { get; set; }
> > +        /// <summary>
> > +        /// Convert a message to a the indicated Protocol
> > +        /// </summary>
> > +        /// <param name="message">Message to convert</param>
> > +        /// <returns>IProtocol</returns>
> > +        IProtocol ConvertToProtocol(Object message);
> > +        /// <summary>
> > +        /// Convert a byte message to a the indicated Protocol
> > +        /// </summary>
> > +        /// <param name="message">Message to convert in bytes</param>
> > +        /// <returns>IProtocol</returns>
> > +        IProtocol ConvertToProtocol(Byte[] message);
> > +        /// <summary>
> > +        /// Compaires if a protocol is the same.
>
> I think it is better to inherite from IComparible. WDYT?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openengsb/loom-csharp/pull/23/files#r1321797>.
>
>
Why did you delete the greeting?  Thats horrible.
@dooremont Could you translate the name & description to French?
Reg exp is not working properly, place more than one quotes in a text enclosing single and multiple words.
Most importantly, when an html link is added via the editor, this modifier is replacing the quotes used in the attributes, that breaks the link.
typo: "observer observer"
Not sure if this is a good idea, UX wise

http://i.imgur.com/sOyyE.png

We've ended up making the user have to select desirable options twice, because selecting the options at the top i.e approval/verified status ends up excluding everything else i.e report description, location, category etc
Nevermind this too :)
I chose /storage/downloads/ as the place to download to as it seemed the most practical.
yeah.... http://cl.ly/1n0d3z1H373r450y3o0Q
Code layout is wrong
Are you sure ``'stdClass'`` has the correct type as ``string`` is not a sub-type of ``object<Zend\Uri\Uri>``?
Are you sure ``123`` has the correct type as "integer" is not a sub-type of "string"?
The assignment to ``$bigInteger`` seems to be dead. Are you sure it is necessary, or maybe you have a typo somewhere?
указать реальные данные
I'll add it back for the SysLog class - so glad you spotted this, @LouisLandry Sorry for not being more careful.
I believe you'll still have to null `this.element` so that IE gc picks it up
I'm seeing this pattern a lot in these views.  Can `link()` get refactored to eliminate this duplication?
"(mensajes y temas que el usuario puede eliminar de otro modo)" - that doesn't really explain anything. The key word missing is "permanentemente" which probably ought to go inside the empty strong tags.
norma -> norm
This function should definitely be a `Command` class method.
if unique is removed there's no null value in database
This does not yet exist anymore.
Why removing this? Then you should also remove CSS related to this class.
This can be ReST formatted. Format accordingly. (Check how lists are done there.)
I'have tried many of generic views, most of them dont work well, so I used the most basic view.
Which one do you recommend ?
Use tuples for constant lists.
Create `common.js` and move this there.
No, have objects outside and then methods inside those objects. For now leave as it is. We will restructure code when we will have more of it.
This is ugly. Define outside function which returns iterator which provides all this data. So first it provides non-mocking value, then the rest.
I checked my other code. I think I use such style in JavaScript:
* object attributes/fields is_with_underscores (because they are often populated from JSON)
* functions and methods are camelCase (because this is not provided through JSON, but external APIs are often like that)
* classes/constructors are CamelCaseWithFirstLetterCapital (because it is more readable like this)
* global variables are camelCase (in fact I don't like this, I would go also for underscore here)
* local variables are with_underscore

What do you think? I think it is a good compromise between external code (jQuery for example) and JSON data.
We already have jQuery loaded and used in PiplMesh?
If post is edited this will switch to true and show when is displayed (similar to facebook without message history)
that is if we will enable edit options. i thought its a good idea.
Put space around `()` in anonymous functions (just the style we use also elsewhere). Please list all arguments event function gets, even if you are not using it (but maybe some developer later on will).
OK, I was thinking about it as it was in module.
That's a big 'my bad'  Fixoring...
It looks like this `FILTER_NONE` constant can also be removed.
Should log something and limit number of retries.
Sí me referie a bash por que se que tiene el comportamiento que la instrucción:

```sh
mkdir {a,b,c}
```
Crea los directorios `a`,`b`,`c`, por otra parte supongo que shells  amiagables como zh, o fish tambien lo hacen.

Desconozco que si `ksh` reconoce estos comodines, por lo que se es que `sh`  no te realiza este comportamiento y como el control durante la ejecución del _script_ la realiza `sh` decidí adaptar la sintaxis a `sh`.
Use `PadRight`.
fffffffffff....
For the change I was talking about, these two functions (`showTooltip` and `hideTooltip`) would have to be changed too add the `.show-tooltip` class to `acorn.$container`, after setting the `html()` of the tooltip.

On `hideTooltip` I don't think we would have to empty the tooltip contents with `.html("")` after hiding it, since it would always be replaced on `showTooltip`.
>   Can you post your concerns there instead?

I didn't mean to start an argument.  I was just responding to your comment
that backticks are evil.  Sometimes they are needed.  I'll shut up now.  I
know the issu
Why is the default for collision "none"?
would like to add "especially when targeting beginner/intermediate audiences". thoughts?
Doesn't dbpedia already make wikipedia references?

m.

On Fri, Sep 14, 2012 at 1:14 PM, Jose Cruz-Toledo
<notifications@github.com>wrote:

> In ns.php:
>
> > @@ -72,6 +72,7 @@ class CNamespace
> >  	'dip'       => array('name' => 'Database of Interacting Proteins','url'=>'http://dip.doe-mbi.ucla.edu/dip/Main.cgi'),
> >  	'ddbj'      => array('name' => 'DDBJ sequence database'),
> >  	'dpd'		=> array('name' => 'Health Canada Drug Product Database','url'=>'http://www.hc-sc.gc.ca/dhp-mps/prodpharma/databasdon/index-eng.php'),
> > +	'dbpedia'	=> array('name' => 'DBPedia', 'url'=>'http://dbpedia.org'),
>
> Peter has made a good point. Every resource in the DBpedia dump files (
> http://downloads.dbpedia.org) refers to an article on wikipedia by
> following the URI pattern: http://dbpedia.org/resource/Name, where "Name"
> is derived from the URL of the source Wikipedia article eg:
> http://en.wikipedia.org/wiki/Name.
>
> However, we are RDFizing the DBpedia dump and not wikipedia's. In
> following the spirit of our other Bio2RDF RDFizers, I think that its best
> to stay as true to the source's data as possible. Perhaps a good compromise
> would be to add an rdfs:seeAlso from every dbpedia resource that points to
> the corresponding wikipedia article page. How does that sound?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/micheldumontier/php-lib/pull/14/files#r1609664>.
>
>



-- 
Michel Dumontier
Associate Professor of Bioinformatics, Carleton University
Chair, W3C Semantic Web for Health Care and the Life Sciences Interest Group
http://dumontierlab.com
"if it implements method `then`."
`bool` could be more descriptive (I know this follows old code though).
I came across a problem when I tried to inherit directly from RoboHydraHeadProxy but it was certainly solvable, so yes, I agree
Perhaps instead of initialising `i` further up, make this `for (i)` loop. It keeps things closer together.
It would be nice if this were vertically aligned better (i.e. the "==" and the "=" all in the same column) -- that's a "snap style" peculiarity.
I think we settled on relative in api, absolute everywhere else. I usually debug in the source directory.
this might also be expensive if it has to be calculated very often.

add as argument and store in caller ?

When we to LOO, do we need to adjust Dom_x each time or is it better to keep it constant? The kernel, aitchison_aitken, only uses num_levels, which could be argued, I guess, should stay unchanged. 
I'm not sure what happens with the loop ``for x in Dom_x[i]`` if x_{i} is missing.
Cool, very compact.
ah no, sorry. It does not return a lazy collection indeed
Why synchronized?
jetty.version is already defined in nexus/pom.xml - is there a benefit of duplicating it here?
@LukeGT Ahh, so this was the real delivery method! Encrypt it and fail at hacking. Now you have the attention. /me leaves Palestine.
OK, I'm sold, thanks!
Thanks Joerg, sorry for my pedantry!
OK, I think it might be a good idea.  If we had similar noise patterns being applied to adjacent or nearby galaxies because of bitwise-close seeds, it seems to me that this might be an issue for variable shear fields.  Any thoughts @rmandelb ?
忙着泡妞？！
What about the "unset" parent case.  Does that work here?
change to take the ID as string, so can use in web console.
maybe change return value to be ID of new entity.
probably throw exception if it isn't a member?
change descr to:
"Replaces the entity with the given ID, if it is a member; first adds a new member, then removes this one"+("; returns XXX" ... and "throws X")
Should this really be static?
pareil la je mettrais un example ambigu qui tromperait l'ordi
pas d'espace avant les ? (pareil pour les :,!, dans tout l'article)
This should be `cocoa.id`.
Thi is really unusual header for open genesis :)
@anuradha-r I think it would have more sense to move these tests to *".find"* context and make them to look like:

```ruby
it "should have first_iteration_start_time attribute" do
  @project.respond_to?(:first_iteration_start_time).should be_true
end 
```
as this is how we check for attributes in other parts of this file. Also separate tests by a blank line.
Removed, BCC is supported in both APIs now. I had originally written that in the code before I implemented CC/BCC for the SMTP API in the library.
This is my Test Mode variable that i set in the jasmine tests.

Maybe i can do `StackMob.Storage.persist('oauth2.expires', 5000)` from the tests instead of this.
It works. This is a much better solution. I will update this PR.
Ext JS 4 does not, AFAIK.
So, on win32, stat is somewhat expensive, but if you just want to know if a file is a directory, you can already get that information just from scanning the directory entries. I added this extended readdir API to return that information.

However, it turns out in the diff code I eventually need to stat the file anyhow, so I just shifted around where the stat was done so that I don't have to do it twice, and I've ended up not using this extended readdir feature. I've left it in in case there is another use for it elsewhere, but I don't need it after all.
@tedivm 

* I agree, my question was about the "DriverInterface". One question though, could you eplain: "there are cases where there will be an object and it will still be a miss"  ?

* You assume that the TTL is global for all the items here, right ? If this is the case we might be able to retrieve it from a different place. I am not saying that always returning an object is a bad, just trying to understand.
Bluh.  I will see if I can revert that file then.  On a mobile.
I don't think the word "archived" is particularly helpful here. It *could* be archived, but the general use-case is accessing unpublished.

How about ```content_api_has_unpublished_artefact```?
i see. i didnt dare to add a model and patch project.
should i rewrite? might be better before we deploy this and need to migrate later.
Пусть сначала хотя бы так корректно заработает. Остальное поправлено.
Если будет проблема только в скорости, то думаю править исправлять все равно не только это.
Это единственный финт который я нашел, чтобы календарь нормально обрабатывал таймзону
Т.е. abc 123 будет считаться словом только abc? Это неправильно. Слово - это всё, что не разделитель/знак препинания.
Исключение здесь и файл не будет закрыт.
reuse this instance for Collections.sort(rankedSingleBenchmarkList, Collections.reverseOrder(singleBenchmarkRankingComparator));
This is exactly  what this code does, try it...
One of the browsers - I think it was Safari - was prioritizing the width and height attributes over the CSS width and height properties. As a result, the video was displayed smaller than the container and it all looked broken.
https://github.com/Youscribe/openssh-cookbook/commit/609c51370787b6badb91fea236d1fd71991ae51c

and my branch: https://github.com/Youscribe/openssh-cookbook/tree/COOK-1889v2
Sounds like a good idea ;)
Sure - they were to be deleted - just left overs from my planning. They will be gone :)
Maybe `NystroemKernelExpander` or `NystroemSampler`?
Robert is Australian and thus write British English :)
Oh, ok. then maybe embedded devices?
I'm not sure if it has an impact on this PR (since its focus is classification) but it's useful to keep in mind that from now on `SGDRegressor` needs to support multi-variable regression too.
i have added the use statement because of https://github.com/symfony-cmf/ContentBundle/pull/6/files#L1R51
I believe this files should have 'platform' in it to avoid any misunderstanding and show that they deal with platform namespace.
enter login page here for showing login page if user tries to restrict area
The body that is executed for each loop iteration is argument 1.

So the code now processes the arguments in 3 ranges 1..3, 4, 5.. (upper ranges in the code are exclusive).

The body argument is passed false for the flag that indicates it is within a child query - and therefore executed a single node.
Moved #include of "build-config.h" from WsDeploySrevice.cpp to WsDeployService.hpp to allow access of CONFIG_SOURCE_DIR definition from the header file.
typo - "...this one *and* gets..."
CATCH on a transform could be a good idea, but isn't implemented at the moment.
Not sure what it should look like...
yes the compare operator wasn't indicating that it required any fields from its operands - which mean if they were rows all the fields were removed.
The preventOptimization prevents this.
refactored very well imho. maybe it was not necessary to transform it into a jquery function but if someone sees this implementation could be inspired!  and the while code of SymfonyBricks in my view should itself be a showcase of best practices ;)
Is there a way around this now, or just a FYI for future? I also am not a fan of hard coding things like this.
this is not consistent with the only docs that I found. If you have things configured like this, can you run: nova get-vnc-console [server_id] novnc and get the proxy base url back?

Do these setting work for you on multi-node?

I was actually thinking that this flag may need to be set on the api node instead of the proxy host, but the documentation was definitely not clear.
pourquoi mettre un espace ici ?
what is the difference between scalarNode and variableNode?
this can just be `if (null != alpha && '%' == alpha.type) stuff here`
nit: is this relevant?
this 4 lines, fixed invalid packages on filesystem (package is installed but not found in the filesystem) is it still implemented?
Would an explicit set of steps to build and run this here be overkill?
Even if body becomes nil, this code will work perfectly fine.
This is already registered by ```Exploit::Remote::DCERPC```.  Even if it were not, it should still be an OptPort, not OptInt.
I took the information from DEPENDS.txt and it says 0.15.
`pyramid_reduce` casts the image to float, so if the input image is not float, then the first image of the pyramid is a different type than the others. I think you should cast the input image before adding to the pyramid list.
This isn't right. should be `self.instance_name in bundle.data`
lisp standard:  double semicolon comments are at the beginning of the line (already indented)
single semicolon is for end-of-line comment.
Cool; nice hooking into global drag and drop stuff
This comment doesn't really apply any more.  It's true for the billboard version, but not for the label version.
I personally would rather see it removed, but I'd rather a third-party make the call.  @shunter ?
I brought this up during the initial material pull and I think Ian said something about the GUID being too long for a uniform name or something, I don't remember the details.  If that really is the case though, can't be just change this to a runtime counter?
I'm not sure what you meant, false isn't the value of the interval.  The value of the empty interval is a frozen instance of an empty interval.
Same `clone` comment as above.
v is getOuterView(), which could be a border if there is one.
It doesn't seem necessary to call invalidate() for the border.
I don't understand. Are you failing silently? Why might canvas be null? It might make sense to add a comment.
What type is color?
Consider replacing MIN_MERGE here with a value between 128 to 256. My own benchmarks found insertion sort to be faster up to (about) 256 elements. There's a lot of overhead to initializing Timsort, so simple insertion sort is faster on arrays up to a certain length.
I think we should look at the generated code and see whether this is slower than the previous code (I don't think so; at best it would be as fast). So we need to keep the old code, but with `immutable` for `T`.
I would add that »dealing with tricky xyz« doesn't strike me as particularly descriptive (if you are just reading the docs, what tricky aspects of .init does it avoid)
Yeah, I was just thinking that. This can probably be done with sizeof's, I'll change it...
Arguably more efficient in all cases.
Ignores individual tests, but not the suite, so its faster.
can we put these 2 files (browser_tests.*) under 'tests/' ?
Seems like it's time to apply that patch I had created two years ago :)
define the custom sanity check in the easyblock
This is not the way the calculation is done on the server. It is `(size_x * size_y) > (max_plane_width * max_plane_height)`.
Worth dropping this comment block? Otherwise, happy to merge.
the previous formatting was more readable IMO
This should have an "is request from admin" guard
Now that you mentioned it really looks strange. 

But I have checked and it is valid. Checkout http://www.regular-expressions.info/. First regex it shows is:

`\b[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}\b `

Maybe I could rewrite it to:

`/^[\d+-]$/`

So it looks less strange?
why don't you use the smoothed velocity any more?