labofsahil/github-event-dataset-2012 huggingface dataset:
Is there a good reason that this method is defined in a conditional, and not just `next = __next__`?  I know the method is not *required* in py3, but it seems cleaner to just define it.
nope, that's why I've changed it :)
Why not keeping on using CMake to build? 

Something like this should work

```
SET BUILD=RelWithDebInfo
"Path\to\cmake.exe" --build . --config "%BUILD%"
```
some explanation of what "back-end" and "meta-data" expectations are here would be great.  it will just help future developers to more easily use this class.
trailing whitespace at the end of this line (I have my editor automatically strip trailing whitespace)
Don't you need to return "result" here for inject to work properly?
No, the jvm parameters for windows is passed by the service wrapper, rather
than directly as it is for Unix.

Sent from my phone, please excuse typos and brievety.
On Apr 16, 2012 10:27 AM, "Peter Neubauer" <
reply@reply.github.com>
wrote:

> > @@ -204,6 +204,7 @@ startit() {
> >
>  -Dorg.neo4j.server.properties=\"${NEO4J_INSTANCE}/conf/neo4j-server.properties\"
> \
> >
>  -Djava.util.logging.config.file=\"${NEO4J_INSTANCE}/conf/logging.properties\"
> \
> >          -Dneo4j.home=\"${NEO4J_HOME}\"
> -Dneo4j.instance=\"${NEO4J_INSTANCE}\" \
> > +        -Dfile.encoding=UTF-8 \
>
> Is there anything that needs to go into base.bat too?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/neo4j/packaging/pull/14/files#r687121
>
got to remember to remove the import of `psycopg2.extensions`, too.
Just merged. I'll be honest recently I've not known which to go with. Is it valid HTML to have these characters as non-entities?
ymfony -> Symfony
You should drop db also on `tearDown()`
This does not set a key
according to PSR-2, it should be ``final private``
This is a good time for 'getOrElse { ... <None-block> }'
Just because I was curious about this myself: http://www.ruby-doc.org/core-1.9.3/Binding.html
Add method to specify RetryPolicy
Indentation

```python
from kitchen.backends.lchef import (get_nodes, get_nodes_extended, get_roles,
                                    get_role_groups, get_environments,
                                    filter_nodes, group_nodes_by_host,
                                    RepoError)
```
Here we should first explain that we are taking the list of tags defined in Chef's top-level `tags` attribute (with a link to the docs)

Then would come the "It is possible to link a button style " part, and last the list of available buttons.
Can you add a link to the Symfony2 documentation? (http://symfony.com/doc/master/book/installation.html)

You can write multine comments with:

    desc <<-DESC
      blabla
    DESC
A resizable panel would definitely be useful, it doesn't even need to be in its own sublcass. Can you move this into Flame.Panel?
If you do that, could you also make sure to use 4 spaces for indentation?
KEYSTORE_PROVIDER specifies the FQN of the provider implementation. I don't think this should allow expressions.
Thanks again for the help, your feedback certainly simplified things a lot.

I'll merge the adding of the DUPs in the next AS7-5083 subtask.
Why sleep?
The same "whitespace" styling issue.
+1 to @etiennesegonzac comment, that should be a l10n string with parameters.

Also, just out of curiosity, why 12 characters?  -> contact.name[0].substring(0, 12)
@gtorodelvalle just commented on the bug.
The reason we are not doing `removeFrame()` here directly is because we would like to have `setDisplayedApp()` to run the closing transition of the crashed frame first. Are you intentionally want to remove that animation?
Is this change intentional/
By using `addEntryFile(path, compression, file, *true*);`, you should be able to listen to file write end and remove those files.
https://developer.mozilla.org/it/docs/XPCOM_Interface_Reference/nsIZipWriter#addEntryFile%28%29
```
zip.processQueue({
  onStopRequest:function () { 
    // delete files
  }
});
```
please avoid unnecessary vars

if (document.getElementById('fbTotalsResult'))
I'm not sure to understand why you delete the FontLight rule and then use it as a font-family.

Also you need a space before the { of .font-light
Sure i understand that but if we remove this css we will need to add `/shared/style/confirm.css` and set in html the messages right?
We can talk tomorrow in SP about this :D
If the user cancels the activity (by press `home`), window manager will destroy the app frame instead of hiding it.

For completeness, I will do `<button onclick="cancel()">Cancel</button>` and have the `cancel()` function do what you'd said.
I'm bit concerned with this practice, it would be worth having the feedback from somone used to think about DOM security issues.
If we create a `<script type="application/l10n">` node before this existing one, we can load abitrary l10n internal dictionary.

Having said that, we have the same pattern with `<link>` nodes, so I'm not sure it really is an issue.
`developer` is already defined. Please reuse it, or use a different entity.
Note: changing these two lines should not be necessary — these are just fallback values in case the localized date/time format can’t be fetched.
Yes please. I like explicit naming: pTotal -> previousTotal sounds good too
This should really be `entry.LocalIP == "" && entry.GlobalIP == ""`, because it's conceivable that a Node could have a local IP but not a global IP, or vice versa.
i'm not sure offhand how expensive it is, but i'm wondering if it's really reliable/necessary? if the process is dead, won't the pipe be broken - and that gets caught below anyway?
Maybe add: otherwise the flow planner will throw an exception.
This should be "sealed" so the compiler can see that any matches are guaranteed to be exhaustive (not a huge deal since field conversion happens before job submission, but still...)
Don't use asserts here - use TypeErrors.
wrong semantic
This line shouldn't be necessary anymore - it just keeps pyflakes happy when using the try .. except ImportError form.
this should use ``elseif`` instead of nesting the ``if`` in the ``else``
this should be removed as the use statement is not used (otherwise the code would have been broken before as there is no new code)
Spec says they're numbers (https://code.google.com/p/selenium/wiki/JsonWireProtocol#/session/:sessionId/window/:windowHandle/size), and RemoteDriver passes them as numbers.  Is it really useful to convert mispassed strings instead of giving an error?
IRCの会話の流れが
remove "QObject::" namespace
What actual output are you ending up with here?
It was already in the bundle. See line 39.
I've just group ui gems and specify exact version for them.
I hate Xcode for this very reason, I'll el remove this
are unix sockets supported at all on win32/cygwin? If not, we can ifndef the whole thing against win32/cygwin... or I think you're not setting BUILD_UNIX_SOCKET for win, so maybe that would be enough
Yes, that's the platform code style.
Please do not alias `set / get / keys, ...` on `Branch` constructor.
Somebody should have yelled at me for the `if(` instead of `if (` previously
you know, we could do this one on the backend too.
I chose to make a minimal change to start, but this is ugly.  changing it I believe will require working with @mathjazz
If there is no legal way to modify active group and / or active group's title then caching the result of `getActiveGroupName()` would be nice.
In other words: if  the `(TabView._window == null)` implies that group items are immutable then caching the result of `getActiveGroupName()` is allowed.
Bootstrap then? I'm just unconfortable with doing (custom) routing in a file that's not in the application directory by default. It could make the impression as you're not supposed to create custom routes.
Same here, Z_ARRVAL_PP should be available as well.
On Sun, Dec 02, 2012 at 10:40:59AM -0800, Jeff McCune wrote:
> 
> Thinking about this a bit more, I wonder if putting `setcode` inside of
> the conditional means that `zfs` will only be checked once, when the fact
> is added.  Do we want the behavior to have Facter check if zfs is
> available every time the fact is resolved so that a persistent process
> could configure ZFS and have the fact automatically reflect this change
> in configuration?
> 
> I've already merged this change set in, so we might just want to change
> the decision tree in each of the affected facts.

Indeed, actually the setcode can only happen if zfs/zpool commands are
available in the PATH.

As I'm used to trigger puppet runs with cron or mcollective, I'm a bit
confused by your message now. Must I understand that when running `puppet
agent --daemonize`, the facts are resolved only when the daemon is started ?
This doesn't seem right, considering the pluginsync feature...

My understanding is that facter is run first, then based on the
informations gathered, the catalog is built, and that the catalog is "just"
a static list of resources.

If puppet installs the zfs CLI tools, or upgrades zfs, and some resources
are inside an `if versioncmp($::zfs_version)` block, you *will* need 2
puppet runs, you can't even use stages. Right ?

So based on my assumptions/understanding, I'd say that switching the
`setcode` and `which('zfs')` statements won't change anything AFAIC. But
you seem to see a use-case I don't. So I definitely won't object changing
this :)
@hkenney empty line here.. :)
This puts a hard dependency on rubygems to use or view the rake tasks. Something that allows the rake tasks to be used if rubygems isn't available would allow both use cases. Something like...

```
begin
  require 'rubygems'
  require 'rubygems/package_task'
rescure LoadError
  require 'rake/packagetask'
  require 'rake/gempackagetask'
end
```
Should we deprecate this?
The FFI gems stores this in a FlagsMap constant, maybe easiest to do that too? See: https://github.com/ffi/ffi/blob/master/lib/ffi/library.rb#L158
this is a binary, not text right?
This was taken from the original file. I'm positive about changing to DateTime or Zend\Date (less problems with timezone), but then I would change also the setter
s/in/to a/
previous exception
InvalidArgumentException on error opening a file ?
UiHandlerSTrategy and the call to super won't be needed anymore
Try to use schema.rb and seeds.rb
Is it test for "empty result"?
If it's null instead, your code will break, so I'm not sure if that's preferred behavior.
@sindresorhus can you apply the patch with your suggestions (also cleanup the `event` argument I carried over copycats)?
Regarding the question about being able to send params to `bindAttr`, afaik it's not possible :|

Thanks.
Trying again. @addyosmani @siuying Does any of you have access to that one?
Same for this: `{asList: true}`
This section does not need to be here. It should be only in the README.md file, to make it simple to keep updated.
This error messaging markup is used in multiple places -- you should move this into a helper.
Ah I see, you're implementing AbstractSubsystemBaseTest. I think it would be good to keep the original test but perhaps mention AbstractSubsystemBaseTest in the docs saying what it tests for you
Why the side channel? Why not e.g. a separate s_req_host_at state?
The convention I've seen is to put the model registration for admin sites on the line directly below the class, and I think we should follow that here as well.
Is there no need to catch the possible `DoesNotExist` or `MultipleObjectsReturned` exceptions?
I would ditch these and just use `class="location"` on the `<dt>` and `<dd>`.
Here is a perf: http://jsperf.com/var-inside-loop-vs-var-outside-loop/4
Nice. No need for additional state :)

