labofsahil/github-event-dataset-2012 huggingface dataset:
Sorry, I don't know.
But, I think it is not used directly. (use Gtk::Widget#window)
Fail... I meant to :)
As far as I can see, this line breaks the build. Please fix that ASAP.
Looks good for me
@citrus sorry for not being clear, when I was talking about the order, it was about the order of the option type : size, color and not the option values (blue, green,...).

I gave a shot and this basically fixed it :

```ruby
  def option_values
   variants.map(&:option_values).flatten.uniq.sort_by { |v| v.option_type.position }
  end
```

I believe the option_values should be ordered aswell :).

Maybe there is a cleaner way to ensure the order instead of playing with enumerables.

What do you think?
Rebased, and pushed to Iliyan, thanks: https://github.com/ipeychev/liferay-portal/pull/47
Nie masz w repo klasy `WordBoundaryTransitionLabel` chyba. Dlatego nie przechodzi testów. Jak wejdziesz lokalnie do folderu repo, to wpisz
`git pull git://github.com/Hazardius/amu_automata_2011.git`. Powinno wtedy być lepiej.
Yeah, the whole thing is a mess. (I think I have an AMD, and it is converting!)

And damn, it took me two hours to track down exactly what was going wrong, as it 
was deeply buried. Gaaah.

On 1/10/2012 12:31 AM, Don Clugston wrote:
> By the way -- with 80 bit reals on the x87, the behaviour on AMD is different to Intel! AMD converts signalling to quiet, Intel does not (or maybe the other way round, I don't remember). So it's not even as predictable as an SSE/x87 difference.
>
>
Frank

Ok ... did a full clean / make and all now seems fine.

Must have been some weird hang over from the different commits.

Thanks for checking.

Regards
Mark

On 15/01/12 21:02, Frank Hunleth wrote:
> I verified that there are no source code differences between the two
> Linux kernels, so the problem must be somewhere else. I did switch to
> using mdev, but I don't think that's the problem. Anyway, Ethernet
> works fine for me. Here's a copy of the build that I'm using right now
> if you'd like to try it:
> 
> http://dl.dropbox.com/u/37770195/beaglebone/uImage
> http://dl.dropbox.com/u/37770195/beaglebone/rootfs.ext2
> 
> I don't have Ethernet start up automatically, so run "udhcpc" to bring
> up the interface and get an IP address.
> 
> On Fri, Jan 13, 2012 at 10:32 PM, Frank Hunleth
> <fhunleth@troodon-software.com> wrote:
>> Just wanted to give you an update. I rebuilt clean with the current
>> tree to make sure that I didn't have any stale files. Ethernet still
>> works for me. I haven't checked the patches again yet.
>>
>> On Fri, Jan 13, 2012 at 11:36 AM, Frank Hunleth
>> <fhunleth@troodon-software.com> wrote:
>>> Ugh. There must be something subtly different with the patches that I
>>> didn't see. I'll double check this weekend. Thanks for helping debug
>>> this.
>>>
>>> On Fri, Jan 13, 2012 at 11:10 AM, Mark Jackson
>>> <reply@reply.github.com>
>>> wrote:
>>>> No ... it's still not working.
>>>>
>>>> I can't ping with commit f9614a157d23ed5d10e5b148e723c70ba611a84d.
>>>> But ping works with commit 10ef7e71118146fd91cbb5fa01f6a3b704b75897
>>>>
The last functional change related to that was 9630b99.
Here is a case where a timeout is throw, or any other exception is thrown.  The user can receive the exception (via an event)...
please tell me this fixes MMS!!!???
I'll send a PR fixing CS issues in general.
This is a row-major Iterator, correct? How hard would it be to allow for column-major Iterators?
Does `2.1.*` include `-dev`?
Is this how I should handle an optional boolean that defaults to true, or is there something simpler?
Seems to have been reverted in commit f4676661e40fbf0dd7688d2dff39862360cf6c0b.
Final functions that do not override anything do not get a vtbl[] slot.
I believe the expectation after a server.save is a running server, by changing this to vapp_template it won't it be a not-yet-booted machine?
do we actually need to fetch all experiments and than iterate over them? probalby we can just delete ontology terms directly from Assay/Sample properies
Let's remove it then - it'll still be preserved inside previous version of this file in git
Yup this is an issue, consider the following:

```javascript
var list = new Y.LazyModelList();

list.add([
    {name: 'Eric'},
    {name: 'Ryan'},
    {name: 'Luke'}
]);

list.revive(1).get('name'); // => "Ryan"
list.remove(0).name;        // => "Eric"

list.item(1).name;          // => "Luke"
list.revive(1).get('name'); // => "Ryan"
// ^ That's not right!? They both should be "Luke".

```
If the implementation of `Y.soon` does, yes.  And gallery-soon does use it.  Speaking of which, maybe it should be pulled into core :)
From what I can gather, it sounds like fixtures are for putting the DB in a known state before executing tests. Is that correct?

How will these fixtures be maintained?
Why are they all separate files instead of a single DB dump (the Elgg DB has highly interdependent tables).
That's incorrect, because these are operating on two completely different data structures.

One of these representations attempts to perform validation over parsed yaml, whereas the other requires the overhead of ValidConfig (specifically, a call to `ValidatorWithNamedPath.build_dict`) in order to function. The trouble is that the ''intent'' is exactly the same, and that's no good here.

