labofsahil/github-event-dataset-2012 huggingface dataset:
Ah, I didn't notice that part.
AlbumTableTest
Can't we use prettyprint here?

We should also remove all the spaces between the property and the property value.
e.g. `"enabled": true,`
can you remove the space between ``!`` and ``$`` ?
use @test instead of testXXXXXXXX()
phpunit is already installed in Travis VMs.
`assertInstanceOf` uses the FQCN, no need for `\` in front of the class name
Take a look at this again ... let's just try to cover all possible scenarios with how names are entered.
how about idention with twig and js?
it shoud be:
```
{% if thread_container is defined %}
var	fos_comment_thread_container = $('{{ thread_container }}');
{% else %}
var fos_comment_thread_container = $('#fos_comment_thread');
{% endif %}
```
or 

```
{% if thread_container is defined %}
    var	fos_comment_thread_container = $('{{ thread_container }}');
{% else %}
    var fos_comment_thread_container = $('#fos_comment_thread');
{% endif %}
```
Update arrived! :sparkles:
I think it needs a to_python like ModelChoiceField

    def to_python(self, value):
        try:
            key = self.to_field_name or 'code'
            value = Code.objects.get(**{key: value})
        except (ValueError, Code.DoesNotExist):
            raise ValidationError('Enter a valid registration code.')
        return value
Drop the whitespace insertion.
In branch 2.0 of this bundle, the BeSimple Yaml file loader override the Yaml file loader of symfony.
The declaration to declare a route for BeSimpleI18nRouting or Symfony do not have the same format but the YamlFileLoader of BeSimple check if the route is a BeSimpleI18nRoute or a Symfony route.

Add a check if the type is `be_simple_i18n` or `yaml` is useless. The YamlFileLoader is the same for the both types.

The type `be_simple_i18n` exists only in master of this bundle and the works with Sf2.1.
Alternatively to marking the entire paragraph as changed, you can also add new words as a "phrase" like this: 

_... &lt;phrase revisionflag="added"&gt; new wording&lt;/phrase&gt; ..._

That way only the added parts are rendered green in the resulting document, making it a bit easier to see the actual changes.
I remember one thing that I dislike about the plot directive: it is not executed in its directory. As a result, the file paths in the file are akward, and it leads to files that are not really readily executable.
Alright, then leave it as is.
I would rather do:

```python
if self.class_weight == 'auto':
    self.class_weight_ = # use the y distribution here as done in other classes
elif self.class_weight is None:
    self.class_weight_ = {}
else:
    self.class_weight_ = self.class_weight
```

The separate trailing space attribute is necessary IMHO: 2 consecutive calls to `fit` of a model with `class_weight='auto'` should be independent.
alan man bayad begam tage meta ro beband :rage:
Newer mind, saw the netfields/fields/__init__.py file while working my way done through the diff :-)
Done!
https://github.com/telefonicaid/Gaia-UI-Building-Blocks/commit/f7210797f6f0d167490f62b451d2e1f1cca84bfe
Great! i've added it!
https://github.com/mozilla-b2g/Gaia-UI-Building-Blocks/pull/29#commits-pushed-078777a
Since I don't anticipate this codebase being very large, I'm not too concerned about global namespace clutter, but I like the way you think!  I'll make changes to implement the first type of namespacing you suggested.
I'm not sure I understand - this doesn't have an input file, this is a source.  How could it cause a problem?
why assigning new variables here ?
Right, well hence the option to make the lib directory configurable. Some people like to freeze the checkout (ex, team environments), whereas you may want to maintain a checkout somewhere else. I don't think we should be removing this option.
This is a new dependancy, can you confirm which mocking library this is.
I think moving this stuff to a separate file is a great idea.
It will cause double Close, which does not cause problems.
Brackets, brackets, brackets (for all 3 conditions here).
There is already written to run install_sn.php :)
Use tabs, please. Everywhere.
May want to do a null check assertion.
Since we can't mock objects the only thing i can think of is to store the old values for process.env.HOMEPATH and process.env.HOMEDRIVE and re-set them when your test is over. Even this i don't really like... because if the test fails midway, the objects won't be reset.
Do you really want me to change it or you're just kidding?
What happens if the action Id is not specified by the user? ie, they pass in an empty string? Should we notify them of this?
its more logical to add it after its created
'params' -> 'headers'
... That would do it.  Let me fix that real quick.
Please could you take a look to this solution https://github.com/doctrine/doctrine2/pull/401#issuecomment-7030156
I assume this will work better for a problem I'm facing. I had some trouble with deployments having war files that are unexploded. 
http://grails.1312388.n4.nabble.com/lesscss-resources-not-working-in-production-tt4429062.html has the discussion for it.