Can you throw in a comment saying something along the lines of "After we have hit the ready state then send the event to any children that are added" or some along those lines.
Perhaps I'm missing something here. So the bindToRoute behavior has been moved to load and the actual bindToRoute implementation doesn't notify until later?
Can you please make sure to remove content of former replies and only refer to what you are answering to? That would be nice to make comments smaller. It's the same as on Bugzilla.

To answer your question regarding the URL you gave the correct answer. So yes, that's what you should point users to.
Okay, filed #16.
I've done something like this myself a few times, but I wonder if it's a better idea to just have the core objects(node, action, etc) define a __json__ and have this just look for that method and call it.  That would allow us to move this to util/json.
horray comma first
I would like to have a default value for level that is SERVICE
Good point, I didn't realize M was available here.  Done.  (I'm not familiar with pull requests, so I don't know if I can update/replace this one, or have to start a new one.  I guess I'll find out...)
Same as above for the formatting.
Not that it matters, but I like Hash.new as well. This is only works well when the Hash is empty though obviously.
hasattr(request, 'user') might be safer... but then again it'll work in 99% of the cases :-)
In fact, the return is still there (see row +415), this is only an effect of the not too smart diff here :-)
My proposed change is simpler than it seems from the visual diff.
I believe it is different - the braces go on the same line, not a new line.
explicit не научился ставить
Ahhhh. Makes sense when I actually preview the file. You can just get rid of the unused code. No need to keep it around in block comments.
Yeah, I think it makes more sense.
Do we actually need this, or is residual code from when we were trying to get the scope right?
Is this name essential to the uniqueness?  If so, you need to be more careful because this can cause conflicts.