However, this entire cross-check is obviated by performing a dummied apply_config, exactly as tronfig used to. So, we'll do that instead, with the note that the entire validation pass remains useful for granular detection of parsing errors, whereas apply_config remains useful for detection of broken functional dependencies like exactly those we're trying to detect here.

tl;dr: replaced by a dummied apply_config.
Switch to ARFU or akka.util.Unsafe
refactor with the case above: there’s too much repetition here
any reason why you didn’t port `futureFlowLoops` from the old tests?
Put these in config
was concerned about the mutability of channel.  maybe was overkill.  removed.
On ENGLISH, I just think that files like this with ENGLISH all over just need one comment.  Obviously when we to the refactor to add multi-lingual messages, we need to review all the strings in files like this anyway, so marking each line is just noise.

On the other, I think so.

I would wait to standardize this to an API directly compatible with the other auth stuff, probably Warden.  The design is for the Account class methods to mediate, and the rest goes on the *right+*account patterns.  The point is that we shouldn't interact with the User object, but with the +*account card, and extend that card model set with the authorization API (generate pw, etc.).

Actually, it isn't much of a wait, this is the next thing I want to do on the end of this chain of development and now that I'm ATP, I was going to do that.

In the past, I have kept adding to my tip branch when I get to ATP, but I'll start a new one on the end of this one this time.  I know you are right and I need to be discliplined about where I'm working.  Come to think of it, I need to add some bash code to put my git branch in my shell prompt.
The RE for split_content.   Matches a the start of the string, so you strip them off until the string is empty.  The first group will be the string before the first chunk group, or ''.  Then we go through the ACTIVE_CHUNKS (in parameter chunk_types as a Hash).  The Hash values need to match the number of groups in the RE for the class so we can calculate the ranges for each class.
just use <AE_FMT_FLOAT and you're good - all formats less than float are PCM and will always be
**§**lifecycle
Isn't this second conditional redundant? Unless I'm missing something, by performing the first check, you ensure that self._text will always be unicode and thus retrieving it will always return a unicode object.
'Observations and predictors dimensions are inconsistent' ?
I guess this question applies to several classes in this file.
I don't expect it will need to be overridden because of the approach taken in next few lines. In any case, I have added docstrings here as well.
Nice! It's about time we had a tool to do this. :)
Shouldn't you set GET[k] or POST[k] only if k is a valid key for each?
this is very likely not right. this should reflect aspect of frame
May I be so bold to suggest that we make the default mode to not stream silence and an as.xml settings to enable streaming of silence. This then matches our Eden behavior. And since the streaming of silence seems to be aimed at older receivers to prevent dropout of passthough mode, seems those are in the minority rather than majority.
No http status code?
Great docstring :)
This is wrong. This filter is not part of Twig but of the Twig-extensions repository (and it is documented there, or should be)
How about defining the default `__main__` as class constant?
Got that!
Why are these emptied _before_ running? Seems like something that should happen on completion (and will prevent retaining views for longer than is necessary).
Can you flip this condition and return early, instead of putting this whole method in a conditional?
Actually, I just realized this is in a category. How would this even work? Might need to use associated objects?
Why not just offer a method to get an `NSAffineTransform` from a `CGAffineTransform`?
Properties should be preferred to ivars whenever possible, _especially_ for objects (so their memory management can be more explicit).
Yeah, but from a Node user's perspective: Do I (as a developer) care?
At least I don't think so, as I want it to "just work", and I don't want to deal with the file format ... ;-)

I just wanted to keep it simple, but I'll change it to "32 and 64 bit" definitely :-)
I'll see what I can do this week.
besoin d'un test pour s'assurer que tu es connecter pour pouvoir voter?
Hmmm.  I guess that the comment was not clear enough:-)  I did some profiling and found that the wrapper enabling the caching generally causes poorer performance.  Basically, it slows down all N calls in the vectorize loop, and generally N is quite large.  Thus, unless a single function call takes a long time, it is better not to cache.  Suggestions on how to make this comment more clear?
I'd add this to the BEGIN block as:

    require Try::Tiny;
    Try::Tiny->import;

Since otherwise it'll load Try::Tiny in the initial init phase, where it isn't needed.
ветка else не нужна. просто return иначе
тут можно проверять deletingCodeTab равенство на NULL. тогда переменную contains вообще можно убрать
Nice. Cleaner.
It's my first pull request to rails. Fixed.
Actually, it looks like that's made a ton of tests fail. Investigating that now.
`Function` nodes are already dearly departed by this phase:

```
ticket/6526 ~/code/scala ./build/quick/bin/scala -Xshow-phases 2>&1 | grep --context=1 tail
                uncurry  12  uncurry, translate function values to anonymous classes
              tailcalls  13  replace tail calls by jumps
             specialize  14  @specialized-driven class and method specialization
```

