labofsahil/github-event-dataset-2012 huggingface dataset:
The port on scheme is mandatory. 

If the port used is not 80, should be informed along with the url like in browser (http://myserver:4567/myproject/mypipe).  If is not informed the HttpClient assumes 80

Also we don't need open a separete thread in this layer. This is already being done in pipe
yah. active resource load the user, then active resource request subscriptions by user id
You don't need "return" in Ruby.
I'm suggesting you removing the chaining in the code above and do this instead: 


volumeControl.mouseenter(function() {
    tooltip.updateContent(that.options.strings.volume);
});

volumeControl.mouseleave(function() {
    tooltip.updateContent(muteButton.options.defaultTooltipContentFunction);
});
так надо просто использовать что-то, что не требует break и все)
a new test case with bigger code added to validate the correctness of the logic. .
s/element_selected/selected_element
I'm not sure what you mean, it's specifically looking for something that can be a class name in order to check the JMS metadata factory.  All of the checking done here is against strings in the `type` column listed here: http://jmsyst.com/bundles/JMSSerializerBundle/master/reference/annotations#type
It should be ``array<ApiDoc>`` or ``ApiDoc[]``. You are mixing things
The request will get retained by the connection, so we can release that right away. 
The connection object should be released at the very end of the didFinishLoading: and didFailWithError: callbacks.
Added your last suggestion as a reference.

In addition to that, I tried to reproduce the optimized kernels. Though I get slightly different results for sigma. You can find the code in the Dropbox folder you shared with me.
Typo: `differentiable`
you mean in _center_data?
Saints at Packers is at 4:15
Probably better as:

   write object, doc, and di file(s) to path of corresponding source file(s)
Or something like:
```
if (p[1] & 0x80)
{
    c = decodeUTF();
    if (isUniAlpha(c))
        goto done;
}
```
but would that be double-decoding? (here, and later when the identifier is parsed)
You need to update the Font apidoc as well
Change this to use _createControlHelper.  Also change the other ones not using it please.
any specific reason why ndk could be None before and can't now?  I think having the ndk as None and autodetected is still valid and especially practical for us during dev as studio should always be providing that value
Done.
Can't have multiple values for a key. I'm not sure we support anything but UTF-8.
Bring back the following instead of using pos_args[0]
parser.add_argument('command', choices=['build', 'run'], help='commands')
not necessary because it is a static method
`Qiuery` > `Query`
Just following the other actions. Might do a clean over later on all of them, just need to get them all straight first.
You can delete references to "Hole", but can be part of a separate commit..
If an exception happens before this point, should the method/catch ensure isactive is set to false?
No this is formatted correctly.
The <programlisting> element, much like the HTML <pre> tag,
is a monospace, fixed width font, and this is now set precisely to fix the issue.


From: Gavin Halliday [mailto:notifications@github.com]
Sent: Wednesday, October 24, 2012 6:21 AM
To: hpcc-systems/HPCC-Platform
Cc: Panagiotatos, Greg (RIS-BCT)
Subject: Re: [HPCC-Platform] HPCC-8119 ParsePattern in HTML (#3567)


In docs/ECLLanguageReference/ECLR_mods/ParSppt-PARSPattrn.xml:

> @@ -397,27 +397,29 @@

>              <entry><emphasis role="bold">PATTERN</emphasis>('<emphasis>regular

>              expression</emphasis>')</entry>

>

> -            <entry><programlisting role="tab">Define a pattern using a <emphasis>regular expression </emphasis>built from the following

> -supported syntax elements:

> - (x)                        Grouping (not used for matching)

> - x|y                        Alteratives x or y

> - xy                         Concatenation of x and y.

> - x* x*?                     Zero or more. Greedy and minimal versions.

> - x+ x+?                     One or more. Greedy and minimal versions.

> - x? x??                     Zero or one. Greedy and minimal versions.

> - x{m} x{m,} x{m,n}          Bounded repeats, also minimal versions

> - [0-9abcdef]                A set of characters (may use ^ for exclusion list)

> - (?=…) (?!...)             Look ahead assertion

> - (?&lt;=…) (?&lt;!...)           Look behind assertion</programlisting><!--*** Note this and the following row entries have been monospace optimized for PDF/HTML*** --></entry>

> +            <entry><programlisting role="tab">Define a pattern using a <emphasis>regular expression </emphasis>built from

> +the following supported syntax elements:

> + (x)                Grouping (not used for matching)

Don't you want to try and make these line up?

—
Reply to this email directly or view it on GitHub<https://github.com/hpcc-systems/HPCC-Platform/pull/3567/files#r1926747>.
Before deletes were completely transactionless, with all that's nice that went with that "feature" (not).

This is a big improvement, but there's a window here, which with the refactoring done already here, I think is reasonably easy to close.
Not for this commit, but could do with some comments to a) explain the window b) explain how could be resolved.