Also, you probably don't need to `s/_/_/` here, which is surprisingly costly.
It's just inconsistent with all other _rowToX functions. As far as I can tell, we only need to do the for loop once, others usages of this function handle a single comment.
Order and indentation are wrong here.
The id of that User object that's in there will already be the u:cam:userid string
Using `!tenant.active` seems like a loose toggle. If the method is explicitly `_startTenant`, should we use explicit booleans instead?
Fixed. Also fixed a bug with the last line.
Please also update the places where config values are accessed currently.
Those 5 imports are not necessary.
I'd go for any alternative that avoids duplicating mutable state that has to stay in sync in a concurrent system. So I'd vote for making the actor get the threads from `ScalaDebugActor`.
@lsmith the RoutingExtraBundle can inject the RedirectableUrlMatcher here. but then we still do not implement RedirectableMatcherInterface - but as the matcher is never exposed by the core router anyways i don't know if there is a problem or if that is enough.
Why do you reset the ver/mask? The purpose of tracking this is to we don't bother to reprocess sub-tables that we've previously seen and therefore significantly reduced the processing overhead. This is particularly important for things like freesat/view where huffman compression is used on the strings and we want to avoid re-decompressing stuff we've already done.
Yes i know, but i think it would be a good improvement, because in development the most people read the codebase not to the documentation.
The [assetic library](https://github.com/kriswallsmith/assetic) does the same in the Filters. 

I would start a PR that add a link to the provider site to each provider, if you wish?
import Data.Text (Text)
import qualified Data.Text as T
block -> callback
Thanks! See 69420f1.
ok. буду доделывать.
Просто любопытно: для чего нужна эта строка?
Понял, спасибо.
в задании не оговаривалось - возможно и ошибку более правильно
ну и раз уж вы решили продемонстрировать свое умение программировать на С, надо было вместо этого цикла выполнить более быстрый
bool *p = mutated;
for (int i=0; i < c; i++) *p++=false;
What does it mean to compare against a constant?  Like } else if ((flag = search_block(godroom, roomflag_names, false)) = 0) {

Would that search roomflag_names for godroom or abbreviated of godroom (false means "not exact" e.g. abbreviated) and if found (= 0) then  proceed?
For the moment we were keeping the code as consistent with the framework as possible, but I agree forEach is a better way to go.
This seems potentially dangerous. Why do we want to expose this functionality?
What if the activity is destroyed and re-created, for example, the user rotate the screen...?
...if we decide to keep this code. See comment below.
looking idx_seed_con it looked like it was similar to `np.ix_` but now I think the connectivity
should take as parameter `seeds` and `targets` where None means all.
we would get rid of idx_seed_con.
you can use : assert_raises

```
from nose.tools import assert_raises
```
the the :)
old man wisdom :)

I agree -1 on copy_raw. It's not necessary as not exposed anyway.
ok I can live with that :)
Because `Promise.timeout` is defined in the play artifact. The iteratees artifact does not depend on the play artifact.
indentation doesn't match opening `<div`
Kind of deceiving if it's doing both removing or adding in the same method. Also, can use mongo set methods to atomically add to the set or remove from.
Also just ran the link checker and fixes some more - so the whole docs should be in good shape now.
本指令可和与<link id="proxy_no_cache"/>共同使用。
    ->
    本指令可与<link id="proxy_no_cache"/>一起使用。
I can't believe I'm having to justify the existence of a feature that I'm trying to patch and improve...

Interpolations are needed because many users of paperclip have upgraded from attachment-fu or similar libraries.   Given there are GB (or even TB) of data stored from older systems, we need to make paperclip adaptable to existing path structures.  Mine is currently different from the default, and I have no desire to migrate all my files from one path structure to another.

