labofsahil/github-event-dataset-2012 huggingface dataset:
Should the args/kwargs order reflect the order in the docstring? If so, I'll be happy to change the code in this pull request, just let me know.
дивись в код. в сутності User дефолтна роль є по дефолту вже. її не треба окремо задавати
Eventually I need to fix this so it doesn't build the entire result dictionary in memory too... but that can come later.
Nope - this is for handling URLs inside of content returned by mobileview. We can't really add language information there per-link...
`item.element` is an <option> element, therefore this returns an empty jQuery object, causing the attr call below to do nothing. Causing the unit test for aria-selected to fail.
please use doubles and move `then`s to new line:

    if [[ "${DESKTOP_SESSION}" == "gnome" ]] && builtin command -v gnome-open >/dev/null
    then

...

    elif builtin command -v open >/dev/null
    then
We shouldn't pretend to support QuickC or Watcom C.
On Sat, Sep 29, 2012 at 02:09:53PM -0700, Hara Kenji wrote:
> More exhaustive test I think.

Yes, indeed. I kept the specific getMember tests too because that implementation
might some day change, and it is important to me that this specific code works.
(I'll probably always use it in a loop with getMember in real code.)
If I'm reading this correctly, we're doing links one by one, rather than in batches? This took performance for me from ~30 seconds to 4 minutes.
Can you check for the two items in `$spec['spec']['attributes']` too?
I will look into the formatting, but I think it might be related to how the web gui is displaying the text. The formatting looks correct in the raw file. https://raw.github.com/pschwartz/HPCC-Platform/c874c1794061cb1dc130b13c06292ee050eae22d/cmake_modules/docMacros.cmake
I think I would only use dbgassertex in a piece of code that is likely to be time critical.  Especially if the test condition is simple.
Make it less long.
This is a global I believe that is setup at load time. This means all the tests after this one are also going to use `ogg`.
I just want to say I have no preferences. I let you manage how author should appear.
Perhaps this should just be called `EmailColumn` to keep it short and consistent with the forms & models counterparts. What do you think?
Eclipse auto-formatting.
Good to know. If we ever do a Coding Guidelines style guide this would be a good thing to include.
Hi modpr0be,

It's doing a jmp back to the start of the shellcode. The shellcode is being stored before the overwritten EIP because there is more space available than after EIP (and until stack bottom). Shellcode isn't being corrupted so it's ok when you reach the RET with overwrote EIP.  Has it sense for you?

The "Metasm" code allows to generate the binary shellcode in a easy way:

```ruby
>> Metasm::Shellcode.assemble(Metasm::Ia32.new, "jmp $-#{2048+4+84}").encode_string
=> "\xE9\xA3\xF7\xFF\xFF"
```
It would be great if this were actually a Rex::Mime::Message, it is for this specifically.
This code was fixed in a previous submission of another module as I use this same method in that module.  I will fix it here and my other submissions as well.
This `if` statement can go away, since `first_instance` checks it for us.
This is somewhat debatable as it will ignore people having a va_file_naming_format but not having it enabled.
"Get's" is a typo here (looks like it was already there).  Shouldn't have an apostrophe.
Why sort?  To minimize fragmentation?
FYI, it's shorter and faster to do this:

    if (configSetArg !== void 0) {
Mobile Web doesn't support global point. Was this supposed to be set for the event as a whole?
Fail early takes priority.
I've started making debug printing conditional on profiler.debug.
Why "var file" but "Document doc"?
"We should forget about small efficiencies, say about 97% of the time: premature optimization is the root of all evil"
-  Knuth, Donald (December 1974). "Structured Programming with go to Statements". ACM Journal Computing Surveys 6 (4): 268.
Sry, I thought I had pushed that.  Should be there now.
If you fall back to getAttribute on both cases, wouldn't it make more sense to just use that in either case?
why not self.setDisconnected()?
в зависимости от какого-нибудь аргумента или типа того
Сделаем `@`, можно будет влезть в стейт. Не знаю. В джаваскрипте всем, в общем-то, плевать на инкапсуляцию. Если тебе кажется, что так красивее, могу переделать.
Oh. Is that a convention we/others use elsewhere? It just seems odd... but it's kinda clever.
To much copy & paste in the readme?
Please use `assertThrown`.
You got rid of the template constraint. It needs to be put back.
Statically-sized array here? That way you avoid allocation.
Since this is private (= of limited, scoped use) there's little gain in abbreviating its name.
Oh nice, I didn't catch that one. Does anyone know if there's a way to make an enum by itsself return a string? So that I don't have to use .key() every time?
Ignore - for [/git-convenience-no-rebase](https://github.com/benseven/git-convenience-no-rebase)
あと、`@remove_tag_prefix`ってどこから出てきたんですか? mixinに定義されてるのかな……。なんかインスタンス変数がいきなり出てくるのこわいから、そうなのであれば元のmixinにメソッドかなんか定義してほしいところですね……。
XMLin makes these transformations:

in: ```<pages><page title="Transclusion Testing" ... /></pages>```
out: ```{ 'title' => 'Transclusion Testing', ... }```

in: ```<pages><page title="Transclusion Testing" ... /><page title="Style" ... /></pages>```
out: ```{ 'Style' => { ... }, 'Transclusion Testing' => { ... } }```

I wasn't able to figure out how to make XMLin behave consistently for these inputs.  So this code normalizes the first response to ```{ 'Transclusion Testing' => { ... } }```.
This is purely stylistic, but I am not a fan of this `deleted` flag everywhere.  AFAICT, the flag is always static.  I would really rather have two functions with descriptive names and assumptions in the docstrings than have an if/then block, but that is my personal preference.  rnewman?  marina?
maybe we can extend that all exceptions thrown by this hook are caught, logged and ignored?
Since this isn't a channel, and we're not sure when/if it will become one, yank this out.
hey i didn't generate the file!
This wasn't supposed to be here
Please use Rails' image_tag helper to generate the required HTML image tag.
i use it here https://github.com/liip/LiipHelloBundle/blob/master/Resources/config/routing.yml#L40
missing header comment for this method
Please avoid using global variables. See https://github.com/dellcloudedge/barclamp-crowbar/pull/301 for related changes.
Here's a theory: ```modal: true``` is preventing ```$a_remove_tag.click()``` from doing anything while the dialog is open, but calling ```$j( this ).dialog( "close" )``` exits the handler and doesn't execute ```$a_remove_tag.click()```.

The description of modality in the [Dialog API](http://api.jqueryui.com/dialog/#option-modal) seems to support my first hypothesis.

Can you try replacing ```$j( this ).dialog( "close" )``` with ```$j( this ).dialog( "option", "modal", false )``` and see what happens? Alternatively, you can replace ```modal: true``` with ```modal: false``` in the initialization, though I prefer the former approach, since modality is a good property for the dialog to have.

If the code is running on your XVM server, I can try debugging this too.
Derp, right. Sorry, old habits die hard.
This one as well :)
потому что этот хук стоит в mouseReleseEvent.
компилятор будет еще дольше компилять личкрафты. ок. ВАЩЕ НИЧЕГО
For Spree 1.0, this should be spree/admin/configurations/index.
Heh, I had actually been digging in that exact file a couple of hours earlier to find out how they register the data segment and retrieve the stack base of threads. It was...an interesting journey.
MINOR: Implamentation?
Could you put `data/DoctrineMongoODMModule/Proxy` and `data/DoctrineMongoODMModule/Hydrator`? Also: does the MongoDB ODM have a proxy/hydrator namespace? that one should also be handled.
Another installation step in the docs is also needed, I will also add the CLI commands to the tests later :)
See here : https://raw.github.com/bakura10/zf2-documentation/d291b93ad3d2b0ce54ae821661c52532cd7492f5/docs/languages/en/modules/zend.form.element.select.rst

This is the complete file from the commit. And it's still here. Strange isn't it ?
I agree with @Ore4444 here.
I think the above should stay, as many will search for it in this context (as they're thinking about configuring modules).
You need spaces after commas in a bunch of places.
```coffee
line = "#{line} [as #{methodName}]" if methodName and methodName isnt functionName
```

Please use interpolation when possible to clean this up. The `+` operator should never be used for string concatenation in CoffeeScript.
if you line-break right after the opening brace, the next line is idented one level under the previous (not aligned with the braces)
@apex13 Missing a `,`
You're right ... I uploaded a previous version and I didn't commited the bind ... :\ - fixed
Right, I'm doing it here to be visible.
Nuthin' sounds quite like an eight 

oh

eight
doh. missed a line when I renamed a var.  fixed in 183ef18efd032611149cfdc558755aa32135616c.
xrange is slightly faster
I think it's worth adding a comment on this getter or a file level comment or someplace that describes the strategy.  The code is clear, but it's not immediately obvious.  Something like this maybe:

The account filter type setting affects all folders.  When it is set to "auto", we will compute an appropriate filter type given the number of messages in the folder and save that to the folder for future use.  Filter type values computed by "auto" will be ignored if the account setting is changed to not be "auto".  The filter type is sent with every request (although it can be optimized away), so the server will notice if we change the filter and so there is no need for us to locally invalidate the SyncKey.
Remove this line it's a yaml file.
в развитии
ENSURE_UI_THREAD MACROS in play and stop need to be removed
Note to self, I think I should be specifying --onetime here
I think is the common practice here to put the most recent changes on top
> Do we need an entire controller for one action?

I totally agree, I also think about that, that's bad for having only create action in the controller.

```ruby
# This is probably overkill, since we only need to update Rating#rate
rating.update_attributes(params[:rating])
```

I don't think it overkills because when I see sql statement in server log, Rails is smart enough to update only rate column.
code will run despite test failure.

I also have no other way testing this.
Why is this in the commit? It seems unrelated.
И в контейнер мы кладем как experium.storage.my_file_provider, то есть с тем именем, которое указали в конфиге.
spam is as bad as getting it in your email!
For the root page of the site, I'd prefer more than just some plain text saying 'Hi'. Probably remove the Login view and make the Login view the home page / home resource. 

My reasoning for this is that this is a demo site for -only- authentication. so instead of making it 2 clicks deep to get to the login page .. it's visiable straight away!