labofsahil/github-event-dataset-2012 huggingface dataset:
About the store_loot, the database is filled with this format:

<pre>
msf  post(gpg_creds) > hosts

Hosts
=====

address        mac  name  os_name  os_flavor  os_sp  purpose  info  comments
-------        ---  ----  -------  ---------  -----  -------  ----  --------
192.168.1.128                                                       

msf  post(gpg_creds) > loot

Loot
====

host           service  type              name              content     info                     path
----           -------  ----              ----              -------     ----                     ----
192.168.1.128           gpg.trustdb.gpg   gpg_trustdb.gpg   text/plain  GnuPG trustdb.gpg File   /Users/juan/.msf4/loot/20121003230736_gpg_192.168.1.128_gpg.trustdb.gpg_599823.txt
192.168.1.128           gpg.secring.gpg   gpg_secring.gpg   text/plain  GnuPG secring.gpg File   /Users/juan/.msf4/loot/20121003230736_gpg_192.168.1.128_gpg.secring.gpg_556776.txt
192.168.1.128           gpg.random_seed   gpg_random_seed   text/plain  GnuPG random_seed File   /Users/juan/.msf4/loot/20121003230736_gpg_192.168.1.128_gpg.random_seed_145482.txt
192.168.1.128           gpg.pubring.gpg~  gpg_pubring.gpg~  text/plain  GnuPG pubring.gpg~ File  /Users/juan/.msf4/loot/20121003230735_gpg_192.168.1.128_gpg.pubring.gpg_413053.txt
192.168.1.128           gpg.pubring.gpg   gpg_pubring.gpg   text/plain  GnuPG pubring.gpg File   /Users/juan/.msf4/loot/20121003230735_gpg_192.168.1.128_gpg.pubring.gpg_057816.txt
192.168.1.128           gpg.gpg.conf      gpg_gpg.conf      text/plain  GnuPG gpg.conf File      /Users/juan/.msf4/loot/20121003230735_gpg_192.168.1.128_gpg.gpg.conf_245681.txt
</pre>

* I think the types should be corrected, as noticed by the store_loot documentation:

<pre>
Also stores metadata about the file in the database when available +ltype+ is an OID-style loot type, e.g. "cisco.ios.config".  Ignored when no database is connected.
</pre>

Just: gpg.trustdb, gpg.secring, gpg.random_seed, gpg.pubring, gpg.conf types have more sense for me

* When using reverse payload the host stored in the database is from the handler listening for a connection. It's a little confusing when reviewing database results. But others modules looks equal, so not sure if it should be just fixed in stored_loot, since it accepts a session as argument.
lib/msf and lib/rex should be included, everything else is third-party
Glad you like it. Have you seen the output of the example?
Should I do it, or were you volunteering? :)  Just checking!
Sorry if I'm missing something, but why does this function have to be written in Cython and not in simple Python? This double loop could be written as

    thresh_image = image > (thresh_image - offset)

(btw, sorry to join this discussion very late)
Hmm, 
symbol-at-point (used for goto-definition)
type-at-point
expand-scope

These are light-weight tasks that shouldn't really demand that you write to disk.  Of course, if people use source-control, they shouldn't *really* care when a file gets written to disk, but I think there is a general expectation that the user *decides* when to write to disk.
This blank line should go after line 4, not before - group the `require`s.
Kill this extra line please. Code should butt up against the docstring.
TODO: indent
Достаточно менять в скрипте, по изменении версии
I support the creation of a set of standard splits of any dataset, but I would favor canonizing a minimal, language-agnostic materialization of said standard splits rather than defining them implicitly by code in a specific language that must be run correctly in a specific environment to regenerate the right splits. Concretely, I would advocate looking for the simplest-possible JSON format that would completely specify, say, ten 90/10 splits with reference to relative image paths within a canonical archive of a dataset. 