I find this entire conversation somewhat inexplicable and wonder if I'm just wasting my time with my PRs.  Maintaining my own patched copy isn't that difficult.
Needs explicit return type, and you can use simply "this" instead of "Settings"
Я в виде непрочитанных храню, как ни странно, непрочитанные статьи на
слешдоте/хабре и паре других блогов. Не хотелось бы потом их руками
снова находить.
Повесь, я настаиваю. Юзеры тебя убьют, если не повесишь.
Acá se está saltando un nivel en los títulos, podría traer problemas. @AlphaGit por que decías que no vamos a poder evitar estas cosas?
I feel like I've seen this if statement before.  Can this logic be consolidated/encapsulated into an appropriate class?
why this.plugin?
See https://github.com/weissjeffm/test.tree/blob/master/src/test/tree.clj#L115  = test.tree/state - you can get this info without keeping track yourself
Get **s**
There's now only one log file.

The log relies on a dependency object (logger).
I did this already, or did I miss one?

On 9 apr. 2012, at 13:25, aparajita wrote:

>> @@ -143,6 +143,26 @@ - (void)initializeEventStreamWithPath:(NSString*)aPath
>>         flags = kFSEventStreamCreateFlagUseCFTypes;
>>     }
>> 
>> +    // add symlinked directories
>> +    NSFileManager *fileManager = [NSFileManager defaultManager];
>> +    NSError *error;
> 
> Yes, pass nil in all the places you are passing &error. Then we can merge this.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/cappuccino/cappuccino/pull/1468/files#r659699
The implementation of the viewholder pattern is wrong. The trick is that you only invoke the findViewById() calls once and have them cached for eternity. See http://stackoverflow.com/a/3832467/44089 for an example.
good catch @hub-cap
the correctness flag can be set by any of the running threads. Using the fure II'd have to collect and then iterate over a list of futures which is a bit more code to write.
Dan I tried! Look how much I had to adapt DistributedExecutorTest to accomodate this change. The problem is that on calling thread I can not setNetworkFuture until I get that future from localExecutorService.submit(call) and the only way I see how to do this is to use non calling thread executor. If you see any other decent way I am all ears :-)
Yes, absolutely. But this would probably require a redesign of the whole component instantiation & injection process, which is probably way more than I can handle, since I'm not familiar at all with this area of infinispan.
I can totally se where your template pain came from :-)
This should probably be:

```python
raise AssertionError("bad proxy scheme: %r" %(self._proxy_scheme, ))
```
I don't think we are using `CHANGELOG` any more. I think this should go in `whats_new.rst` instead.
again, this will just error out, use `warnings.warn("message", mDeprecation)` or `warnings.warn(mDeprecation('message')`
Thanks for the comment.

I could fix the pydoc so that it says it only accept int and [int, int].  And leave the implementation to the people who actually need uneven bin. Uneven bin with no proper normalization is tricky to interpret.

I believe most of use case are the even grid. People usually use 2D hist to see correlation between variables see hexbin example for example. Interpolation is very useful to make make the correlation clearer(assuming user called it with enough bins).

If uneven bin is given, some normalization by bin width is needed otherwise it will give out some picture but it would be quite hard to interpret. hexbin doesn't support unevenbin aswell probably due to the same reason.
Oh sorry.. I didn't read properly the comment. It is fixed (properly this time).
If you actually look at the code there's not much to check. Just the elapsed time and the time slider.

For a smoke test I'm happy with what's being checked now.
mozqa.com exists for this very purpose
Yes I did checking visibility of the buttons but when I checked the test for false negatives I discovered this bug: 
https://bugzilla.mozilla.org/show_bug.cgi?id=813561

This is the best I can do in the meantime.

I'll add a TODO in here.
Typo: Forgeting/Forgetting
I have tested the google-v3 example in several browsers (IE 7 and 8, and FF 3 included) and I haven't noticed any issue.
What does not fit? What is the error?
Include GPL header and your name here
That's a lot of styles to put under a dual root element - didn't we just have problems with IE choking on too many selectors? ;) Can you find a common root name?
Not introduced in this commit, but why is the alert class specified twice?
Just `forms`?
этот файл уже вообще можно удалить
Are you sure you want to override the parameter ``$plural``? This often makes code less readable.
Are you sure the method ``getItem`` can be called on ``$cache`` (of type ``null|object<Zend\Cache\Storage\StorageInterface>``)?
``$OK_RETURN_CODE`` does not seem to conform to the naming convention (``^[a-z][a-zA-Z0-9]*$``). Please also check line 27.
Are you sure ``$element`` has the correct type as "integer|double|string|Boolean" is not a sub-type of "string"?
Which part? I've removed ```OrderedFixtureInterface```.
Since the helper file can be removed, so can the corresponding spec.
Why? Why all these periods?
Spell out the word boolean.
These should be broken up into different test methods so you can get a granular picture of what is breaking.
Oh excellent, will check that out.
Yes, and I think the endpoint doesn't work if you try to send api_sig
Categoria isn't capitalized in the other entries.
The name `create` would be more consistent with other Java libraries.
why all the returns here? we can remove all of those
And also add units in a comment at the end, like:

```
CHECK_ONLINE_USERS_INTERVAL = 10 # seconds
```
Because MongoEngine checks what you are storing in the database. And on reading from database makes sure it is in this format. So it is a ages old question of typed or non-typed data. And I like typed (and semantically full) data more. You can define some other model, but please do not use free-form dicts.
I would simply add another flag to the database. (Twitter authorized.) And first time we do authorization. Then we do authentication. When we will be using API needing authorization and we will be denied, we will simply switch that flag again off. And do some warning, reauthorization or something.
Use [inArray](http://api.jquery.com/jQuery.inArray/). We are using jQuery to make code more portable, so use it.
Yes, Facebook's scope is also wrong. ;-)

Yes, put it as a constant. Maybe really as a module top-level constant.
You can also configure, that `station_id` and `timestamp` should be unique together.
Thanks for your request :-)

