labofsahil/github-event-dataset-2012 huggingface dataset:
Is this necessary?
```
this.factories = factories;
Assert.notEmpty(. . .)
findAConnection();
. . .
```
I think that storing the text like this is perfectly fine, for example if we replaced the text1-text4 with text then what if i only wanted to show the text1 line (unlikely but possible)? I think keeping it the same as the output from the NBT code makes it more standard and flexible when more entities get added.
We agree with you. Thats why we suggest above to fix this.

To set value from "optional" to "required"
https://github.com/anandu/rightscale_cookbooks/blob/pgfixes_rebased/cookbooks/app_tomcat/metadata.rb#L89

and to remove set value lines for datasource_name
https://github.com/anandu/rightscale_cookbooks/blob/pgfixes_rebased/cookbooks/app_tomcat/attributes/default.rb#L48
https://github.com/anandu/rightscale_cookbooks/blob/pgfixes_rebased/cookbooks/app_tomcat/attributes/default.rb#L51
if user has /log directory in his app folder structure, rails will be trying to put logs there first.
and will write to passenger logs, only if /log folder is unavailable or it has no rights to write there.
Can you update your `master` branch? This PR is linked to your fork
@robertbasic @Freeaqingme  ok, updated
this extra empty line between the phpdoc and the class declaration should be removed
型パラメータのコーディング規約によると ENUM は
Enum か E とかにすべきらしいです。
参考 → http://docs.scala-lang.org/style/naming-conventions.html
NAME is one type of property in PHPCR. it is a string with additional validation of being a valid xml name (localname and optional namespace prefix with a defined prefix)
why fallback and not just provider?
More than just sql files -- you might want to put default keys, sqlite databases, and more in this directory -- it's _data_ the module may need to consume, but not necessarily restricted to _databases_.
Sure, I'll fix that this weekend
You can remove the bundle name from the title. We know it is the FOSCommentBundle doc :smile:
I agree that English text should be moved to DisplayConstants as statically defined strings, but that's beyond the scope of this commit.  (I was just sick of seeing "occurred" misspelled!)  I created https://sagebionetworks.jira.com/browse/SWC-271 to capture this.
else前要有一个空格
Yes it does `<Person>` :) 
So, I removed ' and also changed  *d* in *not be d in a*.

On 4. apr. 2012, at 11:13, Mitar wrote:

>> +
>> +    def __init__(self, of, attribute, related_name=None, default=NOT_PROVIDED, null=False, blank=False, readonly=False, full=False, unique=False, help_text=None):
>> +        super(EmbeddedListField, self).__init__(to=of, 
>> +                                                 attribute=attribute,
>> +                                                 related_name=related_name,
>> +                                                 # default=default, 
>> +                                                 null=null, 
>> +                                                 # blank=blank, 
>> +                                                 # readonly=readonly, 
>> +                                                 full=full, 
>> +                                                 unique=unique, 
>> +                                                 help_text=help_text)
>> +    def dehydrate(self, bundle):
>> +        if not bundle.obj or not bundle.obj.pk:
>> +            if not self.null:
>> +                raise ApiFieldError("The document '%r' does not have a primary key and can not be d in a ToMany context." % bundle.obj)
> 
> But `%r` creates something like `<Person>` no?
> 
> And what is that *d* in *not be d in a*?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mitar/django-tastypie-mongoengine/pull/1/files#r643841
This bit I'm not super-keen on.  Let's instead change the signature of `field_from_native(data, files, field_name, reverted_data)`
This line: https://github.com/tomchristie/django-rest-framework/blob/master/rest_framework/fields.py#L178
could then eg. branch depending on if a `._use_files` attribute is set on the Field.  That way we don't need to do brittle type checking.
This almost exactly duplicates `__render_status` in `world.managers.statusiconmanager`.
While i personally think that's fine for testing purposes, you could add a docstring explaining this, perhaps some TODO comments in both files about the re-unification and so on -- just making sure we don't forget about it later.
This can just be `get_res_icon_path(res)` since the remaining arguments default to 32 and False already.
Please fix indentation of this line.
It should be an easy re-base. I just removed the 2 links that were annoying at best and added an actual test case residing in the local, no more whitelisting issues.
sign_up guest
I don't think you need the typehint here.
And looking at this particular example, I'd agree that a category is a better idea here, since the added functionality is more utility-based than CDVPlugin-specific.
Check my comment in your previous comment about the same thing:

"Nope, it can't be a div because we need the click area provided by the label, wich is the one that recives the clicks and sends the event to the checkbox for checking it. (Same behaviour as "for" attribute)"
Что-то тут не так с запятыми, или мне кажется? @pepelsbey призывается.
A comment about the apparent non-existence of saveInSentItems, or it at least not mattering for gmail (which currently uses this version) I think would be helpful for people who look at the 14 case and then this case and wonder what's going on with that (like me!), even if the answer is that we're not sure.
As Byk mentioned on IRC urllib2 should make use of the default proxy by default. I simply mixed it up with httplib2.
паскаль?
so this is static?  should it maybe go in a static initializer?  (is it possible for this constructor to occur more than once somehow during the life of an app?)
Ha.  You couldn't resist could you? ;)
Can we just totally refactor this function out? How many times is it actually called?
This is wrong. Will return true when typeof is undefined. 

And, you have verify undefined like this: (typeof dictionary[value] !== 'undefined') because typeof always returns a String value.

References: https://developer.mozilla.org/en/JavaScript/Reference/Operators/typeof
`new Array()` аналогично `[]`
Add Assert.hasText(...)
Да, можно, только нужно ещё добавить поддержку тегов.
A `slice(0)` returns the same string, correct?
These functions should go in utils.js. I believe we already have something for copying files as well.
Add a comment that the URL should return a JSON response, and add an example perhaps?
Oops, I skimmed the announcement too quickly.