(I wasn't sure myself, which is why I added the test case for this.)
Square copyright header
replaced with pop
Yeah, that sounds good, I think I'll do that.
@haz , In PR #1508 I've given this combinatorics overhaul. I'm asking around to see if anyone can look at it. It already had a favorable review up to the end of @saptman  's work So if you could help in reviewing the changes, that would help.

As for the edges, they are now stored in FiniteSet.

@saptman , if you are around, could you review the changes? Haven't heard from you so am not too hopeful, but I'm just asking in case you have some time to do this.
I was under the impression that abc was the standard way to get single letter symbols in docstrings. Was it changed?
If it is indeed malformed and this char is not a digit this will raise an error. I am not sure whether we should care about this.
Please keep two empty spaces between classes.
I guess that can totally be deleted now.
Agreed. Works without the expand.
no, I do not want to depend on R2SWF; move it to `Suggests`
is `controllerPath` always a subdirectory of `basePath`? If yes, its all fine :)
and this to 
```php
* are 'imagick', 'gd' and null. Null value means that fallback mode will be used: ImageMagick
```
No, i forgot to do a git pull before leaving
so, I should drop checks for zero then ? 
and if value is negative, set it to screen width ?
Interesting use of Object.
Is it better? faster? stronger?
which was a mistake.
MongoDB provides an implicit ``AND`` operation when specifying a comma separated list of fields. Use :operator:`$and` when you need an ``AND`` operation on the value of a single field.
Nothing works unless a valid key is present, I haven't decided which way is best, either have it ready to go with just entering a key and having it enabled right then as I think the user would want it, or have it disabled and require them to turn it on and enter the key.
Yes I assume it's dealt with elsewhere. Key word being assume.
Should be only one space
Why import if not using it?
:)  Yeah, not sure how to specify "whatever the user put in the file" type though...
Thanks for suggestion! :)
511-516 allows search on diagnosis and is called after every event on searchfield. UI freezes when list is very very long. Currently Line 510 check that searching should be done only if last key event is "Enter".
So, only Line 510 needs to be commented
Getting rid of the `guess_bounds()` is good. Even better would be to use data that included bounds so you don't get the warnings.
I don't like having this file depend on debugmode.py. Can you move the list ops_with_inner_function to the file theano/gof/op.py? This will break the circular dependence.
No good reason. It will be a little faster, but I didn't taught of that.
this % is unnecessary
SIGTERM is not 9.
No, it needs that only for the LLVM backend (which isn't used by Cython). It uses the external typedef name if the typedef is external, otherwise it uses the actual types to allow code reuse of expressions of equivalent types. I got everything working, except for one pyximport test, but I think that might be my test setup. I'll try a new PR next weekend.
this is wrong. new features won't be added in a bugfix release, and the phpdoc is wrong
The factory is needed to decouple the computation layer (mean/quantile curve/maps) from the data access layer. 
The post processor should not know where the data come from and where will be stored. 

It is a very common and broadly used pattern, so it does not obfuscate the code.
Ест ощущение что это должно быть `not p_value.startswith('<')`.
why out.println ?
Is that the main motivation of svec?  The repeating zero between 1 and 100 will be encoded efficiently in svec.  So [3, 4] and [3, 0 ..., 4] doesn't make big difference, though I know it adds zero in between.  I concern more about the anti-scalability of the single-node window function.
Check seems sensible.  But now that we are no longer using LAPACK isn't the comment a little confusing in terms of current justification?
I think the automated build failure message is due to the missing "#" at the beginning of this.

(Don't worry about it, I can fix.)
You need to add details about the settings change, including the new values for the three bundled outputs.
The mobile template doesn't exist. Given input only has a desktop feedback form and that you said it works fine on mobile devices, my vote is we ditch the decorator.
Watch out for stray linebreaks. ;)
Homepage goes above `url`, with no newline in between.
The hard-coded path to the sdk should be replaced by `MacOS.sdk_path` (to support Xcode-only installations).

Also note: Now, that we have wx 2.9.x in 64-bit glory, you might not need to do all this stuff.
index 1 holds the requesttoken
index 2 holds the dir
I still need to write a test that ensures that the path of `inbound message -> application that calls msg.reply() -> outbound message` works as expected (this is non-trivial in the MultiSmsSync case which needs transport_metadata to be present and correct for this to work.
Yes. Fixing this requires a major refactoring of the dispatchers, and is out of scope for this branch.
Ditto here?
Joshua Roesslein <reply@reply.github.com> wrote:
> > @@ -41,6 +41,10 @@ def testmentions(self):
> >      def testretweetedbyme(self):
> >          self.api.retweeted_by_me()
> >  
> > +    def testretweetedbyuser(self):
> > +        self.api.retweeted_by_user()
> 
> This doesn't seem to work. I believe this method requires an user_id or screen_name otherwise Twitter sends us
> a "page not found" error.

You're right - my apologies.  Removed this bogus line.

-Jan
This could run into the same problem I outlined earlier, no? If commit is null, this'll throw a `NullReferenceException`. Probably best to leave this implementation as it was. that way, if you fix the `==` case, this is fixed too.
We've started to shy away from this because they tend to overrun the 65k field length limit in MySQL. Probably fine, but do the profile objects have any arbitrary length fields? (Arrays tend to break it, because one person ends up having say, 254 previous jobs listed on LinkedIn).
"averate"
Oops. I guess I missed the copyright on here as well on my initial pull. Would you mind updating?
I don't know ruby, but this appears to be nested double-quotes. Do they need to be escaped? Even if that's valid, the coding convention doc says to use single-quotes inside double quotes, so I think it's safest to keep this code as it was.

There are several other cases in the ruby code that this comment also applies to, so try to find and fix those also. When you're done I'll do another scan of the code. Thanks.
There is some other code that does the opposite of this (converts from a url param string to a param map) that should probably be encapsulated into this object for easier future maintenance.
Yeah, I don't add the event listener to .main-view then as soon as you leave the sidebar-resizer you'll lose the mouse events.
don't concatenate strings in a loop, that is a string copy every time. Use an array and join.
slightly excessive parens in the tests of these loops, which you don't have in test_counting_accuracy
`Time.now.utc` ? Could it ever matter?
Why some tests were skipped?
fix your gitignore, you accidentally overwrote the existing one :-)
svn+http://code.djangoproject.com/svn/django/trunk/#egg=django-trunk
in file, that's just pip install thingy.
OK. I think it would be better that we just remove this language initialization because it depends on the temporary user creation. So can you maybe just remove all this and leave it so that it just reads from the user object and sets the language and this is it?
Of course you can. But you cannot add a name to a set of included URLs (because there are multiple of them). You have to check how to one you want is named. So check the `django.conf.urls.i18n.py` file.
this is not working properly... you were right it's time from module loading and not when field is used...
so i will remove this out. We will have to fill this when we are saving post in db...
Hm... This is still not good. 

Is this a bug?

    a = datetime.datetime.today()
    isinstance(a, datetime.datetime) # True
    isinstance(a, datetime.date)        # True
    a > datetime.date.today()             # TypeError: can't compare datetime.datetime to datetime.date

Shouldn't inherited  instances be comparable?
It is not enough to define a link, you also have to use it above in the list of prerequisites. :-)
This is just in to force a crash (try to save)
I just meant the first "/". Probably shouldn't be there, right?
I don't think this will work. Try using the flickr plugin and erasing 'kittens' from the `Tag Search` textbox. As soon as you unfocus, it reappears.
This semicolon is unnecessary, right?
Also, `butter-dragging` is no longer appropriate. Perhaps `butter-resizing`.
Could you call this something more descriptive? Perhaps `html_element_whitelist`?
NexusClientException ?
If we want to skip find_role for the create action, I'd assume that we'd also want to skip it for the destroy action as well.  Not excluding it may not have any side affects other than executing additional code that we don't need to execute. :)
well that just takes all of the fun out of it :)