escodegen ensures `esprima.parse(escodegen.generate(esprima.parse(code))) === esprima.parse(code)` structurally.

So, I think this change breaks this guarantee, is it correct?
```javascript
code = 'a, b, (c, d)';
esprima.parse(escodegen.generate(esprima.parse(code))) !== esprima.parse(code); 
// of cource this === isn't JS ===. structural equality.
```
$message === ''
Kann man diesen Block löschen?
alsdann... werd ich mir ansehen...
Can we create a URI from org_study_id to make them resolvable? Are these identifiers stable across time?
Should be left as is because it is not referencing the pages model but the method pages.
for example: my.jpeg
For OGM we developed a brand new JP-QL parser which is already able to convert simple queries using Hibernate Search under the precondition that:
the _isCacheProvided_ is needed to workaround a bug in Hibernate ORM: start() might be invoked twice.
Please use this boolean to avoid creating ``new Mongo`` twice, without closing one..  ugly, but currently needed :-|
This should be

```
jobConf.setOutputFormat(anOutputFormat.class)
```

right?
stub or mock?
But you should set this in `settings.py` file. Not here. And set it to tuple, not string. In Python, `('127.0.0.1') == '127.0.0.1'`, you are missing something. Read how you define one-element tuple in Python.
Noo. Use setting from Django settings. Please read a bit about how Django is configured and how to work with Django settings.
Rant: The name of this API is terrible - it should be "suspendDeployment" or "changeState" or something. However, the operation is what it is, and we should match up with the core documentation. I'm not asking you to do anything here, just venting about the API name.
Unfortunately this is a callback from nginx, so it needs to have this signature.
Needs "is": The fetch is started by the main thread.
Correct, see under the Overview header of https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt.
Hmm, I thought I had changed this. And it looked that way in the diff. Just curious if it was broken. No big deal.
Presuming this is to remove the dependency on immutable map?
I remember you said that it just feels right, but non-successive version are a bit strange, don't you think? In fact, the best approach for versioning that I know is [semantic versioning](http://semver.org/), but I'm not sure that we're following it with TwitterCLDR.
Hashtags entities are on a .text property?
设置是否编译进<c-func>select</c-func>方法的模块。如果系统平台不支持kqueue，epoll，rtsig或/dev/poll等更合适的方法，该模块会被自动编译。

感觉也不是很好，有点意译了。
There's another similarly named method, I didn't want to confuse the two and since this is local to the original event I decided to keep it that way
Those last brackets split into separate lines:

```
        'gs_upl': urlParams['gs_upl'],
    });
```
class was removed because is no longer needed
yeah, i save an array of the names of all checkboxes that were selected
Perhaps we can see if @reedstrm has opinions yet on pyramid ? I would rather work from foundations everyone is in tune with than have a black hole that is always at the back of peoples minds.  That generally acts as a drag on new work I find.  Eventually the bozo bit gets flipped on huge swathes of code and its a morale killer
Yes.  That's a typo.  I just fixed.  (The above two lines are also wrong, referring to sets of files.)
Well it's not a bad idea at all but I agree, I'd be surprised it it ever dominated runtime. My question wasn't very clear: I'd not been able to find an end-point `_GenerateFromX` function which actually returned `safe = True`, and was a little perplexed.  Anyway I think it's totally up to you if you want to retain it!  It was more a sideshow distraction to my understanding than something which actively confused other parts of code comprehension... :)
This is already noted as issue #76; I expect it to be addressed by refactoring to use shared_ptr (#107).
There is a distinction here, which is that if you do
```
print galsim.Sersic.__doc__
```
then you get the Sersic dox and the statement that it inherits the GSObject methods, whereas if you do
```
help(galsim.Sersic)
```
then you get the Sersic dox and the dox for all the inherited methods directly.  I didn't realize that.  Now I'm not sure which is better to recommend but I am also inclined towards the latter.
I just tried this (change is in my master branch). This is the output from 'opam install kaputt':

```
The archive for kaputt.1.2 is in the local cache.
Extracting /home/mike/.opam/archives/kaputt.1.2+opam.tar.gz ...
Build commands:
  %{sh} configure -ocaml-prefix /home/mike/.opam/4.00.0 -ocamlfind /home/mike/.opam/4.00.0/bin/ocamlfind
  make all
  make install
```
So it looks like opam doesn't recognize the %{sh} macro. Is this meant to work? Some other config I'm missing?
I'd just rely on tearDown for app.stop. Presumably our other tests cover that service_up is set properly. Or is the intent to explicitly test that nginx stops properly when it was heavily loaded?
Hm, it will log at warn (because `close(closeable, true)` will never throw the IOException).

But that's annoying and not the behaviour I thought it was doing for "quietly".
At some point I'll perhaps add a closeVeryQuietly utility method (or some such).
> Using both `true`/`false` and `yes`/`no` in the same code feels pretty wrong

There is merit to that.

> it seems like there should be a single word for each truth and falsity

