labofsahil/github-event-dataset-2012 huggingface dataset:
I don't think you actually need a contentManager here. You can just take the properties param you get from the migrate method and do  properties.setProperty(FilesConstants.POOLED_NEEDS_PROCESSING, Boolean.TRUE) . The PropertyMigrator framework will persist any changes you make to properties.
@sonney2k There's a small problem switching from SGVector to DynArray. It's mainly due to MulticlassMachine::register_parameters, who need to register the m_machines variable. However, DynArray is a more encapsulated class than SGVector: it's memory buffer and size variable is not directly acceptable from outside. I don't think making them public or making MulticlassMachine a friend of DynArray good ideas. Do you have any suggestion here?
How about 2.3?
Well, it's also doing query.toString(). Which might not be expensive enough to matter.
I'll go with translation_title. I changed it from 'title' as the XML Parser function getElementsByTagName() which is used in the XML import library would end up picking ALL elements in with the tag 'TITLE', irrespective of what element i'm looking to pick out. It's basically for purposes of keeping all element tags unique to avoid confusion.
JSON itself uses double quotes, but you have converted it into a PHP array, so you should use single quotes.

Also, why are you converting it into an array? You're not looping through it, so it is cleaner and faster to keep it as an object.
Are you sure that the method "setValueOptions()" exists?
Are you sure that ``$this->event`` (of type ``object<Symfony\Component...cher\GenericEvent>|null``) is traversable?
Im really not a fan of this. Sorry :S
In one case it is doing a load and setting the _from_db to false and in the other it is doing a new.
Absolutely right. Just adding it works.
You're right the change is not 1:1 with the previous logic. I'll modify soon.

Regards to your suggest change, again you're right. It's not clear why we can't default to `getComputedStyle`. If style getter doesn't work. 

My other thought is, why can't we always use `getComputedStyle`? Was the original idea used to save CPU cycles?
don't need the `contents` argument, it can get the `contents` variable from the outer scope.
you ovveride $default's initialize. With this modification, form won't validate
is this on purpose?
*userInfo* -> *info*?
You have to insert a number at the end of URL to get info about specific station. It works for me?
In fact this is not correct. The proper way is still to run things from project directory, but to run it as:

```
../manage.py makemessages -l <language code>
```

See two dots.
`tree.findtext('.//item/title')` return title text, which is for example `Aries Horoscope for Wednesday, August 01, 2012`.

You want to get out `August`, `01`, and `2012`. So this returns you that:

```
import re
date_pattern = re.compile(r'(\w+)\s+(\d+),\s+(\d+)$')
date_pattern.search('Aries Horoscope for Wednesday, August 01, 2012').groups()
```
If needed, modify `content` class.
Ehm. Why you have one more embedded field in-between? So why not just MapField which maps names to Panel embedded documents? If Panel is in the map, panel is activated, if it is not, it is not activated?
ok solved with padding: 0
Should be `"`. (Yes, also in horoscope is wrong.)
Forgot alignment much?
Empty line before return statement, or simply run `php-cs-fixer` ;)
Codestyle: & placement (you want to make this function const too)
i just went through and updated all these
Does checkstyle complain about the indentations here?
No, this is a view class tightly coupled to Dragonfly, so here it's OK. I've changed.
don't put a space between `ngx_http_request_t` and `*`
Yes, but it will autoload non-function files like REAMEs since the glob will fail.

I am not willing to support Zsh 4.2.x. It's not used by my target audience.
This should probably be an argument to the setup function.

    zstyle ':omz:module:prompt' theme 'solarized' 'dark'
    zstyle ':omz:module:prompt' theme 'solarized' 'light'