gonna have to fix the bug in soynode first.
Snappy is included in the gemspec, but the gemspec does not allow (AFAIK) pulling from git, which is necessary until the main snappy repo accepts the pending pull request.  Putting Snappy in the Gemfile overrides the location that the gemspec uses.  That line will go away eventually.

Rake is not necessary for Kafka-rb's functionality, only for development.  Therefore, it goes in the Gemfile.
This is identical to the `isiterable` of #240, right?  Once that PR gets accepted, this should be switched over to using that version of `isiterable`.  Or, if it isn't done by the time this is ready to merge (very soon, I think), I can just do this directly in master once #240 is ready.  Either way, this is pretty much just a reminder.
Don't like the change from NIs, Q, TN as names briefly here, (changed from call to make_plans in claim_v3 and changed back below in make_plan) just makes reading a little harder
"Wipe out cluster and start fresh" sounds an awful lot like what we do in rtdev:setup_harness. Do you think it's worth splitting that function up so that there's away to take advantage of the `git reset` stuff we do between tests here?
I think this has potential to confuse people. How about "Mined balance will be available when it matures in %n more block(s)"?
I don't really care that much, but why the punctuation change? Just makes more work for the translators IMO.
ah... I didn't realise it was doing that!  Thanks for pointing that out :)
Change this to an annotation please.  In PHPunit it is:

``@expectedException SocketException``
@craigambrose do you have any thoughts?
See above comment regarding group activity
I see. What is the canonical way to setup a size request for a window?
I just want to measure the time it takes (display the TimeSpan) when it happens...
With that, everybody could verify if this commit is usefull and see the time earned.
Could consider a default parameter maybe. Remove `run` and go with:

    def preview(params: DeployParameters): UUID = run(params, Task.Preview)
    def deploy(params: DeployParameters, mode: Task.Value = Task.Deploy): UUID = {
Fixed that. 

Agreed, there's no need to check `len`, but some people prefer not to use implicit checks.  I should've checked the rest of IPython's code to stick to the standard style.
litterate -> literate here as well
Here's the full comment, which you can't see in this diff view:
```
            # On Windows, use os.system instead of subprocess.call, because I
            # was having problems with subprocess and I just don't know enough
            # about win32 to debug this reliably.  Os.system may be the 'old
            # fashioned' way to do it, but it works just fine.  If someone
            # later can clean this up that's fine, as long as the tests run
            # reliably in win32.
            # What types of problems are you having. They may be related to
            # running Python in unboffered mode. BG.
```
You're calculating json.dumps() twice here. Could use a temp variable for efficiency. (Yes, I know it is "fast enough")
what happen if $column->length === NULL?
A test to test the buffer check has been added as well.
There needs to be a ``not is_list`` check for the first half of this condition, I think. Also on line 74 and 201.
client_type 也和 status 一样的处理, 如果你作成一个宏, 更好看.
比如 
#define PROCESS_STATUS_CHANGE_MACRO(mmeber) { \
           if (member) { \
                 s_free(buddy->member); \
                 buddy->member = s_strdup(member); \
           } \
}
I'm pretty sure this line doesn't compile. You can't use the bitwise function | on booleans, so if it does compile it's treating it as

if (block.getTypeId() == (Config.getInstance().getRepairAnvilId() | block.getTypeId()) == Config.getInstance().getSalvageAnvilId())