You're not alone in that belief. We've had issues to pick one and get rid of the others. We could probably do without `on` and `off`, but CoffeeScript has gone the route of multiple boolean aliases, for better or worse. So really it's just up to you whether you want to make use of them or not.
bah donner le + de thunes aux plus downloaded et en meme temps recompenser les inconnus. comment tu concilies ca?
Este comentário não está descrevendo a funcionalidade
No results: No, it returns a 200 with no body. There's already a test case in the SDK for this.

For real time searches: it returns 204. It won't loop at all.
this is a further issue that needs to be fixed, but now that we have a lot of filters, what if we would like to share filters over with another stereotype? we shouldn't copy/paste the same function, that wouldn't be nice.

Maybe we can take these functions out of this object literal, and just give pointers to them; like

    class: {
        filters: {
            iterator: iteratorFilter,
            paramIterator: paramIteratorFilter
        }
    },
    entity: {
        filters: {
            iterator: iteratorFilter,
            paramIterator: paramIteratorFilter
        }
    }
    
    ...
    
    function iteratorFilter = function() {
    ...
hmmm.. is the slot arg useful at all?
Oh, I see what you are saying. I thought it would be lame to keep adding more settings for each new API method we call instead of having that knowledge built in to the code. At this point, we only support two calls so it doesn't really matter either way though.
Yes, but ``veritcal``? :D
you come out of nowhere and appoint yourself an admin on our server, then think it's rude when i tell you to revert it? also, 'thankful'? some communication difficulties here i think.
I would keep them ordered :)
You're the most familiar with this code - is this code still around for a reason?  I'd be inclined to delete it.
Might as well change it to the real VID name which is from http://www.obdev.at/products/vusb/index.html "USB_VENDOR_ID_VUSB"
but maybe work that up as a separate patch to not feature creep his pull request?
Typically (even though we know what type of object will be returned), it's Objective-C convention to make the return type `id` for initializers.
Ah yeah, that makes total sense and shouldn't do things like see if the user can authenticate afterwards.

It still seems wrong to me that the test passes if I comment out the code being tested.  There has to be some kind of expectation we can setup to make sure the code is doing something.
Сначала фильтрация, потом сортировка
Total (nanos):    6404942256
Total (msecs):    6404942
Total (millis):   6404
Total (seconds):  6
Single (nanos):    64049422
Single (msecs):    64049
Single (millis):   64
Single (seconds):  0
Сначала сортировка, потом филтрация
Total (nanos):    7508879738
Total (msecs):    7508879
Total (millis):   7508
Total (seconds):  7
Single (nanos):    75088797
Single (msecs):    75088
Single (millis):   75
Single (seconds):  0
Зачем это нужно? Calendar.getInstance() уже возвращает текущее время.
Oops--you're totally right. I'll fix it right now!
yay better implementation!
yes but this will have to do for now.

and we need the atomic add here too.
These demos can go into the docs directory, either in the form of loose-standing demos or tutorials.  See also:

http://scikit-image.org/docs/dev/auto_examples/