The 'file's here being detached/deleted were locked by the transaction immediately prior to deleteFiles(), that calls the doDelete()'s.
I think the transaction commit could avoid releasing pending deletes, i.e. retain a cached list of IDistributedFile's it's about to delete, the doDelete()'s can then detach() only, avoiding the window, which currently allows others to gain an exclusive lock to the files in question.
I dont know, but there is. @ghalliday maybe you can explain?
From the helper....

	virtual const char * getFileName() {
		return "~REGRESS::MyThor::DG_Parent.d00";
	}
а зачем мы этот конфиг тут держим? он отличается от config/config.test.php.dist?
"өөр нэг хувилбар" гэвэл илүү оновчтой байх
`we need to use a script.` -> `we need to use a content script.`
One isset is better than additional 5 lines of code. DRY
I think this is more a matter of a bad name, cached_permission_with_groups does check groups AND the user.
Why use parseInt? It seems rather fragile. Regexp are made to simplify this use case already:
```javascript
string.replace(/%([0-9]+)/g, function (token, index) {
  var replacement = params[index - 1];
  ...
}
```
This is shit.
There is one, but that's just being used on the dash. Isn't it kind of irrelevant that there are links in the popover since it fades away before you can click them anyway? We can just use the artist tags, but add an extra class (`.tag-inline` or something) that makes them optionally display inline instead of stacked?
Magin Number 2..
Needs to RAUP -- see http://codeflow/Dashboard/review/display/mirobers-42c848286808456b80c01ef2645fb339

make sure we have test coverage for this

joe
Really I needed to add this delay because my test machine was consuming resources heavily after installing oracle and beehive. With this delay, finally, was sufficient to let the WMI service to process the mof file. My hope is this delay wouldn't be really necessary in a production environment. My test machine was really heavy loaded running Oracle Database and Beehive.
Fixed +:1+
It seems odd to only use the disallow failure message here.  With an integer matcher I would expect to see the integer failure here instead.  Maybe we should just use:
```ruby 
failed_matchers.last.failure_message
```
In theory the last submatcher added (at least in this case) would be the most specific so that would be the one we'd want to see.
Some formatting issues left here. ;)
Ugly, but you can do:

```css
#todo-list div :last-child .listItem {
    border-bottom: none;
}
```
@lsmith77 what really highlights that the issue is gone is the next testcase where different order are used. Having only one example could mean that the issue is still there but with the reverse order :p
No docstring?
Yea, maybe it should be moved just to config.
This file has 22 JSHint warnings that need to be fixed.
Actually, I think the best move is to return `Cartesian4` since this is Renderer level, and most - perhaps all - consumers will treat the return as a `vec4`.

Despite @shunter's expertise, I still disagree with you guys about the current code - everything is about trade-offs, and the trade-offs in the current code are not balanced.  There's a lot of code, doc, and tests that do nothing for a trivial increase in usability.

Anyway, returning `Cartesian4` solves this, and also avoids transforming the data between too many different formats.
- new hook for viewer
Thanks for pointing it out.  :smile:

I'll correct.
See how to use it via the other providers. I need to return a (negative) int.
No, this is stupid!
And then you would need to convert the DateTime value to a string, using a ISO8601<b>FROM</b>DateTime
Missing space. Also: acronym should be pluralized, too: "access control lists (ACLs)"
Could you quick replace "var" with "return", add "api" to the variables being declared at the top, and remove the "Ti.UI" dependency?
I've created ticket TIMOB-9182 to set the nativePath default value to null.
You can turn RELATED_NAMES_TYPE into an arraylist and loop through namesHashMap instead.  Then check during each iteration if the namesHashMap key is in the arraylist.  This will make it so you don't have to loop ALL of the name types for every entry.
I'm wondering if it would be better to just do something like:

    for (i in paramsSet) {
        (function(sizeType, sizeValue, startType, startValue, centerType, centerValue, endType, endValue) {
            // code goes here
        }).apply(paramsSet[i]);
    }

