labofsahil/github-event-dataset-2012 huggingface dataset:
I probably named this class, but is just `Rectangle` a better name than `BoundingRectangle`?  Or is the former too generic?
Public properties need doc.
Yep, you're right.  Scott has a utility function for this that we should do a system-wide check on.  This isn't as simple as the lhs find and replace, so I'll just fix it here for now.
I was hoping that we could pass ownership to the `Command` object and call its `destroy` function, but I guess the common case is that we will have multiple `Command` objects pointing to the same resources so we would need to reference count or explicitly/implicitly assign ownership (like the `Framebuffer` and `VertexArray`, which are really just containers).  Did you have any ideas?  This isn't necessarily in the scope of this pull request.
Since this is a Sandcastle example, I changed it to this:
```
var modelPosition = new Cesium.Cartesian3(positions[i], positions[i + 1], positions[i + 2]);
var worldPosition = this._modelMatrix.multiplyByPoint(modelPosition);
```
will fix that
Can we change 'already_module_ids' to something like 'already_included_module_ids' or 'included_module_ids' ?
This needs to go away. Ti.UI.TextField will do all of this stuff for you in MUCH less code
the check for isFinishing and instanceof should be wrapping the call to show I would think?
Shall it be replaced by `mouseenter` and `mouseleave`?
Do you think I should add some more content that's really more about implementation? The a2mod is perfectly acceptable, since it's a resource type that operates behind the scenes and isn't meant for the user interface.
This can be done simpler:

Resources.toString(getClass().getResource("/parsers/getConfigWithOSPF.xml"), Charset.forName("UTF-8"));
and again (throughout)
gratuitous use of mixin (not just here, but all over)?  How about just:  enum limit  = codeUnitLimit!S;  Does limit need to be one of the char types or is an int ok?
Nitpick: You inserted some \t characters in there. Replace them with spaces?

Ditto inside `T* emplace(T)(T* chunk)`, line 3408.
in this simple example, I'd prefer not to refer to the autogenerated imports, but instead import watchers explicitly (like you are doing with dart:html above)
shoudln't it be ``not`` ?
I meant the openshift profile :-)
`switch`.
I use the metadata down there at line 74
If I output the urls, they worked with and without the "core" being specified, but I agree, there should be a default.  I've updated the PR with a default.

Thanks!
hmh maybe better will be ```fos_rest.request.param_converter.request_content``` because what if later someone will have an idea of making another param_converters?
I always break that one. Thanks for sharing, good to know.
Often the answer is surprisingly 'no', Uri is great for getting and parsing info, but not for building new URIs from others afaik.
Think I'll merge as is, then just add our own OptionParser.get_usage
Sounds much better; didn't know that existed. Don't think persisting both stacks matters too much. I'll push some updates tomorrow.
Inherit from ArgumentError maybe?
Whenever I use `assertNumQueries`, I document in a comment exactly what the expected queries are. Usually you had to figure this out in order to get the right number for the assertion in the first place - if you don't document that info, you're condemning someone to redo all that work whenever the test breaks or the relevant code changes.
Should be using `.on()` and `.off()` in this function
Just kill it. Actually in the current compiler, deprecating something is *far* more painful to users than removing it. So it's better to remove it completely.
Nobody could be running on Itanium at the moment anyway.
(Fix Submitted) memset the whole struct to avoid further issues.
admin settings to expose the results limit. You can fill out the numbers you want to see exposed, like: 10, 25, 50, 100. These will turn into links.
Shouldn't this be contentType?
The "connection-factory" is not a *required* attribute in the schema, so we probably want to provide some validation at this level.

In fact, I think it should be consistent with the existing redis inbound-channel-adapter where we use "redisConnectionFactory" as a default bean name, e.g.:

{code}
String connectionFactory = element.getAttribute("connection-factory");
if (!StringUtils.hasText(connectionFactory)) {
    connectionFactory = "redisConnectionFactory";
}
{code}
We are assuming that the expression will always yield a String. What if it yields a java.io.File? Do you think we can have a check if the passed instance is of type java.io.File too?
We think it would be better to replace the curl_host configuration option with "host" and "base_url" configuration options. Thanks for pointing out this deficiency. Would you mind changing that?
Varnish does not, and AFAIK has no plans to add it. Pull requests accepted?

https://www.varnish-cache.org/lists/pipermail/varnish-misc/2012-March/021796.html

Saint and grace mode (ie server configuration) rather than origin server directives are the supported mechanism.
The exception is only thrown when someone willingly manipulates the input (blank ID or invalid URL). So I don't find it worth implementing this for the client side.
Unfortunately I'm not sure.
You can try to remove it: if there is no modification on the Sample behavior, it means it's not useful. Otherwise adding a comment why it's useful seems necessary.
=> The WEBOTS macro is a good way to fork the framework. (The code of Olivier related with the read only motion file should be encapsulated this way)
not yet removed?
У Тебя опечатка в слове *классифика**Ц**ия* =)
Should we say it's based on PEP 8? @hzr
User-Agent is not settable for browsers that support XHR2. It always returns the current browser's user agent in this case and cannot be overridden. While this use to be a viable approach, it doesn't work with most modern browsers. In these cases, the server(s) will just have to be updated.
On Thu, Mar 8, 2012 at 10:13 AM, Daniel Pittman <
reply@reply.github.com
> wrote:

> > @@ -0,0 +1,17 @@
> > +module Facter
> > +  module Util
> > +    module PuppetSettings
> > +      class << self
>
> `def self.with_puppet` is generally preferable - because you can clearly
> identify each function individually.
>

Got it, I'll keep this in mind.

For what it's worth, I was livid when I had to go learn about the
eigenclass when I was first getting started with Puppet a few years ago.
I'm surprised at myself for repeating this mistake of using class << self
with the intent of defining a class method.
There actually needs to be a type field, or the doc validation fails
0 and "" are not valid values. The only valid values are instances of Ti.UI.PickerRow.
Lets the Application override this method to call the IP bypass flow, or the regular flow.
This seems to work, but I am curious why you stopped using the factories here?
dev1 will be merged with #100. You must use 0.7.2, because this is _dev.php script - it must update each PR individually.

And yes, you can use `-` in UMIL. Even though I am used to use dashed versions (using `-`), php's `version_compare()` replaces it to dotted version as you use. I consider this being less readable (any yet is should be `0.7.2.dev.2` to be exact), but it is just my personal opinion...
I'm using it for integration tests in APE [my fork] and I don't have jta in the classpath - looks fine :)
Just ran this through webinspector:
translatePath("local://this/is/a/double/slash/local/path")
"local://this/is/a/double/slash/local/path"
translatePath("local:///this/is/a/triple/slash/local/path")
"file:///accounts/1000/appdata/WebWorksTest.testDev_ebWorksTest99346a60/data/../app/native/this/is/a/triple/slash/local/path"

local:// paths are left untranslated
This is not called anywhere?
Thanks for TestDox link. They were renamed because I wrote an entirely new file when I didn't understand that `utils.py` was actually a testing file.
I think you should combine this with the File write method