That's just my opinion, though, and it may not fit the way scikit-data already works!
This doesn't show up in the generated doc.  I think it is missing an `@exports`.
What happens now if you put a movie in your download folder manually?
i was taking a queue from iOS.3dmatrix.yml update that arthur made recently, i thought that is how it was to updated. Either ways i will put in a description line pointing to the newdoc (which i forgot to include in the commits) for the sample code in the next update.
Might want to remove the "REMOVED IN 1.9" ;-).
grammar (now instead of know?)
Innner rect radii can not be the same as outer rect radii
innerRadius = outerRadius - padding;if(innerRadius < 0){innerRadius=0;}
It's never used outside of the context of being a loop iterator, but this is an undocumented style guideline so this can be accepted for now.
This should be `valueOf` :)
As num doesn't change how 40 is used, go with var.
This module is for intel stuff, no? Why add GCC thingies?
no its probably fine , just need to monitor the test runs/changes on CI builds.
`emitter` is a better name than `ee` if you're concerned about length.
или это плохо наверное, оно ж глобальное будет (
generateCaptch
AFAIK all tests pass. I am adding some more now and will push a new commit shortly. Let's see whether things still fail for you then.
This class should go into a separate module `bioconductor.py`.

We're not doing that yet for Python packages (numpy, scipy, ...), but we should.
Sent from my iPhone

On 31/07/2012, at 11:48 PM, Pete Muir<reply@reply.github.com> wrote:

>> +         courtesy of <a href="http://translate.google.com">http://translate.google.com</a> for demonstration.
>> +         My apologies if they are less than ideal translations.
>> +         </p>
>> +         
>> +         <p>
>> +         The different methods of the JAX-RS service can be access using the URLs below:
>> +         </p>
>> +         <ul>
>> +            <li>
>> +         <code>http://hostname:8080/jboss-as-logging-tools/rest/greetings/<b>name</b></code>
>> +         <p>Simple use of localised messages (with parameter) and logging.</p>
>> +         <ul>
>> +            <li>Returns a localised "hello `name`" string where `name` is the last component of the URL.</li>
>> +            <li>Logs a localised "Hello message sent"</li>
>> +         </ul>
>> +               <p>Example: <a href="http://127.0.0.1:8080/jboss-as-logging-tools/rest/greetings/Harold">http://127.0.0.1:8080/jboss-as-logging-tools/rest/greetings/Harold</a></p>
> 
> Should this be a relative URL to handle deployments to other than localhost and on other ports
> 

*facepalm* Doh! And other pop culture themed references to me missing the obvious. :-(

Fixing


> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jboss-jdf/jboss-as-quickstart/pull/320/files#r1273119
It could loop indefinitly. But in a separate thread and always with a 30 second pause while invoking the method
Shouldn't we do this then?
sneaky.
Is this using the AuthenticatedUser as the owner of the pull request?
This line is missing a closing quote, and breaks the validity of this JSON file.
Can we DRY some of this code with the general raxa time code? Perhaps we can pass in optional arguments through the `params` argument.
Fix alignment if this affects the resulted README
You don't use this any longer
as this uses get_file_status() on each file and get_file_status fails the whole request if it cant get a file it means that trying to get the contents of any dir that is not readable fails.

Usually this is fine cos mcollective runs as root but if selinux for example denies access to the process you will get no results rather than say partial results.

<pre>
% mco rpc filemgr list dir=/etc

devco.net-0                              Request Aborted
   Could not list directory '/etc': /etc/libaudit.conf - permission denied
   File details: {}
</pre>

Also not sure if converting the filename to a symbol is needed here, eventually mcollective will switch to JSON and symbols in data like this wont work - safest to just keep those as strings now in the case where its a deep nested structure
I don't think that PuppetDB is compatible with Puppet 2.6, but it wouldn't hurt to remove this anyway.  Thanks for the note!
I think ``exclusion_policy: ALL`` is missing here.
Can you remove one of these empty lines?
This can be removed as well I think.
Sass appears to have [@warn](http://sass-lang.com/docs/yardoc/file.SASS_CHANGELOG.html#_directive), although it is off by default in Rails. :unamused:
incresing the default value for keepAlive might reduce the likelihood  of hot spot invalidations ( @Sanne can explain this in more detail, or even better blog about it :-). I'd increse this value to 1 min.
Thanks John, I have opened an [issue](https://github.com/matthew-brett/gitwash/issues/3) in that repository and trust that  it will be updated appropriately.
looks great!  I just had to add an i+1 since steps start with 1 instead of 0.  But very cool!  :)
Bugzilla entry?
Let's leave the file extension unchanged.
MINOR: Comment refers to log_writer::get_real_process_name which does not exist.
use sprintf("%s?generatedBy=%s", createUri, generatingActivity$properties$id).

just a suggestion...
Hmm, i wonder if it even should be legal? Unless I am missing something name="@testBean1.property" should resolve to literal '@testBean1.property', right?
Isn't this only true if there is an explicit reply destination?
Thanks, Gunnar, for explanation and your so wide view.
However, we can get here **file path** and convert it to `LiteralExpression`.
And mark this constructor as @Deprecated

Nevertheless, my apologies, if I try to discuss here the task out of the issue.
nam**e**space 

throw an <classname>BeanDefinitionParsingException</classname>
@Ocramius  Yes.
Y U NO FIX THIS?
Yep just an oversight, updated
I'm not sure, but I think this needs to be an Array.
After thinking about it, we should also override the construct method and have it also return an unknown type.
you will want to fix this, or else the test will always fail.
Could we put this highlighter/ folder in js/highlighter. I don't want to clutter the top-level directory with libraries.
think this rather needs to be a functional test, since the SQL will be different for each supported platform.
the test should also better be added to the phpcr-api-test suite ..
Thing is, using a property makes it less obvious for a user that wants to instantiate authentications himself. With `get_authentications`, you can just subclass and override ... with a property - yes it works too, but it is somehow ugly.
Also, if you refer to Django's class-based views, the `get_form`, `get_form_class` is used, not a property.
Let's keep these are they are for now.  It *might* make sense for them to be this way around, but if so it should happen as a separate pull req , that affects all the relations tests.
Could you use `.pluck(:published_at)` ?
Some additional tests to cover case insensitive path comparison would be good.
Make this static otherwise it'll have an object relationship.
It'd be nice to do something in the future to avoid hardcoding the src of the sidebar.
Well, the negative classname is meant to be for a kind of style applied in actions that implies "Cancel" or "Delete".
If you can provide any example of better naming please, do it.
Yeah im pretty agree with it, that description is old, and works whene we did not have the code merged.
I will change for something more accurated
The Container adaptor does not enhance the deployment. The Protocol does.
I think this would make more sense as:

```
if (!totalBytes)
  totalBytes = Math.max(1000000, bytesSoFar);
progress(0.1 + 0.7 * bytesSoFar / totalBytes);
```
Should this be a configuration option? I'm not familiar enough with differences between `closurebuilder.py` and `calcdeps.py`.
Or alternatively declare some "null"-string that is valid anywhere.
Using the keyname in this "null-string" seems to make no sense.
Why is this moved 1 tab to right?

Also, `break` should be on the same level as `case`:
```
case 'somecase':
   do_stuff()...
break;
```

Whatsmore, you do not use tabs, but spaces. Please, switch it to tabs.
На ычане сажи нътъ. Детектировать енто поле нам ни к чему.
Ну не знаю, у меня в лисе всё работает. По вертикали ничего не правил:

![Snap1](https://f.cloud.github.com/assets/1483359/32584/7445edc0-502c-11e2-8f2b-1675908f359e.jpg)
Do this on the request instead.  `get_authentication`
Do we expect a particular value that can be tested with toHaveBeenCalledWith?
regular configure will work right?
It has been placed on its own line.
Jak lista będzie w liczbie mnogiej do pojedynczego towaru możesz użyć pełnej nazwy.
What's this template based on? The Stop Climate Chaos Coalition?
A corriger : "qu'elle meure"