Just give me a shout if you need a hand; I don't expect you to do all the heavy lifting yourself!
But `_RidgeGCV` is a private class so users aren't supposed to use it...
'setup' is a valid single word when talking about programs, but if you want to use both words, 'set_up' would be better. If this breaks the API, perhaps add the setUp function with a deprecated warning.
Would be good to `%timeit` the two alternatives on a small to medium dataset just to have an idea.
hmmmm, yeah, maybe ;)
Why not just return the number of pixels fixed?
Fixed [=
Is it possible to put a comma at the end of the last element? Is yes, please do so.
Use `getOneOrNullResult` instead of catching the exception
would be clearer to say at query load time.
This code should really be commoned up with the code above.
What  do you recommend that the syntax be changed to?

The translate (line #551) in question is this:

<xsl:with-param name="path" select="translate($outputFilePath, '/$', '\:')"/>
I've always been in two minds about this, it's not start() that they are typically initialized in, but init(), which is very early call, and called once (not repeated per child query). 'grouped' is case in point, where it is initialized in init() , so constructor version is strictly needed.
Anyway, it is cheap enough for the constructor to initialize all, even if some repeated in init() ( or start() )
So I have been tending to do that in new activity constructors, or refactored ones, and for new vars, but not going over existing vars.
I think this is about to be removed anyway.. only called from testDFS()
It's called ecl.ini (lower case)
I think it might be better to make `enabled_apis` a parameter with these as defaults. This way the user can override if needed.
This is wrong, you need to switch to the apache license for this file.
Can you add (feel free to pick one / many / all):
 * a link to the related code in Namespace module
 * a reference to bug 720666
 * a comment about this workaround in this bug, or the platform bug that will prevent using weakmap workaround in namespace (bug 673468)
 * a note about what is done in events/core so that the weakmap is inited
Removed this line, and changed code to make sure we don't execute code that's unneeded.
In order to determine cross-domain-ness we need to compare again against the window's port.
Wouldn't this do the same every time? Unless you seed the rng with the clock, or something?
OK - I will merge your PR and then uncomment the call to tempname. Seems
good to me.

We still have to deal with cleanup, but I will ping the ipython-dev list
for some advice on that. Maybe have the call to Matlab.stop come together
with a cleanup operation of these temp files.


On Fri, Dec 21, 2012 at 2:15 PM, Steven Pav <notifications@github.com>wrote:

> In pymatbridge/matlab/private/multipart2struct.m:
>
> > @@ -35,8 +35,10 @@
> >      otherwise
> >          subdata.ContentData=subrequestdata(w(i)+1:end);
> >          [pathstr,name,ext] = fileparts(subdata.Filename);
> > -        filename=['/' char(round(rand(1,32)*9)+48)];
> > -        fullfilename=[config.temp_folder filename ext];
> > +
> > +				% [NULL,filename] = fileparts(tempname());  % more conventional;
> > +        filename = char(round(rand(1,32)*9)+48);
>
> As I noted in the comments, probably
>
> [NULL,filename] = fileparts(tempname());
> clear NULL;
>
> would be better. I suspect that the tempname utility in Matlab is
> wrapping Java UUID code, because it produces strings like
> tp3269c81a_fa39_420b_9822_7fb678dfecc3.
>
> One can call the UUID code directly however, as follows:
>
> import java.util.UUID;
> jUUID       = java.util.UUID.randomUUID;
> UUID        = char(jUUID.toString);
>
> (This runs in Matlab R2007b)
>
> I agree that the possibility of rng collisions should not be discounted. I
> suggest uncommenting the fileparts(tempname()) part as an easy fix.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/arokem/python-matlab-bridge/pull/3/files#r2493334>.
>
>
Unable to use `openstack::nova::controller` due to double declaration of class `nova`. So instead, I added the contents of `controller` to `all` and made  changes to make it all work together. I will admit it is a bit sloppy.
Checking if the list is empty would do the same `(not (empty? (living-authors (:authors book))))`.
Better default would be NULL
Lets imagine I don't know what ilog is. How do I let the user know the connection was established?

I'm going to grep the framework for use of ilog and see if I can figure it out, but it would be easier if you could let me know. Thanks.
Chris nailed it, the reason is to work around Meterpreter pivot issues with unbound sockets. We can either fix the Meterpreter side or modify udp_scanner.rb to support both automatically (break into two sets; one for pivoted IPs, one for not). The goal was to have the module handle a "run_host" scenario, but use "run_batch" on the backend with a shared socket to make things much faster by default.
I would expect RGB-images to be a more common application than multispectral images. Would it make sense to replace `spectrum` by `channel` or `layer` ?
Nit: but we usually don't need empty newlines after docstrings.
The convention is that `context` is almost always the first argument.  Did I write this?  Sloppy.
We should also be able to remove it from `renderForPick` since the pick pipeline calls both `update` and `updateForPick`.
Same comment for other places.
We're actually Fort Wayne, IN. Thanks for including us!
Fill behavior is already documented in the moduleAPI description for autoFillHeight. I made it more clear that we are describing the fill behavior.
These are bools. Actually we would need `any` here, but is not available yet in Python 2.4:
Summing bools is the next best thing (although ugly, I admit).

```python
>>> sum([True, False])
1
>>> any([True, False])
Traceback (most recent call last):
  File "<stdin>", line 1, in ?
NameError: name 'any' is not defined
```
Actually, @jzaefferer do you know if there was any reason why this dialog was down there?
You probably meant to remove just lines 29 (old) or 31 (new).

I don't think that's the case here. For the name based virtual hosts you want a default Apache NameVirtualHost/VirtualHost as '*' and for Puppet resource name you want something that uniquely identifies it's resource -- which is not the the VirtualHostName (vhost_name), but can be a concatenation of ServerName ($servername) and Port ($port).

It looks good to me in a way it's right now, it just adds parameters of apache::vhost into apache apache::vhost::(proxy|redirect), doesn't invents anything new. Moreover it makes these two definitions usable. I think they were meant just as an example, but not for serious use.
Probably better to open an issue, asking why this is the case.

I believe it is so that a non-SSL version can redirect properly.
Trying out the comment on individual lines!
This shouldn't be part of the problem.  You are displaying "Indefinite Integrals of Binomial polynomials" at the top of each question.
s/statment/statement
No, we haven't been consistent about a color for the final answer. I find myself choosing colors based on different things for different exercises. For example, sometimes it makes sense to use cool colors versus warm colors to add another layer of subtle cues about what things are related. So the final answer doesn't always end up the same color. I kinda like the flexibility to present each problem the best way possible rather than decreeing that the final answer must be a particular color, then having to work backwards.
I suggest we make these undocumented aliases. That way existing code will continue to work and new code will not use them. We can remove the aliases entirely at a later point.
Use $(D normalRNG) ?
`s/Elements/Element/`
Put natively supports a range as input, so you can replace this with `put(target, source);`.

I know the `for loop` wasn't yours, but if you touch it, might as well improve it.
On 07/24/2012 02:03 PM, Pete Muir wrote:
>> +    private final ParallelAccess parallelBean;
>> +
>> +    /**
>> +     * Constructor to prepare the client-context.<br/>
>> +     * There must be a jboss-ejb-client.properties file in the classpath to specify the server connection(s).
>> +     *
>> +     * @throws NamingException
>> +     */
>> +    private AsynchronousClient() throws NamingException {
>> +        final Hashtable<String, String> jndiProperties = new Hashtable<String, String>();
>> +        jndiProperties.put(Context.URL_PKG_PREFIXES, "org.jboss.ejb.client.naming");
>> +        final Context context = new InitialContext(jndiProperties);
>> +        String lookupName = "ejb:/jboss-as-ejb-asynchronous//AsynchronousAccessBean!" + AsynchronousAccess.class.getName();
>> +        LOGGER.info("Lookup Bean >" + lookupName);
>> +        accessBean = (AsynchronousAccess) context.lookup(lookupName);
>> +        lookupName = "ejb:/jboss-as-ejb-asynchronous//ParallelAccessBean!" + ParallelAccess.class.getName();
> Do we need the double // here?
I don't understand what you mean here ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jboss-jdf/jboss-as-quickstart/pull/290/files#r1224031
You can remove this, we've removed the png fix for IE6 as it was causing IE6 to crash
Why were the RDoc tasks added?
perhaps this should be a log message?
I think the docs here and on the next 2 lines should be indented.
You should definitely list yourself as one of the contributors, if not the main author.
Should something else happen in FX35 here?
The logic for this calculation is wrong. This concept refers to the _scheduled_ return date that is recorded at the end of a visit. Loss-to-Followup should be based on actual visits, not scheduled ones.
I didn't think that the HTTP2 library allowed non-strings in the header. Have you tried passing non-strings to see if it works? If not, it would be safest to replace this, to make the release easier.
I was wondering if this doc is the right scope to say how to download that.
I understand this is just an exception notifier, but should be in a log, not puts.
This is embarrassing, next time I will not try to hurry a commit
We can instead use window.location.replace on it.
An array cast seems to be missing here for people that pass just a string, otherwise we need to type hint the array in the parameter.
oh cool.  great tip!
This is a string. I'm only talking about parenthesis in control structures and function calls. Maybe i need to make this a little more clear.
I was influenced by createEvaluationContext() in HttpRequestHandlingEndpointSupport, which creates a new instance for each request.

Since we set the root object, doesn't that imply it's not thread-safe?
No, that's what I was trying to say. It does not. The worst case scenario you may end up with few lingering temporary queues, that will be deleted at the system restart. However let me see if i can move some logic to the onInit method, then may be this whole argument will be invalid
The main change, shifting $dsl before processing arguments. This allows a cleaner syntax in the plugin code ($dsl->keyword)
What if you start with:

    var paths = [ path.dirname(lessPath) ];

and then do the `forEach` for the paths? Then you can remove lines 144-148.
I don't think arrays should be considered primitive types. In JavaScript, arrays are just objects with a few extra methods/properties.
I had a coworker confirm that there are multiple `COMMIT`s on RHEL as well.
@apex13 Thanks for spotting, I also updated other references.
We shouldn't be adding the .architect files to the repo. They only carry user-specific state so there's no use in sharing them.

In fact if you want to add it to a .gitignore that would be helpful.
Nevermind, check my latest commit to see what I have: serializer.object['user'] is the right object.
Note that this will probably fix the `ValueError: The least populated class in y has only 1 member, which is too few` issue below.
4 spaces:

    $ cmd


Le 28 nov. 2012 à 20:24, Léo Willian Kölln <notifications@github.com> a écrit :

> In Resources/doc/README.markdown:
> 
> >  
> > -Add this bundle as a submodule:
> > +Run the composer to download the bundle:
> > +``` bash
> What is the standard markdown for command lines? Found nothing about it, sorry.
> 
> —
> Reply to this email directly or view it on GitHub.
>
I am guessing that this is a work in progress (otherwise there are a lot of declared but not used variables)
done and its working
It should be <ul role="listbox">
should be 

```<li role="option">```
This was oddly one of my first fixes, surprised it didn't survive. Must have gotten munged somewhere. Thanks for fixing.
> These are generated on the server for existing projects for identification purposes. New projects coming from plugins arrive with coordinates. In maven projects we use the GAV. In freestyle projects we try to figure it out from the build properties or an explicit parameter (For Example the ant task). For freestyle projects put the build / job / task name in artifactId and build number in version.

Ok. so I figured correctly, that it needs to be provided by the agent, but erroneously used a GUID as the default for freestyle projects rather than the job name - I'll update this pull request accordingly.

> Any new insights on the maven functionality ?

I've meanwhile figured out a way to start with explicit project type selection by the user initially (i.e. Maven vs. freestyle vs. ...), yet allow to simply plug in an automatic project type detection later on, so the latter won't hold back release 1.0.0; as mentioned I expect to submit a (separate) pull request covering the Maven functionality over the course of this week.
These property names should be constants in TiC (baseURL, mimeType)
What about `IdentityFactory`, because it just return the same data/instance ?

With a `retrieve` / `get` method ?
Could you use ```isset``` instead of ```!empty``` ?
Argh. Of course. I'll fix that.
You mean using require(...) every time?
I don't think they are moving away from attaching extra objects to navigator.
I'd like to see the default binding router do more than just return the binding the same way as previously.  Adding a version of dot notation nesting would continue to support single layer binding objects while also allowing for more organization and namespacing in larger binding objects.

I'd like to be able to write binding objects that are more understandable and readable without having to write my own binding router.  

For example: 

```javascript
var bindings = {
	list: {
		"foreach": function() {
			return {
				"foreach": ...
			}
		}
		item: {
			"name": {
				"text": ...
			}
		}
	}
}
```

```html
<ul data-class="list.foreach">
  <li data-class="list.item.name"></li>
</ul>
```
That test already exists in ui-framework.key-input.
`var mUrl = $sn.getAttr($(this), 'url');`
Use just 'Clear the chat box',
Ah, name collision right? Perhaps you can name the local path variable something else?
what happens if its already in complete state?
@jeffheifetz Yea i saw it. Thanks.