Regardless, I'm pretty sure you meant to use || instead.
Thanks for fixing those ^^
good question, I've seen it quite a time in customer data, especially if the data was coming from outside sources (of course they should skip those properties in their domain layer, but you know how they are)
Can't it be simplified to `if (false === $sent || feof($this->stream))`?
This also checks if the stream is closed or eof
"현재 EPUB은 이 컨테츠를 위한 미디어 오버레이가 포함되어 있지 않습니다."
should be
"현재 EPUB은 이 컨텐츠를 위한 미디어 오버레이가 포함되어 있지 않습니다."
Seeing that the engine is still not officially announced shouldn't these entries be marked as unstable or at least testing?
Vanwaar de naamverandering?
Bundler shouldn't be required in the project itself. People who use this library should not be forced to use Bundler themselves.
maybe you should test the result of toArray() instead of asserting that true === true :) if it fails PHPUnit will mark the method as failed.
You don't need to use them. Still better then now, or? Maybe you should go into detail whether you agree with this PR in general.
`getClientData` is deprecated in favor of `getViewData`
```php
for ($column = 0, $count =  $this->getNumberOfColumns(); $column < $count; $column++) {
* No, the position is a query string parameter and not a placeholder on the route
* No need to use `url` as we know that everything is under the same host anyway.
Properties (including functions) are case sensitive. This should be `toString` (same for `r.os.tostring`).
can you fix it?
Yes, they may all change to this, but it could break the app and there is not much benefit in that.
Note: this does exist on python3: http://docs.python.org/3.3/library/unittest.html#unittest.TestCase.assertWarns
I don't think you need to re-register the signal handler - you're thinking of libevent timers
the basic idea about htonll and ntohll is to handle with 64bit long and double.
1. check ENDIAN uses "if (*reinterpret_cast<const char*>(&num) == num)"
2. divide the 64bit long into high_part 32 bit and low_part 32 bit
3. use htonl or ntohl on both tow parts.
4. combine the tow parts into a new 64 bit value and return
Probably just because the options hash hasn't been updated yet. We can change that.
I just know internally that this math would be happening twice. Although, you'd need to parseInt it. Leave it as is.
This needs to be an HttpException, if error != 200 and TcpException for everything else (connection timeouts etc.)
I'll remove it latter. Thanks!
I do not get this. Are we changing the 'counting' algorithm here? This code is regarding the click handler on concrete entries. But no counting is done here. In fact, the counting is always done by means of the getSelectedEntries().length procedure instead of maintaining a variable with this count that we have to update considering all the cases that may arise (clicking on a item, deselecting all, selecting all, etc.). Since we have the selected log entries with the class 'selected' assigned we can get the number of selected entries from here ;-)
add bugzilla # number below, thanks.
they could easily be, but I have been bitten by having generic event names in handle event.
If I needed to add another click handler I would either need to detect which element is which or go back to this method.
See the previous (
  comment about this syntax
)
+1, we had better exit the false condition at first to reduce indentation.
Yep don't move it, just make sure *every* `custom_dialog.css` file gets updated.
Maybe we should update the selector in search.css
c -> contact
Well, that breaks everything.

I am personally kind of satisfied with just `BaseForm[x]`


The first function should be fixed now, e.g.

    >> BaseForm[x, 2]
     = x
`.` ?

Which value do you put in the  `baseClass` property?
There's a copy constructor for Stats, so you can just supply a Stats object to new Stats(stats).
Not sure if there is a way to detect this somehow ... however, I guess there might a gem for it :)
I didn't look at that. I was just removing semi-colons. Generally, I only like `i -> j` if it's a key/value relationship.
We need people to hit this action every time. Browsers can cache HTTP 301 redirects - so we have to make sure this page is served up.

The PageRank problem @bradleywright mentioned were not discussed - so I cannot comment on it.
When is this going to be called with `None`? If we don't expect it to happen, is there any reason to handle it here?
is this right? looks suspicious to me.
This makes me think that I need to migrate "_respondBasedOnResult" to the "session decoration".

I will.
i've never had to use `update_all` before, so +1 for an alternative
Node factory interface
New config options have to be described in doc/Configure.help
move logic  to account role  please
Also link to new project should exist even if user has  1 or 1000 projects to create
I would like this on `Front` instead. Let it be responsible for preping itself.
For now I suppose we don't have socket in win32 and I do not set BUILD_UNIX_SOCKET on this architecture.
This set the compile_dir to what the current default is, but allows it to be configurable.
Let's make this a bit more concise. How do you feel about:

> To write a own custom filter, ...
I think this file should be added to .gitignore and remove it from git repo
We should try to valide options before doing anything, especially registering listeners. (avoid memory leaks)
So that we should execute validateOptions before `EventTarget.initialize.call(this, options)`.
(It wasn't the case in the old code, so ensure running tests after this change)
@lloyd - excellent note, I should have put that in there myself.  Perhaps instead of having a discrepancy with the upstream, I should just change upstream to pass the name of the message.
this change adds 5 new database functions, and removes 1.
One is to redirect after the click, so we can simply use it as URL in the new tab.
The other is to just count clicks. As described in issue #5
For another example see mozmill!

It's official repository is on GitHub, but it's tracker is bugzilla.mozilla.org.
Check the it's [changelists on AMO][1], it uses bug numbers instead of pull numbers!
(The commit messages refers to bug numbers also - but this is not the subject here)

As I noted in my previous comment, the issue numbers are more informative than the pull request numbers.

[1]: https://addons.mozilla.org/firefox/addon/mozmill/versions/
I know it is not a test, but I don't know what metrics would be
used for that kind of operation aside visual inspection...

On Thu, Jul 26, 2012 at 4:33 PM, Nicolas Pinto <
reply@reply.github.com
> wrote:

> > @@ -50,3 +53,18 @@ def test_simple_segmentation_with_provided_path():
> >      assert np.abs(pe - 0.010183) < EPSILON
> >      assert np.abs(re - 0.020385) < EPSILON
> >      assert np.abs(we - 0.0) < EPSILON
> > +
> > +
> > +def test_warp_2d():
> > +
> > +    # -- read in y_pred and y_true from PNG images
> > +    y_pred = misc.imread('./y_pred.png', flatten=True)
> > +    y_true = misc.imread('./y_true.png', flatten=True)
> > +
> > +    # -- compute the "warped annotations"
> > +    y_warp = warp_2d(y_true, y_pred)
> > +
> > +    matshow(y_pred)
> > +    matshow(y_true)
> > +    matshow(y_warp)
> > +    show()
>
> what is this?
> this is not a test...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/npinto/bangmetric/pull/9/files#r1247925
>
Agreed.  Will remove that part.
ok, so either we need a new method to set the connection we want to use, or we warn users that this only works with the default connection.
File is configured in UTF8 on my side and CR are converted to Unix LF
I have asked our UX folks to comment on this, also.
oops, messed, and force pushed from wrong tree. please hold.
Same here, can this be broken up into multiple lines?
I thought I saw others doing it but they might be similarly buggy. Any way you are correct. I would just drop 1.8 support though.
The idea is that the lazyLoadHeader function looks for GenericHeaders and converts to non-generics (i.e. To)
Please move the variable out of the quotes: 
$dsn[] = "port=" . $port;
Actually, the original is correct. If $wsdlClass is an object, it would hit the second condition, which would then test and ensure it's a subclass of Zend\Soap\Wsdl. (is_subclass_of can take either an object or a string as its first argument).
This is duplicated with the line 45 but with a different value
Still sounds fishy. But probably not worth investigating more.
Not a huge fan of calling it home.js.... perhaps stats and then have a small bootstable script block on the home view that calls into everything.
Meteor core doesn't provide hashchange event listeners... in one of there examples they propose to use backbone.js (backbone.js router to be more precise) for this purpose...

i wanted to make this demo app as simple as possible, so i removed the filter links and replaced them with text elements and just listen for click events...

if you prefer the approach with links, i could:

a.) add a backbone layer (using backbone router) and listen to hashchange events

b.) simply listen to click events and react accordingly, but not considering hashchanges
Line space between assignments and control structures.
should be ``elseif`` without space
Ok.
Does it enhance all methods? Or could it benefit from extending the class?
Anyway some more class documentation on all the larger classes would be nice (like here: which Sf class is enhanced, why and how)
Once again, ```lt``` is already evaluated.
Spaces everywhere
That's certainly a more descriptive name.  I've been trying to think of what else might end up using it, but haven't been able to think of anything, which doesn't really say much 'cause I think we all just use our favorite vim shortcuts and neglect the other 90% of them (I didn't even know about | till today).

The issue I see is, when would stickyHPos be set and unset?  Certainly setting it in the functions that want to declare a stickyHPos isn't an issue, but do we want to force every other function to set it to false?  Perhaps you had a different idea of how to do it?
unless f.object.new_record? is better
why are default_start and default_end methods, but default_interval is a property?
yeah let's not worry about it until we do a post-Kuma performance pass.
You don't think an image should be mandatory for the image plugin?
To make this more readable I would include the argument name `do_login=True`
Added support earlier for passing configuration(s) to `start_rel_from_url`
It looks like the indenting got a little wonky here. Also, if you're up for making lines 10-13 translatable, that'd be awesome.
replace google with some fake site, like YOUR_WEBSITE
I don't think this is right. If there is not apprentice scry then this'll throw a NullPointerException.
Ah ok. thanks.
As an extension to @AaronMT comment on this, I might suggest pointing to http://mozqa.com/data/webapps/mozqa.com/manifest.webapp for the single web app install test. We can reliably control where this webapp manifest is, how it's changed, etc since it's hosted on QA infrastructure.
Oops. This snuck in, but it was needed as well.
Ah hey, I see it now.
There are just 4 points here, so the square isn't being closed. If you add `true` as the last element in the path, it will auto-close the shape (and make the starting/ending vertex look better than if you just repeated the first coordinate)

`v = [[x - size, y + size], [x + size, y + size], [x + size, y - size], [x - size, y - size], true];`
Remove block comments, as this was just debug for proof of concept.
You can't put chestplates in the leggings section :P
Hi Sarah,

I agree that it's not nice but also can't think of an alternative without
partially replicating the element's content which is also bad :(

See the conversation I'm having with Paul about it at
https://github.com/aptivate/econsensus/pull/59/files#r2318455.

Jo

On 5 December 2012 09:58, Sarah Bird <notifications@github.com> wrote:

> In django/econsensus/publicweb/templates/decision_detail_snippet.html:
>
> >                  <dd>{{ object.get_feedback_statistics|get_item:rating }}</dd>
> > -            </a>
> > +                {% if "edit_decisions_feedback" in organization_permissions %}
>
> I've seen this pattern in other places in django templates we use - e.g.
> on the Aptivate website but i think in other templates for econsensus too-
> where you're deciding whether to wrap something in a link tag or not (you
> have to close the tag which means two ifs). I don't know what a better
> pattern would be. One thought though - whether this is a good idea or not
> may depend on the efficiency of the particular test. Django templates are
> not particularly quick so you wouldn't want to do this all over the place
> on something that was heavy to evaluate.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/aptivate/econsensus/pull/59/files#r2318531>.
>
>
i addressed this in mvp/_design/util
should make this member private, and add getter/setter methods.
So users can subclass accessor methods
urllib2 uses urllib itself to do unquote.
Doesn't work because this.collection is undefined at this moment.
what is the significance of 'oae-performance-files' ?
Is it necessary to call a function that performs 2 extra queries?
It looks like you have all the data that's there:

```javascript
var comment = new Comment(commentId, body, contentId, ctx.user().id, created, level, commentReplyingTo.commentId, false);
comment.createdBy = ctx.user();
callback(null, comment);
```
I think we can provide a bit more information on the error here. Also, is there any cleanup needed in this case? like closing the stream on completion, or is that done by the storage backend?
Should be createComment as well.
I think you want a hash here?
ie: ```var members = {}; ```
On Tuesday 11 December 2012 01:33:12 Mirco Dotta wrote:
> The reason is that askOption has some additional logic for catching
> exception and it has a timeout (which prevents the IDE to freeze for too
> long).

Good point, I've updated the commit.

-- 
Mirko Stocker | me@misto.ch
Work: http://ifs.hsr.ch | http://infoq.com
Personal: http://misto.ch | http://twitter.com/m_st
Anything in particular that doesn't sound right?
Ch. 5 ii
Please format to something like
 ```objc