This should save a few bytes.
UIControlContentVerticalAlignment return type
``else:`` not necessary here, because you return in the exception handler.
Please rename this file to "index.html"
Don't put comments on the same line. Comments go above the code, with a capital first letter and a space after the slashes.
Well it was suggested in issue #2 and I thought it was a good idea.  It looks good too.
I wonder if callback arguments will be the same in both cases.
does it look only strange in this browser or is there really a formatting issue? (spaces/tabs)
Combined with the use of the term "Type" (which, as I've said already, is somewhat redundant), this sentence here seems confusing.

How about something akin to
> The FQSEN SHOULD be followed by a description, to specify what the Structural Element is being used for.
I THINK it might be because password_hash[element] in 1.9 is actually a String and not a Fixnum, and you can't XOR a string natively without opening up the String class and creating a ^ method.  I might need to cast that to Fixnum for it to work on 1.9.  As painful as it is to admit, I don't have RVM setup on this laptop so I can't test it....yet.  I'll try and get it setup and see what I can figure out tonight.
Fixed in 8fa1838
отстой какой-то эти две переменные на одну структуру данных, но лучше я придумать не могу; может абстрагируем хоть?
here you're not useing show.
If you don't use it at all, I'd remove it from the struct.
if you use get, then you also pass it along to caller. is that your intention? it could have unexpected side-effects.
Please use brace on its own line throughout.
And as I said, it makes about as much sense to say that a multimensional static array isn't a static array as it does to say that a yellow car isn't a car. 

I can see an argument for `staticArrayDimensions!int` giving `0`, since that's obviously an error condition rather than a valid number of dimensions for an array. But `MultidimensionalStaticArrayElementType` has no such error condition. It tries to treat scalars as if they were static arrays, which I contend makes no sense and is inconsistent with `ElementType`. It should either give `void` or not use a template constraint to prevent scalars working with it outright.

Clearly, we're not going to agree here, and this discussion is going nowhere.
ditto

Also, the combination lambda + named function is a bit odd. Leave it to only one example, otherwise just use several lambdas, one per line (as I assume will be the common idiom).
right, fixed that.
typo :o)
Both are a bit too clever but at least your doesn't count on overflow. Now use this version.
In the documentation of std.format, it seems to me that "a compound format specifier" equals to "a pair of two specifiers, %( and %)".
Then, in this case, _specifier_ is better, IMO.
_If you wan to stop it_ -> _To avoid this behavior_ : This is good. Thanks.
_add $(B '-') flag_ -> _add the $(B '-') flag_ : This is difficult... I can't decide which is better...
Can you fix up lookup_machine() too?
Can we add a couple tests that check that the generated DOM is correct for 'if'?

So far we have checks that show whether or not something is shown, but we should check whether it is placed in the right location in the DOM tree.
The only difference is the sample is expecting mdv_one_views.tmpl.dart instead of what is generated mdv_one_views.tmpl.tmpl.dart http://goo.gl/mrmgV

SRCFILE="mdv_one_views"
SRCFILE="$SRCFILE.tmpl" # now SRCFILE == "mdv_one_views.tmpl"
OUTFILE="$SRCFILE.tmpl.dart" # now OUTFILE == "mdv_one_views.tmpl.tmpl.dart"
set the define flag as constant, add the constant to the variables
> Ah, I think what you're implying here is that the overall timestamp of a collection is not the same as the timestamp of the last modified record?

Yes. If you pass X-If-Unmodified-Since on a POST, the server will check it against the collection timestamp, so the value you pass in has to be the timestamp you got when you fetched these records. If you pass in the timestamp from one of the individual records, it will always be earlier than the server's modified time, unless it was the *last* uploaded record.

That is, your code will work if you upload *all* client records, or include the most recently modified one in the set, but it won't work in the general case.

> performing an HTTP GET will actually update the collection's overall last modified timestamp but no changes are made to the individual records' timestamps? (wish server documentation was more specific...)

GET won't change any timestamps. What you're trying to do is detect when someone else might have modified one of these records since you fetched it. The mechanism to do that when you're processing multiple records is to track the time at which you *know* nobody modified it.