Dark should be default.
This feature flag is more an expression of desire -- do we need to set it to false to prevent the ad from showing up?
I think we need a comment about how this method has become final.
Just to be clear, I'm not going to do any editing today, so you won't clash with me.
I think it is no big deal if the warning gets redrawn over, but at least it can be tracked down with :messages.
Hmm - not on firefox for mac. (But I've made that change anyway, since I guess it helps in linux) - thanks.
Do we really need this file, or can it be deleted? If there's anything important and different here compared to the normal test log4j.properties then can/should we change the normal test log4j.properties instead?
Does this return 0 if apach2 was not running? I presume so, but thought I better ask.
Right. That might work better, actually.
http://sphinx.pocoo.org/domains.html#cross-referencing-syntax
This needs to be `from pages.edit_profile import ProfileTab`
yes we can.
We don't have an event yet. This is one of the many follow-ups I need to do with Mounir.

To be honest, I don't think we should actually update the *whole* screen here anyway. None of the SMS app code is going to survive a big SMS database because it always fetches *all* records and redisplays everything all the time. Also, if we miss the boat with the 100ms, nothing will be broken since the display already updates with the pendingMsg.
I have heard that streching the background-size force the rendering to be done in software instead of GPU at the moment. Is it correct? @cgjones ?
I think we should leave at least one example of specifying the history required here, e.g. `%history 1-5`.
I pulled this in locally - there's an odd character at the end of this line...
That was a typo. I wrote "rude and thankful". I mean "rude and UNthankful".
What is the "// Testing code" comment about just above this line in the file?
What do you think of these going into a common
pios_mpu60xx.h
This seems suboptimal to end up here.
Really? The docs say:  "On failure, it returns 0 if $wp_error is set to false, or a WP_Error object if $wp_error is set to true."
И перед каждым StackTraceElement тоже...
Зачем это? Сортируются строки, а не слова.
А кто будет закрывать?
оу, не заметил в документации, что Close выбрасывает IOException, оказалось, что надо было смотреть в документации на Closeable
There's no client-server mode implemented for tomcat/ews
I see it right [here](https://github.com/josegonzalez/homebrew-php/blob/master/Formula/abstract-php-extension.rb#L179)?
OK i will remove it
does `w/` mean `with`? don't save two chars for nothing :)
No they are not different, its just two ways of doing exactly the same, in fact its merely an optimization and could even have been an implementation detail in the client.
So to recap, `delete($meta)` and `index($doc, $meta, $options)` with type sniffing for `$meta` so its possible to do `delete(1)` if the index/type is set on the bulk.
実行環境という言葉に違和感が。cleanして消えるのはコンパイルしたコードなので、それは実行環境？？なんだろう。
xbmc does this: AC_DEFINE([USE_EXTERNAL_FFMPEG], [1], [Whether to use external FFmpeg libraries.])
its set when any of the 2 external flags is used.

so we'll need to check for that here: https://github.com/xbmc/xbmc/blob/master/configure.in#L2744 and pass the correct flag
`s0` is only used once, so this can become `s += np.median(s)  # comment`
Thanks for the input @dwf, I was just about to ping you :-)

I do understand contrastive divergence and the idea behind PCD, and I also understand why @amueller doesn't like it :imp:  what was unclear to me was the dynamics of the `n_particles` parameter. Also I was referring to its description in the docstring, but since I'll be writing the narratives, it is probably important for me to understand everything too.

@ynd's code doesn't have any stopping condition, but an `n_iter` parameter (that must be optimized via grid search). I will check how you do it in your gist. 

Since I got your attention: is there any value in increasing the k in CD-k after some time? IIUC it helps with the density modelling part (makes the learning closer to actual ML) but that's not really important here is it? The implementation here should IMHO be as close to black-box as possible, and if users are on to something good they should use or extend the one in `pylearn`.
для чого перевизначив BaseRegistrationFormType?
how about just:  size_t bitsToUse = (size_t.sizeof * 8) + 3;
Can you name `capture` something else (like `return_value`) so that it doesn't collide with the `#capture` view helper?
This still isn't the right order :(
Have you created a JIRA task to track that NYI stuff?  I can't see it from the commit description.
Espace en trop devant entier et de vant Le nombre d'entrée à retrouver ?
```unless @local_authorities.nil? || @local_authorities.empty?```
![](http://weknowmemes.com/wp-content/uploads/2011/05/trollface-problem.jpg)
You know what, fix it FASTER!
It accepts relative paths and get the base dir in that case.
This is the only use of outputPath/pluginsFilePath.  If you fix the original translation of outputPath you wouldn't need to translate again.
Actually, why not use IMultiException->errorMessage(..) ?
This is not good enough. I forgot to elaborate on this. 

r- on myself.
This view doesn't seem to require auth at all.
It may be usefull to add a link to the main Widget tutorial, as we explain there the whole content script/event story?
This looks harmless, but it does seem like the line above that calls "unless is_a?(Array)" is trying to do the same thing.  Would be interesting to know exactly what the data type of "vals" is in the case that you're debugging, because it seems like only one of the two lines should be necessary.

It's true that this exact same pattern is used in :realize, though.
This variable name and comment may be misleading and inaccurate.  Perhaps margin-before and margin-after would be more appropriate.
etc/test.txt から引っ張ったきたので、何も考えてませんでした。  
確かにmessageはやめた方がいいと思います。  
カウンターでも良いですが、依存関係を増やしていいならmockを使った方がいいかなと思ってます。
Credential stores it as a long, so storing it in Postgres as a bigint saves the marshalling to/from java.sql.Timestamp.
tba_config.py, or a new file called constants.py. Happy with either.
No. But it more comfortable for me
 


Gorshkov Aleksey

Grid Dynamics
 +7 952 203 9796
Skype:aleksgorpsk
www.griddynamics.com
email: aleksgor@gmail.com

On 23.10.2012, at 14:04, dennisyv <notifications@github.com> wrote:

> In .gitignore:
> 
> > @@ -9,3 +9,6 @@
> >  target
> >  hadoop-hdfs-project/hadoop-hdfs/downloads
> >  hadoop-hdfs-project/hadoop-hdfs-httpfs/downloads
> > +hadoop-common-project/hadoop-common/build
> > +hadoop-hdfs-project/hadoop-hdfs/build
> > +hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/.clover
> do we really need these ignores
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
`do` is necessary only if you need to `<-` or discard return values in the body. Since the function is returning the value of a single expression, it's not needed here.
Should be an elif
0.upto(error.length-1) do | i|
Same as above:

<pre>
return :abort if (res and res.code == 404)
</pre>
you technically don't have to return e since it was the last variable touched, it is returned implicitly. just sayin
This line is redundant with including the TcpServer mixin
returning 0 is silly, just return
This is a lot of metaprogramming, which we generally try to stay away from (for one thing, it makes things harder to debug). Can you rewrite these as "real" methods? Thanks!
I think this can just be `AllowValueMatcher.new` - Ruby should be able to find the constant.
No trailing whitespace. Sublime Text 2 (which you said you are using right?) has an option somewhere to automatically remove all trailing whitespace.
So that we know when the request is valid for. This allows us to do  "updated since" requests
I really don't think this is helpful each item is search_result(item) which refers to peoples profile picture. Lets just leave it the way it was.
Yeap, forgot about that.
I added a comment to the code.
Same comment for `getSpecularOfMaterial`.
Looks like it's used in Columbus View, too.
Need spaces around the parentheses. Ex: if (x > 1) { }
Is there a reason we can't save off the image width and height whenever we change it in order to avoid calculating the property values each time we try and access with the width or height via kroll?
of course I mean Collections.emptyList() - also removes the need to use ArrayList where it should just be List.
I had forgotten that linter hated that kind of stuff.
Могу убрать `@`, но first всё равно нужен. Или что ты предлагаешь?
it's OK to do it in the declaration, it's the same though.
Sorry, it's impossible as function can't return an expression tuple.
Looks like *C++*'s `forward` is a joke as it accepts the only argument.
To @9rnsr:

It's in pull #952 now.

`mdimSArr.to!(int[6])` may be useful, but it isn't the point of `asFlatStaticArray`. `asFlatStaticArray` isn't a `to*` function as it doesn't create a copy. And it is important that it doesn't require to provide any types explicitly.
… instance.
@obviousjim there are a handful classes that derive from `ofBaseVideoPlayer`.  I just fixed them and will PR against your branch.
We are returning data, not the sql.
This code is original but could be changed acordingly.
I added that line because it seems that setting the attribute is insufficient. Please perform the steps listed at the top of the PR. The functional test is in tests/functional/button-markup.html

I'm currently trying to figure out why this weirdness occurs. If I can find the cause, maybe we can get rid of this line.
We try to keep each classes' own logic adhere to itself and avoid static class/methods as much as possible. Because over the time, those static methods/classes can become hard to manage.
Will do.  My apologies, as I have a lint script that automatically cleans up whitespace and replaces quotes to single quotes in cases of strings that do not employ format interpolation.
We should probably name this ``doctype_whitelist``, no?
I don't understand this (you are preventing it under built-in visitors here)
Так насладись их доступностью теперь!
I think we should remove (not deprecate) the whole CustomRequrieedFields class, its PHP logic remainder is the same as RequiredFields from what i can tell.
This name doesn't clarify that this object is JSON-specific. It probably should. In future when we support other formats we might have a base `CaseImporter` and then `JSONCaseImporter` and `CSVCaseImporter` subclasses.
Yeah, if Gary didn't merge it we can probably fix it, but for this particular tests these versions are harmless
While not beeing a blocker until we decided how to to documentation:

epydoc fails with: ```Bad dotted name```

The problem is basically:
```python2 musicbrainzngs/__init__.py```:
```ValueError: Attempted relative import in non-package```.
This is the same with every package using relative imports.

Though it works with our normal program, do we actually need to use relative imports?
The reason I used the hard coded value is to allow the value to be changed without causing any problems (just like you want me to rename __CG__ to __RG__) 

I'll change it to hard coded if you want me to
Sigh... no I did not
`doRequest()`
Changing that now. :watch:
For some reason, I only just noticed this file now... is it supposed to have no tests?
What was this about?
I would indeed find it better
Ok, keep me informed when you know more about that.
i agree. the functional tests in phpcr-api-test would need to run queries against the query api that check the different situations.

i think the test that is missing such cases is this one:

https://github.com/phpcr/phpcr-api-tests/blob/master/tests/06_Query/QuerySql2OperationsTest.php
permissionDef and aclHelper aren't available in this class .. in the ORM version the aclHelper is added through the constructor
> Methods which shall be annotated with parameter or return value constraints

Methods to be annotated with parameter or return value constraints

What about adding a note about why validation of static methods is not supported. Same for the statement in "4.1. Requirements on classes to be validated" where static fields and methods are excluded.
az nazare fasele ha va tab ke moshkeli nadare!! :D
Yeah, figured you'd make a comment about that. So, the default Map when you declare one is immutable. I wasn't quite sure of the best way to pass a mutable HashMap into something that accepts Map, and I didn't want to explicitly require a HashMap on the method definition in case we wanted to change the backing structure later. What's the proper way to handle that without the toMap?
I know i wanted to change this as is a legacy clanname....
Because this is now in the layout I think it will now appear when you're not actually on the map view won't it? Which won't work very well...
Do you actually need the filters above?
Ah, no more cordova at all, eh?
I think this is OK. We can always change it later if we need to :smile:
-were
Good point, I will add it and add "For internal use only."
you should remove this step completely, since we don't want anyone doing it.
Is is possible to set time zone, recurrence, participants and reminder?
We'll look into it
you should move lines 9 to 15 outside the if for -scm
The Jake on the webplatform will have to be done even on SCM.
Isn't present/absent a better idea.
You can just rely on their package to work properly as you're not wrapping it in an interface with potentially other impls