const NSStringCompareOptions compareOptions = NSCaseInsensitiveSearch | NSNumericSearch | NSDiacriticInsensitiveSearch;
if (_bonjourServiceName  != nil  && [_bonjourServiceName compare:service.name options:compareOptions] != NSOrderedSame) {
```
Apologies for not having a style guide.

But also, why NSNumericSearch?
This line has trailing spaces. Please remove them, and check for any other similar lines.
It's been done in other drivers as well + the ifs aren't that readable either.
7e83f32e3844ba8ee34c8a79ee6fa3f558fcb46a

We don't need this. :)
Это будет работать неправильно при длине массива не кратном 100. Например, если длина массива будет 50, здесь всегда будет получаться 0, независимо от процентов.

``` objective-c
int numOfMarksForMutation = ([self.DNA count] * X) / 100.0;
```
Здесь ошибка. Надо считать количество неравных объектов.
This will not work on Python 3.
Throws an exception:
RuntimeError: dictionary changed size during iteration
Now objs_computed.
I don't think this is necessary.  Only the code from lines 567-574 will work.
I would do the simplest thing for the user, which is what you are doing.  The user can always reconfigure it if they want.

I'm mostly concerned that you aren't including the message as part of the exception context.  That is often critical for tracking down the source of an exception.
*stream
Ok, now I fully woke up so I can make more useful comments :)

Please get rid of this and all the calls to `page_indicator()` as they are meaningless in reflow mode anyway.
We tend to shy away from assignments in conditionals. They can appear to be bugs.
same here verbose should be documented in docstring
app.use(function (err, req, res, next) {
    res.writeHead(500, { 'Content-Type': 'application/json' });
    res.end('{statusCode: 500, message: "Something went wrong"}');
});

like this?
Use a generic name here not one related to any real ongoing client projects (since everything here including what I am typing right now is public).
The dot should be a semi colon.
It's maybe better to use a single code block with comments:

    $ php app/console twig:lint src/Acme/ArticleBundle/Resources/views/Article/recentList.html.twig

    # or by directory:
    $ php app/console twig:lint src/Acme/ArticleBundle/Resources/views

    # or using the ``@BundleName`` syntax:
    $ php app/console twig:lint @AcmeArticleBundle

And the `@BundleName` syntax is called the bundle shortname in the quick tour docs, it's maybe better to use it here too for a consistent documentation.
头部信息
=>
请求头
Does this work if you just do `t(".sign_out")`? If not, can you change the locales file so it does?
use () for Unit
Nice. Just one more addition: I believe you should not add other errors than the `subset` one. Besides, I think `allow_blank` is not a valid error message, it uses `invalid`. I can think about something like this:

```ruby
if value.blank? || !value.respond_to?(:each)
  record.errors.add(attribute, :subset, options.except(:in).merge!(:value => value))
  return
end

value.each do |elem|
  if !superset.send(inclusion_method(superset), elem)
    record.errors.add(attribute, :subset, options.except(:in).merge!(:value => value))
    break
  end
end
```

If the value is blank at this point, it means it already passed the `allow_blank` check in `EachValidator`, right?
Wdyt?
I believe that SymPyDeprecationWarning is turned on by default (one of the reasons we wanted to do this in the first place)... ?
Didn't know about the third argument in string.replace()! Very convenient.
You should also add current_space to the cache so that it is available the next time it is requested.
can we not add deleted=False to the find_all calls? What if we _want_ to find_all with deleted=True? This will be needed in the mgmt api
@danberindei Hmmmm, good point. Let's leave it as is, but it should really be documented, and not just for the transaction manager lookup configuration option, but for all similar ones, i.e. mbean server lookup, transport lookup...etc.
Cool to see you checking it all :)
yes those - among others - were the doubts that prevented me to make such a pull request myself as I don't knot how this fits in the greater design.

Aren't there tests already checking for proper cleanup?
I'm able to rename the file. I'm unable to stage the renamed file to commit.
Your solution is made to rename the file (Already done) not stage for commit.
Please fix the param name in the phpdoc
I tried it that way, but since each axes instance is utilized many times over, it ends up looking cumbersome
@WeatherGod Fine with me.  I was not aware of np.hypot.  Has it been in numpy for a long time?  Now that I know about it, I will use it!
This is exactly what I wanted.
по описанной идее за MISSING должен отвечать примитив а не Form
I think we should add a note saying that `mockWin` has to do with testing. We could use a even more specific name, like `olMockWin`.
for to avoid call two functions
Use querybuilder helpers instead of composing queries like this, any reason for doing this?
there is $this->escape view helper
i was only fixing tabs ;) changed.
Přikláněl bych se k tomu, co píše Zbyněk -- nechat schedule bez parametrů a udělat nějaký wrapper stranou.
No chance for me to know that this is used in pro.
... wups this variable is not used anywhere, going to take it out.
what happens if you provide default with more than 1 arg?
It's better to use PHP's version compare utility instead of a string compare.
I thought I made this comment yesterday, but I don't see it now...

Should we note if a page is being flushed for eviction purposes (to free up memory), and we won't be able to evict because of skipping an update?   That way we could return early instead of continuing forward with I/O that won't solve the problem.

Also, how does this interact with forced eviction?   I'm thinking it's OK because we won't skip any updates during a forced-eviction reconciliation, is that correct, so evictions because memory is getting tight won't be a problem?
Are you sure the method ``getContent`` can be called on ``$this->asset`` (of type ``object<Assetic\Asset\FileAsset>|null``)? I also found this on lines 89, and 100.
This ``if`` statement, and the following ``return`` statement could be replaced by ``return $this->buffer === -1 || is_array($this->buffer);``.
Are you sure ``preg_replace('/^        ...compile()->getRegex()))`` has the correct type as ``*`` is not a sub-type of ``string|array``?
add `the`
I think this should be:

    install_requires = ['requests>=0.13']
Or we want to add the values to `setInnerData()` in view than resetting it .
Figured as much.
I don't think you meant to change the indentation of the label tag here. There's one of these below too, in prependedtext.html
Why build this set if you're doing to build a dict right afterwards with the same keys
how about: Run all matching files in the same process
I'll change it back but that's message is too technical IMO for the reality of the problem.
Any particular reason you are dropping the type hinting here?
yeah .. maybe make it a variable to choose to enable this?
Space after `,`. Use something else instead of `i`. What is `i`? Minutes? Hours? Intervals?

I am also not sure if this is the right approach. What if some entry is missing in the database? Then it will select wrong entry.

I would do multiple database quires here. For each time new, where you limit entries that they have to be for future from then on and select first one available.
What should be done here?
Bruh. Don't do it like that. Then define form without those fields.
There is no problem calling python on Linux, it should invoke the same Python executable as "/usr/bin/env/python".
I just pushed a change reverting that.
@dkubb btw did you know that memoize has been removed from AS in master?
Subtotal is now being used.
this is no longer required for Trinity?
the .diff files are custom changes we did to the external libs
```ruby
@ invoices.reject(&: sent?)
```
If this is a simple example, then a storage adapter isn't needed at all. The built in default that Zend\Authentication\AuthenticationService comes with works just fine.

Storage adapter is a useful option that some users may want.
It doesn't seem to work with a `'multiple' => false` element. Try that :
<pre>var $input = $field.next('.chzn-container').find(".search-field > input, .chzn-search > input");</pre>
installer.pl is not run on upgrades and I wonder if the permissions won't be overwritten by dpkg on upgrades?
file = Math.random().toString(36)[2..]
Hmm. Either this was gratuitous or the tests weren't comprehensive. Either way, not good.
Oh, nothing to see here.... That was from me hunting around to figure out where my backups were going when I wrote to a time stamp previous to a delete. The answer? Probably under a couch somewhere. 

I did this so that the table pool size was one, and I can stall my integration test, and use hbase shell to examine its contents. Thanks for catching this!  That was silly of me to let slip by.
Right now I have to turn this main function directly in the command line to get it working.

When I try to run **circusd-cluster** I receive the following error::

    $ circusd-cluster prototype1.ini 
    Traceback (most recent call last):
      File "/usr/local/bin/circusd-cluster", line 9, in <module>
        load_entry_point('circus==0.6.0', 'console_scripts', 'circusd-cluster')()
      File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 337, in load_entry_point
        return get_distribution(dist).load_entry_point(group, name)
      File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2279, in load_entry_point
        return ep.load()
      File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 1989, in load
        entry = __import__(self.module_name, globals(),globals(), ['__name__'])

Any ideas?
Space before and after `=`, possibly in more lines
Is that a classname?
but if `diff == 0` nothing changes, right ? Therefore we don't even need to call this.change_overlay_size`(diff, diff);`
The is-loading template is probably shown only really short, right? That's all this does? Maybe it's not needed, since as soon as you have even only one resource you'll see something. May be less flashy.
Use braces for the `for`.