labofsahil/github-event-dataset-2012 huggingface dataset:
Sure I can iterate through it.   I'll change the 'i' for clarity; was leftover from my internal testing.
Action должен быть суффиксом, а не префиксом.
`count` is costly, you should check `(not (empty? ...))`.
Now I'm following it. I'd need to tweak the indenting for my eyes to see it more easily.
```bash
if (( $+commands[latex] )); then
  # stuff
fi
```
I see here a user-visible string ...
A try with addClass and removeClass but for me doesn't work ($('pass1').removeClass('input_ok input_error').addClass('input_ok')) or is this wrong?
Use keyword based interpolation, not position. (It is easier to translate because translator get a variable name and know what to expect.)
Something like this does not work?

```
from django.utils import translation
current_language = translation.get_language()
try:
    translation.activate('sl')
    date = datetime.date.strptime(slo, "%d. %B")
except:
    translation.activate(current_language)
```
Nothing yet :-) I don't know anything about jquery, but I was reading about accessing html elements and their values. And I don't know any other way how to  access a particular element in a for loop (for example free spots of the third station) than adding id value for each div element in a loop (but then I also read having a number as id is possible only for HTML5, but is not recommended). So I'm still without any ideas.
Looks good, except the lack of a blank line :-)
To save `jsTags.length` loop iterations, each with its own redundant `jQuery.nodeName`, `script.type` test, and `scripts.push` or `fragment.appendChild`—all of which have the same effect on the call stack.
Remove the commented out things.
i don't know how to solve that issue
currently there are lot of places where Position initialized as a POD type:

> Position pos = {some coords}
Was bewirkt das?
Yep, will merge.
Mismo comentario: ¿podrías indentar este bloque 2 espacios?
Tell us what books you're looking for and we'll comparison shop for you.
We still need this `return` to avoid attempting to collect the values produced in the loop.
With scala you can also do this: sys.error("Not implemented")
Having an update_bound would work here too.
Is there a reason ngx_http_pagespeed_determine_options can't just return the global_options, rather than having to know that NULL means use the global options?
Because we set session.* options in session_init() we can even skip config object usage and use ini_get() only here.
Reading up some more about FFT methods, I found http://books.google.nl/books?hl=en&lr=&id=AyJ9xrrwDnIC&oi=fnd&pg=PA203&dq=density+estimation+multipole&ots=K_IBmhGVTY&sig=yrzJOhpPe4d2-2i_xCbeFpdE6hE#v=onepage&q=density%20estimation%20multipole&f=false which gives the following scalings:

    O(N^D log(n^D))    # FFT
    O(D N^2)     # direct summation (like in this PR)

It also gives a reference to an empirical study (Wand, "Fast Computation of Multivariate Kernel Estimators", 1994) showing that for D=3 the speed-up of FFT over direct summation is at most 5 for 10k samples. This scaling is the reason that FFT methods are mostly restricted to 1-D.
Support should not have a capital letter.
Your new code ignores the parameter. Is it intentional? In that case explain (and remove the parameter).
Comma, again.
OK, I've pushed something that does this.  In order for me to make sense though,  I had also to incorporate a bit of what you wanted regarding the size params... Although I found the curly bracket idea a bit visually confusing, I've tried something else that achieves something similar via slightly different means.  Thoughts welcomed.
(P.S. I don't think this would need to be done on this PR, I think it would deserve an issue for discussion and it's own PR)
These magic numbers are a bit hidden in this part of the code.  It's not a massive improvement, but I think these would be better either as named constants defined at the top of the module scope or, probably my preference, as a kwarg with default value of 25, e.g. `max_mu=25.`. And with a short note in the docstring.
Good spot!

--
Barnaby Rowe
Postdoctoral Research Associate

Department of Physics & Astronomy
University College London
Gower Street
London WC1E 6BT
United Kingdom

On 5 Sep 2012, at 13:43, rmandelb wrote:

> In INSTALL:
> 
> >  
> > -ii) Mac OSX  (#TODO: Expand this, maybe move it to a Wiki FAQ)
> > +The solution may be to install Boost C++ manually, which luckily is relatively straightforward. This
> Sorry, I AM a pain!
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
This isn't quite where you meant to put this comment.  But I think I found the error you're talking about on line 1318.
we've been alphabetizing these classes recently, can you reorder this?
Hm, though I now realize that site.name isn't actually being used anyplace other than this script. With an eye towards the open source community, should we be doing that? I guess I'm imagining definitely pulling it in outside the templates, and possibly in templates too, inasmuch as any of the templates are meant to serve as examples to others.
Pas trop envie de faire un tuto sur Upstart, mais je vais preciser les qq lignes les plus ikmportantes par contre !
Yeah, I guess you're right. I would like to fix it eventually though. I prefer that the code matches the native functions it call.
Interesting. Everything is super unaccessible right now - that will require some bigger work.
Static now uses the AssetsPlugin instead of kicking this stuff off on its own.
Use `&gt;`
To send the `submit_label` via `:locals` is an ugly solution.

Since you have the `@group` instance you can use [new_record?](http://api.rubyonrails.org/classes/ActiveRecord/Persistence.html#method-i-new_record-3F) to chose the label. Something like

```ruby
 = f.button :submit, submit_label
```
I just pushed a re-write of the cukes--let me know what you think about it now.
Ahh, good point. I read too quickly.
It's used in test methods below.
If you are going to take it that direction, then why not an actual class variable (@@decimal_places_cache)?  I was trying to understand why you guys got rid of the constant in that patch and I figured there was no ability to override behavior/contract of [] easily without making the constant a specialized type.  Although there was a mention Constants should not change state in one of the comments.  The constant argument did not quite make sense to me since every class is actually a constant (by Ruby constant definition rules), so I am a little confused.

I will change it to whatever the consensus is though?  Still want an instance variable on class or is a class variable ok?
Per IRC i agree that apps/web site expect a white background by default.
same deal, should this be in this pull request?
This should probably get updated in the docs, too. I think it's docs/settings/settings_local.prod.py.

I'll add it to my laundry list.
Yes, I was talking about putting it into namespace Common.
They have a github, too. Perhaps worth a `head` ?
As long as you don't have a Facebook style apache/mod_php bug. #amiright?
the `true` at the end means the object is persisted, no?  That doesn't seem to be the case here
Зачем здесь Vector?
Всё равно некорректно: исключение в конструкторе BuffererReader или Scanner - и FileReader не будет закрыт никогда.
И надо использовать try .. finally
Во-первых, в одну строчку записывать нельзя (по code conventions).
Во-вторых, надо дать понятные имена полям.
@christianv yes, but also calcentral probably wouldn't be instantiated as a angular module, and thus wouldn't know how to handle calcentral.controller(...).
Как то странно, почему переменная устанавливается из константы?
Либо параметром, либо просто юзать константу внутри класса
2 blank lines required here
This is excellent. Feel free to make other sorts of reports.
Is this link resolvable internally?
done, yeah, git rocks !
> @ogrisel can you explain?

Why me? I have no idea without diving further into the numpy code. Maybe @GaelVaroquaux knows?
Could you please add a comment on why the copy is necessary?
> I will contact the original owner, as though the dataset is _primarily_ synthetic, it is not 100% synthetic.  There is a portion (particularly, the transition offsets) that was created by hand or by some real robot, and it is this part that makes the data look "interesting" rather than just a straight or curved line.  I could potentially create a synthetic generator function, but the output would look more artificial than the dataset provided.
>
> In addition, the dataset provided contains reference output I compare with the `KalmanFilter`'s output to check validity.

ok fair enough. The file is however 328K large while other datasets
are rather small (< 60K). Can you see how to reduce the size? using
maybe npz or a smaller type (float32 or float16) or joblib
compression?
`KernelPCA` exposes its `KernelCenterer` too... I wouldn't mind making it private.
я спочатку робив за прикладом документації, та зараз побачив, що це було зайве, заберу
"return unless @_shouldUpdateRasterizedPath" would be nicer
But `ix` isn't a manifest constant.
This code looks very strange selecting `1`/`0` based on a comparison that would give those values, then comparing them using `<`/`>`.  I know the existing code does something similar, but it it a lot less weird when using the `MATCHxxx` constants.
cette route n'est pas appellée
repeated style float:left;
This works equally well on non constants.  I could call the function expressionsEquivalent() instead, but it is only currently used on constants.
Why? Did the previous way break your check.py?
All this strptime hacking can disappear once we start using a form instance instead of manually picking through request.GET
Hi, after some search in the code, these options indeed happen to be true or false depending on being set with ```--option``` or ```--nooption``` (https://github.com/openstack/nova/blob/master/nova/openstack/common/cfg.py#L528-536).

However, these options cannot be passed a value on the command line or nova.conf with ```--option=value``` as the nova-* commands  complain :

    # egrep "verbose" /etc/nova/nova.conf
    --verbose=1
    # nova-compute
    Usage: nova-compute [options]
    
    nova-compute: error: --verbose option does not take a value

I am quite new to puppet, but do you think it would be feasible to modify the ```nova_config``` provider to raise a duplicate resource definition error when using both forms of option definition ?

Another option would be to use the ```--config-file=PATH``` switch in ```nova.conf``` and put the other values in a .ini config file instead but it would break all that has been done so far with ```nova_config``` …

What's your opinion about all this ?
Yeah, I'm not a huge fan of push models either. A lot of the code included in this app is simply porting (and writing tests for) the existing system we use at the Synchrotron. Hopefully this app will form a solid base that can be improved incrementally (read: refactored mercilessly) as we have more time.
It is impossible to have an int-numbers as a string within a php array key ;)
This one's probably your call, so I'll just present the case. Because of the lack of OAuth support to get a proper current user classes back at the time being, we'll need to do 2 calls to the Canvas APIs, which both have to return in order to continue. I could, use the nested callbacks pattern but I feel like it won't be as clean. I'll open up another branch for you to see how it looks and let you decide.

The other reason I threaded Deferreds throughout the code was for Init call. It seems to present a clearer overall picture of the callbacks that get threaded in sequence for the page to load in that one line, instead of tracing the call from init throughout the code.
The business reason behind this conditional isn't very clear. Can you extract a method that would make the intent more clear?
> For example if a function could return the concrete type List I would choose IList or IEnumerable. 
Sometimes better return List because of performance, using interfaces slower in some situations.
Only specific url's about the technique please, it looks good: http://www.pentestgeek.com/2012/11/05/finding-logged-in-users-metasploit-module/
Ignore different filename - I just copied and pasted the code in. I hit this exception today, although previously it worked past this fine. Worked fine after running enum_domains so maybe that initializes something better?

meterpreter > run post/windows/gather/credentials/enum_gp_local_passwords 

[*] Enumerating Domains...
[*] Finding the right buffersize...
[*] 0 domain(s) found.
[-] Post failed: Rex::Post::Meterpreter::RequestError stdapi_railgun_memread: Operation failed: The parameter is incorrect.
[-] Call stack:
[-]   /opt/metasploit/msf3/lib/rex/post/meterpreter/extensions/stdapi/railgun/railgun.rb:144:in `memread'
[-]   /opt/metasploit/msf3/modules/post/windows/gather/credentials/enum_gp_local_passwords.rb:174:in `enum_domains'
[-]   /opt/metasploit/msf3/modules/post/windows/gather/credentials/enum_gp_local_passwords.rb:55:in `run'
meterpreter > getuid
Server username: MYDOMAIN\user 
meterpreter > run post/windows/gather/credentials/enum_gp_local_passwords 

[*] Enumerating Domains...
[*] Finding the right buffersize...
[*] 0 domain(s) found.
[-] Post failed: Rex::Post::Meterpreter::RequestError stdapi_railgun_memread: Operation failed: The parameter is incorrect.
[-] Call stack:
[-]   /opt/metasploit/msf3/lib/rex/post/meterpreter/extensions/stdapi/railgun/railgun.rb:144:in `memread'
[-]   /opt/metasploit/msf3/modules/post/windows/gather/credentials/enum_gp_local_passwords.rb:174:in `enum_domains'
[-]   /opt/metasploit/msf3/modules/post/windows/gather/credentials/enum_gp_local_passwords.rb:55:in `run'
meterpreter > run post/windows/gather/enum_domains 

[*] Finding the right buffersize...
[*] Tested 500, got 21 of 108
[*] Tested 1000, got 42 of 108
[*] Tested 1500, got 62 of 108
[*] Tested 2000, got 83 of 108
[*] Tested 2500, got 106 of 108
[*] 108 domain(s) found.
[*] Enumerating DCs for blah
^C[-] Post interrupted by the console user
meterpreter > run post/windows/gather/credentials/enum_gp_local_passwords 

[*] Enumerating Domains...
[*] Finding the right buffersize...
[*] Tested 500, got 21 of 108
[*] Tested 1000, got 42 of 108
[*] Tested 1500, got 62 of 108
So do you want it to be a separate parameter?
True, but I'm actually back-tracking about whether sigma actually needs to be adjustable (sorry to be so fickle). I don't think it makes a big difference either way (whether it's removed or kept adjustable using `param`).

On the other hand, my comment that the adjustment of  `sigma` is broken applies to the current implementation:


    from skimage.data import camera
    from skimage.filter import threshold_adaptive
    binary_image1 = threshold_adaptive(camera(), 10, param=10)

    ---------------------------------------------------------------------------
    UnboundLocalError                         Traceback (most recent call last)
    /Users/Tony/<ipython-input-21-dce9059be443> in <module>()
          1 from skimage.data import camera
          2 from skimage.filter import threshold_adaptive
    ----> 3 binary_image1 = threshold_adaptive(camera(), 10, param=10)
          4 

    /Users/Tony/python/devel/skimage/skimage/filter/thresholding.pyc in threshold_adaptive(image, block_size, method, offset, mode, param)
         72             # automatically determine sigma which covers > 99% of distribution
         73             sigma = (block_size - 1) / 6.0
    ---> 74         scipy.ndimage.gaussian_filter(image, sigma, output=thresh_image,
         75             mode=mode)
         76     elif method == 'mean':

    UnboundLocalError: local variable 'sigma' referenced before assignment
"can be used to apply forward and inverse coordinate transformations between..."
Moved it up to line 26
As far as I understand, this function should just read a single char, but reading only one it's not working, reading 1024 is just a quick hack to make emscripten stdio work with node on commandline. I will investigate if there's a better solution for this, but at least reading 1024 works as expected in most interactive prompt use cases (input is bigger it just fails). Do you know a better way to do this?

Thanks
The code for our examples should be formatted the same way the rest of our code is.  It should also actually do something (or at least pretend to do something).  For example, change the url to point to a czml file and add comment to the function blocks like //process CZML and //Clean up now that we are done.
Not sure why we are drawing a satellite on the globe.  I can see some users poking fun at that.
Exactly.
```
Increasing a texture object's maximum degree of anisotropy may improve texture filtering but may also significantly reduce the implementation's texture filtering rate.
```
from [EXT_texture_filter_anisotropic](http://www.opengl.org/registry/specs/EXT/texture_filter_anisotropic.txt).

I wouldn't be surprised if this is pretty significant on some cards.
another good catch. Often unsure about `require` vs. `include` :)
It could be worthwhile to make sure that the version of node meets our requirements (v0.6.0 or whatever).
This should based the MSG id off the parent type.  The example I provided shows:
public class TabGroupProxy extends TiWindowProxy

where TiWindowProxy is the type that is extended.  TabProxy extends TiViewProxy
The line removals here make the function name (apllyChildProxies) seem odd now.
no insertAfter
Better still define them as constants on Ti.UI.View so that if later we want to implement a vertical wrap we can do it clearly.
@BlainHamon  I see, well I'm going to make changes and write functional tests.
Note that DIP is different for iPad and iPad Retina in terms of DPI due to the device screen density (132 and 264 respectively)
В Python можно писать что-то в духе "def func( **kwargs):", где kwargs словарь с парами "переменная: значение". Тут видимо как-то аналогично, нет?
Видимо качестве "словаря" используем объект с аналогичными парами. В самой функции проверяем наличие полей, если поле присутствует, то оставляем значение, если нет то значение по умолчанию. 

function Event( eventvars ) {} ? // evemtvars = { Call = "Событие", Description="Новое событие", Private=false}
Should we throw an exception here? Otherwise, the user has no idea that their increments failed.
Это что-то питоновское, по ходу (класть несколько классов в один файл). Ну то есть, я не особо против, но помойму не-питонисты так не делают.
@9rnsr @andralex probably you could also give some input regarding the point @shoo has raised?
Use single type of quotes, I guess ', since there's no string interpolation.
Acho que o internet, ou web, nem tem tanta diferença. O problema mesmo é o "sobre a web" em vez de "na web". A informação desatualizada é sobre PHP e está na internet, entendeu?
All the versions should use a property in the version.groupId.artifactId format or something similar to the version properties that are already in the POM.
Can we make this more specific?  Say `sync.notification.configure.engines.saved' and perhaps say "Firefox Sync will now sync your XXX, YYY, and ZZZ".  Harder, I know, and maybe not worth it.
Null check for cursor.
no parent adaption is needed in the context persistence?
is it really discover and not rather load? basically all you need is to get all classes in  the bundle
This is exactly the same way that it worked in the old video grabber - but sure a comment never hurts
So the invariant is that it is up to the event handler to call `ConversationKeybindings.saveKeyBindings` whenever it performed some action on the global variable `bindingGroups`?
Your entry in the `"package_name_map"` is not in the correct order. Please move it down to the other `sublime-*` packages.
You mean as opposed to while they're using Firefox to do the dishes?
typo on enevlope
Why $.each() ? Is underscore not loaded?
I don't think the backslash is needed, could you test without it please?
We are replacing constant string with two additional method calls: Format and TypeAttr get method. I think in that case it would be much better to remove TypeAttrInObject property at all and replace it with direct call of JsConfig.TypeAttr.
As far as I can tell, you have already validated all input parameters higher in the call stack, which makes validation here redundant.
toUtf8() would seem to be more correct here, filesystems aren't generally in latin1
Ссылка на self не нужна
the 'works' isn't needed here
The latter one looks more thorough, but it's LGPL and we can't have LGPL code in cucumber-jvm.

I'm asking the author (whom I happen to know) to waive the LGPL for us. Will update here.
hmm, that's very true.  Yeah, I agree with your thoughts here.  I split these tests out.  I actually noticed a bug that I wasn't checking that the suites added were actually added to the test case.  Since I was specifically testing just one thing.  I think that practice has lots of merit!  :)  I'll fix the over-asserting all over the place, too.
this can be handled via the `defaults` keyword argument to `get_or_create`, and avoid the extra save here
I don't think the 80 character restriction really makes since in this example, breaking up this line will only disrupt the flow which will make it harder to read.
Hmmm...

I hate to suggest it at this late stage, but don't we really need a mapKeyExpression? Seems odd we can use an expression for the store name, but not for the key of an entity we put in that store.
Yeah. Mark and I talked about that. Its OK. Left it like this on purpose
Could you please un-cuddle these to match the code style in the rest of the dist?
The name of the logging resource needs to match the name of the resource setup in the default.rb recipe (i.e. "default"). The reason is that we are "reopening" the logging resource named "default" which already has the provider attribute set.  If you supply a different name (i.e. "do_server_start", you are creating a new resource object with no provider attribute set.  This applies to all the "do_* recipes.
Thanks, I missed that bit when skimming the changeset.
Why is the result a session or `false` and not a session or `nil`?  Methods that return objects or `false` are not particularly intuitive.
а почему именно со скобками?
$a->engaged = !$result; ?
The only one that really needs to be public is `EliminateHole`.
here again we have to think about concurrency. it might be better to handle this with an UPDATE statement with CASE statements to determine what needs to be changed in all the relevant rows.
This should be "@return mixed" to follow the docbloc standard
you mean all the converters? makes sense now with the changes i did in the compiler pass i guess
Aww, this code is LGPL. Ag is Apache 2.0.
I dont think so. Anyway all tests are pass. They use only two values for this property: "some huge text string" and "some even bigger text string"
Link needs to be full type name (Ti abbreviation doesn't work). So: [Network.httpURLFormatter](Titanium.Network.httpURLFormatter) will take you directly to the property.
"Storage operations should be directed to that node."
Sure. I'll change that in a bit.

As a note, I based this on the code in [refinery-pages](https://github.com/resolve/refinerycms/blob/master/pages/db/seeds.rb#L35) . Would you like an update for that too?
Maybe add a link to some Netcat resources.
Looks like the iOS Cordova devs are amenable to having the means to load a remote start page too.  I think we'll be able to normalize this approach across platforms.
Ох. Да, надо убрать. Сейчас попробую его чем-нибудь таким заменить, чтобы и в python 2 и в python3 работало... мне просто было лень запускать его указывая интерпретатором python2, в арче python3 по-дефолту.
This should probably read "Invalid feature - failed to find the id attribute"
Could we put a comment in about what we're waiting for?
Good idea. I'll have deploy-tests call deploy-automation as the last step.
I renamed this boolean and labelled it incorrectly.  I'll rename it.  Basically, autoDefer is by default set to true.  false is the one that allows things to run in the background unhindered.
"À" l'exception, plutôt que "Avec" ?