For an individual record you can use its modified time directly.
shouldn't there be add and remove methods for the hooks? at least in addition? IMHO this is something which could very likely change at runtime?
ah, commented. even worse ;)
same for thos changes
Addressed in Rev bb023c5.
I'd still have to  call `.trigger` on each node. This seems simpler than the indirect `.each(... .trigger('reload-drug', drugID) ...)` that would follow. I don't think the added code and mental complexity is worth it.
Are you talking about a property or about a method? We're using single-line property definitions only in case of implicitly defined properties - and no single-line method definitions. I also like single-line properties better, however.
There's no dependency; SendMessage is not a unit test (tests are marked with [TestMethod] attribute).
can you keep this comment?
Willst du hier nicht schon ein PARAMS entgegen nehmen... so dass z.b. mindestens ein Firmennamen vergeben werden muss?!?
it would be nice if one day we optionally do support returning them. for people that just maintain a small set of routes.
one thing that would definitely be great is if we could support ``app/console router:debug [route name]``, but this will require some refactoring in ``RouterDebugCommand``
I just found an interesting [article](http://www.codinghorror.com/blog/2009/04/death-to-the-space-infidels.html) by Jeff Atwood that is covering this topic. Still reading, though.
This doesn't seem right.

`Context#logical_path` should `==` `Asset#logical_path`
So: $(IMPDIR)/core/%.d : src/core/%.d ? Should work.
Actually ignore that, PBYTE and LPBYTE are aliases to the same thing.
Let's look at this from other side.
Should whether we have the functionality ```setWaitForTasksToCompleteOnShutdown(true)``` at all, if we already has **rescheduling**?
Quote from ExecutorConfigurationSupport#setWaitForTasksToCompleteOnShutdown JavaDoc:
> Switch this to "true" if you prefer fully completed tasks at the expense of a longer shutdown phase.
So, if my Messages should be delayed .e.g. one hour and there many of them (e.g. thousands), I won't be very happy when my Application Server will shutdowning for a long time.
I propose to remove ```setWaitForTasksToCompleteOnShutdown``` at all and and rely on rescheduling ability.
Also it should be documented: 
"If you don't want to lose you mesages after application shoutdown, use pepsistence message store on delayer. Delayed messages will be rescheduled after application startup"
I hope this doesn't mean we have Thread.slee() somewhere
@garyrussell, I have some doubts about this solution. If it is **hangAroundFor = 0**, test is failed. It says: 'cannot bind, address already in use' :-(
We need to document/WARN that using a shared JdbcTemplate should be used with care because it's fetchSize property will be set to 1.
From a testing perspective, it is simplest to find something that works then just stick with it unless we encounter a blocking problem. (Like the old quote says, “Better the Devil you know than the Devil you don't”)

This thought is echoed  by some of the comments on that thread ask "why do you want the latest?"
we could use a mibit link
@Maks3w @Xerkus I agree!
一見メソッドの終了タグに見えるので、インデントしてはどうでしょう。
Or rather to remove the dependency of the comments on the code, just say "respective ActiveRecord helper methods."
Here is ``gettingStartedStacks.png`` again.
Shouldn't this just be an ```assert```?
Whitespace could use a quick cleanup - lots of 2+ blank lines, trailing whitespace, etc.
Really this guy should be coming from a CDN ... from where: idk... :/
> But based on mongoengine source code it seems that both pk and id always point to same:
> 
> https://github.com/MongoEngine/mongoengine/blob/master/mongoengine/document.py#L91
> https://github.com/MongoEngine/mongoengine/blob/master/mongoengine/base.py#L770

No, look at lines 773-776. id is only created if meta['id_field'] is not defined. The user can set it to something other than 'id'.
Should at least be final. Why are the others public and this one package private?
It feels to me like a lot of copy-paste from NextPageField. Maybe time for a build_url(params, request=None)?
Accept

	;; Is the request uri the same as the redirect URI?
	;; Use string compare, since java.net.URL.equals resolves hostnames - very slow!
	(if (= (.toString (java.net.URL. (name (:scheme request)) (:server-name request) (:server-port request) (:uri request)))
	       (:redirect-uri oauth-params))
[Array.shift] ? Really? But why? :)

I would like to remove an element from a specified index, not the first!
As I read, shift (and pop) removes one element from the near (or the far) end of Array.

But [Array.splice] removes element from the needed position. See the ["After removing 1:" Example](https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/Array/splice#Example:_Using_splice) at MDN!
I would like to do exactly that: searching for an element, and remove it!

Of course I could use [Array.shift], but then a temporary Array and a loop is needed too.
succès (accès grave)
So close, but missing a triple equal here :)
The `ctor` parameter is not required.
I am lost. "When validating is done" - which validating? Not sure what you try to say
From an object I created as anonymous through Islandora:
<foxml:property NAME="info:fedora/fedora-system:def/model#ownerId" VALUE=""/>
nemidunam chera az hamun aval bedune !important kar nemikard.
az to menuye 
project -> open repo ro bezan va address jadid ro behesh bede
Good point. I could probably keep `rowCount` on this one and change the other one to something like `rowArray`.
oops! I had missed the break (double miss considering that when I saw the double-loop I should have figured it would break).  I should have eaten my snack sooner to help my brain out!  Yeah, that's totes safe, but I appreciate the comment.  Thanks!
you should add:

    if (!$client instanceof ClientInterface) {
        throw new \InvalidArgumentException('Client has to implement the ClientInterface');
    }
Doc desactualizada
@jkeshavarzi @nukulb I'm making a similar change for websecurity and want to know if I should be doing this too. Should we be allowing case insensitivity or booleans without a string ?
Just checked and it does behave this way.  We are covered for "Dismiss Selection" with action id "Cancel"
Do we want this method to have callback as second parameter to pass it to invocation.invoke(.., callback)?
add getter methods and make the members private
If I understand it right, the check for equals() does unnecessary extra work when the two are not equal.  It can be removed.
no i've just copied the code, since it can be used together with Apache License (see comment) and I've the experience that sometimes there are bugs, and it's easier to detect and fix them if you have the code available. The library alone with a bug is basically useless.
Both are parameters and their names are not listed in this comment