labofsahil/github-event-dataset-2012 huggingface dataset:
Nuspec'd up

To build everything in release mode, and push to Nuget, just run this one-liner:

'apiguid' | .\NuGetPack.ps1

Everything uses package dependency refs in the nuspecs *EXCEPT* for log4net 1.2.9, which actually includes the binary in the release.
Updated profile URLs to GitHub API v3
Replace RemoteFunctionCalls to use XHR POST method

Solves the issue where GET requests are hitting the URL length limit with large query strings.
YouTube deprecated most_viewed for most_popular

Besides the updated code, I removed the ?> at the end of Youtube.php
fix ors backup log //see comments
Fix build break around dynamic links - pulls in some of rob's import changes and Andi's nagios changes. [2/6]



 .../app/models/provisioner_service.rb              |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)
Openengsb   3080

Technically it is finished, unfortunatly, the .Net Bridge has problems to translate the Notification model and NPanday 1.4 cannot compile WPF applications (see dev-list discussion for further details) - so what should I do with it?
Rails 3.2 support (+ assets)
Fix propelDouble List widget

I'm trying to fix double list some of you have an idea about why i've exception :


Expected argument of type "scalar", "array" given
500 Internal Server Error - UnexpectedTypeException 

On save ?? @willdurand ?
In the command admin:setup seems to be a little bug.

In the command admin:setup seems to be a little bug.
When I choose "own" as skin an error message comes up:
[ErrorException]
  Notice: Undefined variable: useAssetic in /vendor/bundles/Admingenerator/GeneratorBundle/Command/SetupOrUpgradeCommand.php line 120  

I tried to fix it in line 120 but you'll see the changes anyways, I guess ;)

Bye,
Tobias

P.S. Symfony2 with bundles makes programmers life a lot easier :D
Implement nestedset list view for propel orm

With drag & drop to organize tree
Disabling pagination
Use jQuery treeTable Plugin & jquery UI

Doc comming
Adding Carey's talk video. 

Also, removing the links "on Vimeo" because I kept clicking them by mistake; YMMV
Correções

Olá Tizzei,

estou enviando minhas edições. Houve alguns conflitos e os resolvi sobrescrevendo os arquivos pelos modificados meus. Isso pode dar algum problema nos arquivos da pasta docs. Se eles foram alterados após aquele pacote zip que você me passou com o projeto, talvez você tenha que sobrescrevê-los. Outra opção talvez seja você escolher os commits que quer pegar e fazer o merge no seu branch.
Updated for 3.6

Updated metadata to work with 3.6 -- works flawlessly after my previous updates for 3.4 compatibility
New sponsorship info and reduced size of content font
uses new event system for CB1.2.3-R0.1
Additions for Tutorials
Updated XMLImageService to at least compile, uses BaseActor
Brazilian Portuguese translation

take here brazilian portuguese po and mo :D

cya!
Small fixes and tests for the listener
Ics

pick and choose what you want out of this.
Replace incorrectly used BLOCKQUOTE by BLOCKQUOTE_PLAIN.
Yeah, the value is there for maximum compatibility. It would be pretty trivial to allow `addPjaxParam` to be a string, which would allow the parameter to be customized, but I wasn't sure it was worth adding that until someone asked for it.
can we avoid very long line ? 182 char ....
How did you even notice this? *impressed*
Thanks! Needed indeed :)
Nice.
Why "Tenderfoot"?
Also, I don't really like the self.retried_connection. Can you remove that?
I would use the inplace list.sort -method:
Your now replacing a public field object with another -- thus someone could have a stale reference to the old version.

something like:
     
    resources = chief.external_resources
    ...
    chief._insert_into_suite_structure(resource)
    ...
    resources is not the one to use

IMHO this is a bug waiting to happen
http://railscasts.com/episodes/196-nested-model-form-part-1

Nested Object Forms 使用起来很简洁，可能刚接触需要了解一下
Смотри что я сделал: https://github.com/antono/glog
На этом работает http://antono.info/
Сэкономишь вермя :) Тебе еще конференцией заниматься.
@nikic: Semantically, `flush` is a tag, not a function.
One of the benefits provided by BufferredLogger was that log messages from one request would not interleave with log messages for different requests. Is there a replacement for this functionality?
:+1: bro
On Wed, Jan 4, 2012 at 12:32 PM, Christopher Smith <
reply@reply.github.com
> wrote:

> All the changes have been made and is_algebraic returns True for 1/(sqrt(2)
> + sqrt(3))**2 ... is that ok or should it not be a power higher than 1?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/smichr/sympy/commit/25816c74a7c504428cfb112bbca883ff26804be8#commitcomment-836571
>

I do not have an example in which `_denester` denests with the addition of
these cases
(p.is_Pow and p.exp== -1, or  p.is_Pow and p.exp.is_Integer); allowing
power higher than 1 is anyway
consistent with the name `is_algebraic`
Celtic Minstrel authored 7 months ago
EvilSeph committed 7 hours ago
Thanks, that was fast ;D
Why didn't you push it, yet?
Is there something not working fine?
I think django 1.5 is planing to drop support for python 2.5 :)
в принципе правильно если использовать только встроенные средства Рельс

но мы на своих проектах используем cancan для этих целей (я постараюсь описать как в письме). Cancan позволяет подойти гибче к правам авторизации. Это очень важная часть
Oula !
The indentation is off for this line, and the next 2.
Can   be merged
i was wondering about this hehe
Something is wrong with the header returned by Alloc_small in makeClosure, So the Closure_tag detection in shiftClosure doesn't work.
i have fixed your bug and i'm just playing with it unpaused and it looks very nice...
* we will have to fix the drawing method [now it's: for all children { draw edge and draw the child } but when a node has too many children, the edges of one child start to collide with its left sibling... so we need 2 passes: for all children { draw edge} for all children { draw the child }]
I like `!argv || !argv.length` better, but who am I :-)
Should we check that we wrote what we meant to?  But this code seems hopeless... without a buffering scheme there's nothing we can do here, and we're not even doing anything to protect what we send.  Love, can we nuke this code?
Idem here.
Yes, we need to be able to take an option, version, and ensure that that version of the package is installed
lol, this doesn't work at all! I am fixing!
how does this not break tests? lol
ja wäre cool, ich sehe du machst dir Gedanken. Sehr schön.
Es gibt in der Repository Klasse in der CreatePropertyClass-Funktion eine zeile

throw ExcRepository("Err: Property exsists");

Du kannst hier den STring mit nem coolen Fehler-Code ersetzen. Z.B "PROP_EXISTS" und die exception dann in AddProperty catchen.
The latest beta build consolidated a whole bunch of settings files (and I bet the symlink problem may be fixed too). I'll have to look into this and see what's changed and if any of this is needed anymore.
The only problem with putting that in the bundle config is that it's
up to the **app developer** to set it correctly not the _bundle
developer_.  All the other config options are config for the app
developer (location, handles, auto).  I guess choosing not to include
it is a choice for the app developer, as long as they know the
consequences.

On Sun, Feb 19, 2012 at 12:57, Taylor Otwell
<reply@reply.github.com>
wrote:
>
> I think it might be nice to have a "namespace" option in bundle config. That way we don't have to do all that guessing in auto-loader. I'll add it.
>
> On Feb 19, 2012, at 5:v28 AM, Phill Sparks<reply@reply.github.com> wrote:
>
> > Why?  I liked this feature!  Now you have to start eloquent (and others) before you can use them - I'd rather not ('auto' => true) these - that's not the point.  And having to Bundle::start('eloquent') in every model just isn't elegant.
> >
> > Do you have a suggestion for how to register bundle namespaces?
> >
thx alot. not need more this logging, all problems determinated.
english word "anal" and russian word "жопа" - not synonyms...
In linux works perfect.
8-[ ] какой .NET? чистый x64 машинный код без всякого нета. 
если неверится - можно во внутрь-то DLL-ки посмотреть на импорт функций...

а ASM код - FastMM.pas =)
там два варианта кода - Pascal и ASM. Паскаль для наглядности в основном (т.е. показывает то, что написано на асм), ну и для отладочной байды. сами функции работы с памятью написаны на асме и хорошенько так оптимизированы...

Сергей, всё чётко работает, неверишь - поставь сам и проверь...
I'd get it added to the docs if I knew what it was... let me know if you find a reference to the discussion.
I just saw this! Thank you very much :) I'm going to include it with the version 2 of the grammar.
Oops! My bad, when merging in antont's pull request.
Thanks! This was discussed on the mailing list w.r.t. problem in Python(x,y) on Windows, but as it did not resolved the Mayavi issue completely, I waited for more info and finally forgot about that. So thanks for the reminder and patch!
Нельзя съедать InterruptedException!! http://www.ibm.com/developerworks/ru/library/j-jtp05236/index.html посмотри главу "Не поглощайте прерывания!"
Why commented out ?
Вообще-то тут файлы по алфавиту отсортированы. Перенеси
Check my additional comment on the pull request #471

This has already been discussed before (I did not see it)... no need to check empty or false, this is a non valid value.
hi. the changes to generators.py in this commit - are they intentional or did you accidentally re-commit an older version ?
At least in a microbenchmark where I locked/unlocked in a loop, yes.  lock()/unlock() was faster than synchronized(mutex).
Done. I also added the ability to specify the internal cypher used by Fortuna.
-Wextra is not supported by gcc 3.x and causes a compilation abort
Using prop instead would make Chosen require jQuery 1.6 -- I'm all for maintaining backwards compatibility if it can be done simply like this.
but then there's someone called foo~ and you've just colliding and you have to check and go into a loop and i've lost interest
Add a test here so we don't regress.
e.g. only install debug if(NOT MSVC_IDE) or if(NOT CMAKE_BUILD_TYPE)?
for bonus you might check both states, hidden true and hidden false. to cover the failing case that display is === 'none' for both which might happen if there is an issue getting the current style (which we try to avoid with the fake body).
That would be very nice. I appreciate.
oh nice, it's below :P
Changes done Thanks.
This looks interesting. Did you get the inspiration from Andri Sydorchuck's boost-sweepline code? He is doing something very similar at:
https://svn.boost.org/svn/boost/sandbox/SOC/2010/sweepline/

I think we need a way to estimate the quality of a Solution. In vertex_positioner I am now using simply the difference in distance to the three defining sites of a vertex. I'm not sure if this is good enough. The quality number could be stored in each vertex, so that statistics on the quality of all vertices could be generated. We could then run all examples&tests with different solvers, or different implementations of the same solver, and compare the overall quality of the diagram. Using high-precision number-types is more accurate but also slower. So there is a trade-off between accuracy and speed.

If you could re-name these solvers so they are distinct from the current ones, then I could merge it into master, and they could be turned on/off by #ifdef's or some other mechanism. OR, it could even be possible for the user to select a set of solvers at run-time?

The real show-stopper right now is a parallel line-segment case for LLLSolver. It happens for "-" (minus-sign) or "+" (plus sign) with certain fonts from truetype-tracer. When two line-segments are parallel the offset-distance (t) is not a good parameter at all for defining points along the bisector edge.

AW
Moved user_resource to be removed last so that other resources can be safely deleted
neat trick: `colspan="0"` will ensure that the `colspan` is always equal to the # of columns
Take a look at http://help.github.com/mac-set-up-git/ so you can make your name come up in the commits. Right now its "AppDev University"

$ git config --global user.name "Firstname Lastname"
$ git config --global user.email "your_email@youremail.com"
I surrounded the mergeRequests_fn call with a check for codebases to quarantee only the most basic cases (as discussed on February 22). 
With this commit a user can ommit the check by writing an own mergeRequests_fn. This may result in a merged request with sourcestamps that origin from more than one sourcestampsets, that is not tested yet.
а чем-то гарантируется размер шрифта? Может стоит сделать toLower ?
why khtml? o.O
I'm working on all of the remaining elements now.  If you want to join in on the fun just let me know.
...vilket @torandi och jag kom fram till på irc efter att jag ställt (och fått svar på) denna fråga: http://stackoverflow.com/questions/8711055/ruby-negative-zero-float
This will break when iOS 10 is released ;)
вот эти строчки я хз откуда.
This shouldn't be in the commit
"Select the lowest platform with which your application is compatible" > Actually we do classloading tricks so that new phones can benefit from new features, and those new features are not loaded on old phones.
So you can safely put target=android-15 and it will still work on 1.5
If you could provide a test for this change that would be awesome.
for the most part we are using ``null === $this->path``
i just did a grep over the whole code and there is more instances of dirname calls. in ObjectManager there is sometimes  ``$parentPath = strtr(dirname($from), '\\', '/');``

i wonder if we should not abstract this into some helper or something and forbid using dirname in any operation.
thanks mate !
Merge then ;)

On Wednesday, February 29, 2012, Martin Luessi wrote:

> looks good
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/agramfort/mne-python/commit/37d9fb4313d3fd5c237f70feb6023dcd2de50b08#commitcomment-1027050
>
Dummy line
could be one line, not big deal... `return isSuccess(response.status) ? resp : $q.reject(resp);`
Oh right.  In that case though I think there's the reverse problem where a constant stream of stat updates will make the timer expire..
@rsudev Yes, must be StrategyFlag.PARSE_TILES
darn - forgot to turn off auto-strip of trailing whitespace in my editor. please ignore spurious whitespace changes
你不是用践兔了么，那咋是Debian
This part of the code isn't doing much that's terribly useful, it's just to let us use the pretty syntax to include the components, but it does also take care of some dependencies.

The idea is that when you do `Grisaille.core << :model`, it will attempt to load a file using the pattern declared a few lines down in this file (i.e. not visible in this excerpt) which is `"lib/grisaille/%s" % :model`, but if that file isn't there, you get a `LoadError`. We rescue that and instead guess that the path was referring to a directory and try and load `"lib/grisaille/%s/_core" % :model`. So that seems really convoluted, but it just means that when you want to use the models provided by grisaille, you say `Grisaille.core << :model`, and it will load the core models library. If you look in the file `"lib/grisaille/model/_core.rb"`, you'll see that the actual models are all declare with ruby's `autoload` keyword, which means that the file containing them will be loaded the first time they're mentioned.

You can track the loading of files by looking at the events fired (and logged) as this happens. So for example, try:

    $ irb -r./lib/grisaille
    > Grisaille.core
        # => []
    > Grisaille.core << :model
        # (two events fired)
        # => [:abstract_helpers, :abstract_domain_model]
    > Grisaille.models
        # => []
    > Grisaile::DomainModel::Artist
        # (1 event fired)
    > Grisaille.models
        # => [:artist]

(The bits in comments are what IRB should return after each command).

As for predefined helpers, the only ones used in this excerpt are:

  * `require_relative` - a keyword to require a file relative to the current file, with an implicit `.rb` file extension
  * `File.expand_path` - expands out `../` and friends in the file path, relative to the second argument
  * `__FILE__` - the path of the current ruby file
  * `File.join` - sticks path fragments together, adding slashes if needed
  * `File.directory?` - is the argument a directory (as opposed to a file)?
  * The `a % b` syntax is to substitute the second argument (`b`) into the first string (`a`) at the places where there are markers (like `%s`)

That help?
I don't think a whitespace change added multitouch?
I'm not sure what to do here. I believe that preg_file2() produces better results than preg_file() - I have not replaced preg_file() because it would mean rewriting many of the checks. Maybe a rewrite of all checks would make sense down the road?
You weren't supposed to look at that commit :). It was just convenient.
Tried it manually again here: af3bd79de5
This is somewhat hacky, you can see that the method actually does more than just describe the object, it borderlines on getting into view territory and should probably use an AnagramListFormatter or similar.
Really thanks for ALL your work. The best!
No, most summoned creatures SHOULD actually reward experience..
this fix for what?
for billion damage of deep wound?
thanks for your reply
:) sounds great
Same question about the `... 1 = (select 1 ...) ...`
xˡ @N͵ĕҿǲˡ

ĶfǺ܏Ť

ðȷ÷waˣһDɺw

FһЩDڵvDܛ.........]пs֮N혵ܛ T_T

--
Juanito



2012/1/23 huangz1990 <
reply@reply.github.com
>

> ˼ԸĽĵط fork 
>
> ˣͷ汾 Juan ڷĻά᲻鷳
>
> ڼ淢һ bug ˣڷ滹ٸһΣԵĻǲǿһһ汾֮ٷת߼ת壿
>
> ӦöȽ٣Ҳظ޸ģJuan Լһ°ɡ
>
> 
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/JuanitoFatas/acl-chinese/commit/c6da431b47da5fdd21bab13c809c729c28d4f0f4#commitcomment-891210
>
I think the shared directory should be moved to /refinery/news/shared/ rather than being inside /items

What do you think @gogogarrett?
Looking like it is starting to take shape. Next step is to control turret both pitch and rotation. We must get the distance from the camera to the target!
<a href="http://www.keepautomation.com/"> KEEPAUTOMATION Free Online Code Reader</a>
> are you sure that the `delayed` will not cause `rng` to be cloned in the same state on each worker hence defeating the goal of random sampling?

If you look closely, you'll see that rng is not passed to the workers,
but sampled in the main loop. Thus this is not a problem.
Sorry @main-- I really needed to get these changes out :( I'll spend some time this weekend tweaking my IntelliJ auto-format.
Nice - didn't know about `:button`  and `:submit`.
Dude, your code is totally flawed. WTF
You need to make provision for the provision.shit
Get rid of storage pool. Fetch hosts by cluster id.
Sorry I don't understand and I don't see the rotation ??

Help!
Changed remote name and fetch data.
well, there is 2 things to consider here:

- currently, the interface does not enforce the getId() method so it would be a BC break. And it could be an issue with other bundles using an interface with a getId method as it would make them incompatible (isn't OrnicarMessageBundle doing this ?)
- adding the method could improve performances a bit here as retrieving the object by primary key is optimized in doctrine ORM (and probably in the ODM too)

And this will not solve the ACL issue: the security component always uses the username for the ACL. There is no way to change this if the username is not immutable. @schmittjoh wouldn't it make sense to add an interface to get the acl identifier ? If the user class does not implement it, the acl system would keep using the username (so fully BC) but the dev would be able to use another field as immutable identifier (typically the primary key)
Haha, yeah, the test was to see if it'd autoconvert my line endings. :)
not the solution I had in mind, but I like that it only requires one call to `Isk::img_exists` so we don't have to memoize to reduce network traffic.
BEst is to request this on the SB forum : http://sickbeard.com/forums/viewtopic.php?f=4&t=2227
I tried adding the "." handler as suggested, but this get's never gets called.
Looks like I'm missing something obvious.

An example would be appreciated. Would be even better if you could cover the "rm" event as well. Sounds like a good idea to clean up in order to avoid zombies.

Rainer
Ok thanks, I'll try it on my build then and will report if it works ;)
Not sure I understand this why are you checking for len greater than 0?  Also would this be better in a try/except to catch a KeyError?
It can be done in the caller or inside here.  It depends on how we define options['wait'].  I consider options['wait'] means waiting until the VM is (created but poweredOff) or (poweredOn and IP address is ready). So I add the logic inside.  Maybe in some case the VM is not configed as DHCP and IP address is not set , so checking ip address is not meanful here. I'm ok to remove it :)
My trick worked. Took a `us-east-1` ELB hostname, pasted into the R53 Alias Target box on the web console, edited the region to `ap-southeast-2`, and it showed me this:

```
Alias Hosted Zone ID: Z2999QAZ9SRTIC
```
I think this error message should explicitly name the installation flag that the user needs to remove to fix the problem (`--user`).
For the sake of typing (and bytes) can we go with `yui3-rg` as the classname?

Maybe there was a discussion about this that I messed…
I do not want to store this information. It makes moving blogs between containers needlessly difficult.
It's used for the tests (OAuth20AccessTokenControllerTests class here). I set a "mockito logger", to *verify* after, that the right log has been generated.
Again, I felt it served some beneficial documentation purpose to tell the size of assertion handles.
Once again, it's certainly useless. I'll check...
git shortlog -s -- debian
     9  Francis Russell
     4  Graham Markall

So I think this is a lie.
Wouldn't it be better to wait for a specific color?  That way it'll compensate for any kind of lag?
This seems like duplicated code, collapse and reuse?
So that if someone in a constructor calls it, it will get a value. but that doesn't really make sense, cause if that would work OK I might as well have done it as a val.

But, I'm rewriting the activation right now in a separate ticket, and came to the conclusion that it is better to not expose this internal endpoint and sendprocessor. If the user wants an endpoint, they can do camelContext.getEndpoint(endpointUri) and 
if they want a sendProcessor they can get one doing just: new SendProcessor(endpoint) with that created endpoint.

I don't know why I didn't think of that earlier, maybe cause I'm rewriting that piece now.  

so in this case (and this version) I will just make them private.
Isn't it weird that it works with java.lang.Integer but not RichInt?
What does the breakOut do here?

    val routees = Vector.fill(nrOfInstances)(context.actorOf(routeeProps))
Shouldn't this be the FQCN?
I should probably add that I verified that within our code-base copy() is only ever called with a new name argument
BADASS!
correct, that makes a good comment :-)
This is used by the tasks thing, so I think it will be a leftover and get deleted.
harder to read
Sorry - I don't understand what that means
long lines are evil ;p and there are several about pep8 is your friend and mine.
if you see the  base class, it defines it as classmethod
I'm actually work on this issue and will probably have it for on upstream pull request really soon.  In the more spiky parts of the action_* work, I found more loading issues and was running in circles on it a bit, but I think I mostly figured it out and I just want to integrate the ideas directly off develop.

That's the branch I would add some of the other small changes here too.  I'm trying to only add things upstream that will reduce the difference and merge easily.
Since I understand the issues are there regardless (both with and without this PR) shouldn't we just merge this PR so it at least solves the other issues?
please leave some space above comments.
1) I could use some tests here to ensure that the various input formats (e.g. with and without trailing slashes) are split correctly.  
2) Not including a hash-destination (mypath/*.py#mydestination) may not be a useful input case (since the package could not then be referenced in code, which is kind of the point).  May want to pull this directly from the input directory anyway.
3) Is a criteria other than *.py ever useful?
I would rather have a separate function `calculate_opt_priority()`. Doing two "passes" over the list of option dicts isn't any more expensive and it's easier to explain.
The way it currently works is that in POUserFunc for example, it blindly asks for a SchemaTupleFactory. If one is available, then great. If not, then it returns null and the client code will use a TupleFactory instead.
Instead of storing to `code` and returning at the end, you could just `return` from within each `if`/`elif`, would save a couple of lines.
"Will you allow the Forge team to receive anonymous usage statistics for this instance of JBoss Forge?"
Or do you offer to move ace_ prefix out of setStyle method ?
I change "should not" to "MUST NOT" (I'll keep also the uppercase).
Why not use Math.exp instead of Mat.pow(E, ...)?
I take it you deleted this because this logging is no longer useful? That's fine, just wanted to make sure it was intentional.
plan is to create TaskList struct that holds an array of todays task plus its size
reset yes, to zero no.
hello sorry to bother again, i need some C advice.

it works all quite good now, and i found out what you mean by " you'll probably need some C code that negotiates between C and C#". its the t_atom type which is quite alien for another language. so i tried to get rid of it by simply converting the atom array to a string and passing that to C#. so i wrote a method but it seems to have memory management problem, can you have a quick look at it and propose a better way? or maybe PD already has a method for that?

the code i have so far is here: http://pastebin.com/G827KVen
@cournape: Re; LD_PRELOAD: You really think it would be easier to add a new shared library (multiarray_alloc.so or something) containing just these functions and then arrange for it to be dynamically linked to multiarray.so on all supported platforms? IIUC that would require at least one of: screwing around with RPATH, installing the library into a systemwide shared directory, or using dlsym directly. And it still wouldn't work on Win32...

@thouis: I may not be understanding what you mean about complexity. IIUC, with the callback case, the data collection code looks like:

```
_heap_log = []
def record_heap_action(action, inptr, outptr, size):
  _heap_log.append((action, inptr, outptr, size, traceback.extract_stack()))
numpy.something.set_malloc_callback(record_heap_action)
```

With the log version, it's something like
```
_heap_log = []
def poll_heap_log():
  log = np.something.alloc_log_fetch()
  if log:
   stack = traceback.extract_stack()
   for entry in log:
     _heap_log.append(log + (stack,))
```
plus a bunch of arcane gunk to set up a trace function; I'd have to stare at the sys.settrace docs to work out how that would look. Plus this adds much more overhead (assuming most lines of code are not actually allocating any numpy arrays). So the callback version is significantly less complex on the user side, IMHO.

(I'm assuming that people will use some front-end tool that then produces reports based on these logs, or dumps kcachegrind-compatible output files, etc.)

There's still the question about re-entering Python from malloc/free call-sites, but really this *should* work. Every DECREF is already a potential Python re-entry point that might execute arbitrary code.
No danger of overflow? The product of the `dims` should fit in npy_intp if they were the product of one of the array creation functions.
Все, вопросов нет.
Have just tested with some Japanese characters, and didn't see any problems with encoding.
otherwise i will need to pass nil as the first option. does not seem semantic (and is not pretty).
as per this patch, we can pass nil or :existing
This is likely change the before_filter options hash, I'd rather not delete the options but create a dup hash without the options here.
I believe we should raise an error here. Otherwise people will get constant  ActionDispatch::Session::ActiveRecordStore not found error. Tell them it was moved to a plugin and that they should set `config.session_store = ActiveRecord::SessionStore`.
Why not remove it completely instead of commenting out?
Good news, that's already warning-free (not that it exactly compiles as given) since I've been tweaking this all day.  My test case version (2 warnings trunk, 0 me) is

    trait Y
    trait Z extends Y
    class X[+A <: Y]

    object Test {
      def f1(x: X[_ <: Y]) = x match {
        case _: X[Any] => // looks a little funny; `Any` is outside the bounds for `A`
      }
      def f2(x: X[_ <: Y]) = x match {
        case _: X[Y]   => // looks better, let's allow this (too)
      }
    }

More test cases gladly received, especially involving higher kinds and contravariance.
Oops. So that's how these variable things work, I was wondering how people avoided hardcoding CC all over the place. Thanks for the input.
typo --> Despite
this should be added to the properties of the map. it denotes the coordinate system used and is necessary for wcs anyway.
Looks simple enough and should work in Python 3.x as well:

http://docs.python.org/dev/library/tokenize.html#examples

The only difference in there is the use of BytesIO instead of StringIO.
Please use a while loop if it's just walking through the indices one by one.
``.real`` to ``.integer``
I have corrected this mistake.
So this changes the output? What is the motivation here?
parens preferred over `\`
`(int)$day===7` would be just the same as `$day==7` ;) I think last one is better.
Specify issue number.
public function disableMobileTheme()
    {
        CookieUtil::setCookie('zikulaMobileTheme', '2', time()+3600*24*365, '/');

        return $this->redirect(System::getHomepageUrl());
    }

But why not just redirect to the current URL? It might be better for the user-experience.
Have we talked about this? Seems like a reasonable if obtuse way to just get text output of my violations if I don't care about tracking them in a database.
Done, thanks guys!
Remove the whitespace  before and inside the brackets, please.
This code is wrong. It breaks the possibility to use closures or functions as controller whereas the parent class allows it
Sorry James M. Greene,

I didn't know force update removes comments associated with code segment. The
following is your original comment.

"That should definitely be window.location.host instead of
window.location.hostname. Also, I'm not sure if this works when using the
triple-slash short-form for File URIs on the localhost (i.e.
file://localhost/blah is equivalent to file:///blah) as I don't recall offhand
what the Location object will return for these properties when using the File
URI scheme...."

I just replaced hostname with host, you are right. :)

On the tripple-slash issue, I tried file:///* on Chrome, Firefox, IE7 and IE8.
They all work. I think it working. Maybe I don't understand your question?
I see where you've been shortening variable, names, but at least in cases where it isn't an issue of line length can we leave the more descriptive name (like leaving ````questions```` instead of using ````qs)?
On second thought, cancel this remark - because you are forced to do this due to the API change from a String to an Endpoint - so I get your motivation.
It is important to make sure that "register" only gets called once per process, but registerSubType() should be called once per jsonMapper.  That or we need to change the semantics of the register() method to expect multiple calls and only do it once.
are 10 what?
not sure what the literalized ``count`` refers to.
I can't take this part of the patch, very few systems have a "python2" binary including my laptop :)
1. That does not make sense. When someone reads "options" the first think I believe that person would think is that that every parameter is optional. That is not the case, all parameters are required. Not delivering all the parameters means that it will not function properly.
```<imkingdavid> erikfrerejean: if there was no blank line before EOF, it would have a "No new line..." message
<imkingdavid> my editor automatically adds new lines at the end of files
<imkingdavid> or well, it makes sure that there is always a new line at the end
<erikfrerejean> imkingdavid: they don't show up
<erikfrerejean> So might be a github bug, don't know. I look at it and they aren't there ;)
<imkingdavid> erikfrerejean: https://github.com/phpbb/phpbb3/blob/develop/phpBB/cron.php there doesn't appear to be a new line at the end, but one is there
<imkingdavid> i think github just doesn't show the last line of a file if it is blank
<imkingdavid> and then it has a "no new line..." message when there is no blank line there
<erikfrerejean> Don't know that, but even the "raw" one doesn't show it https://raw.github.com/phpbb/phpbb3/develop/phpBB/cron.php
<erikfrerejean> So that wouldn't make sense :/
<imkingdavid> well, it's an issue with github. check bantu's PR https://github.com/phpbb/phpbb3/pull/834/files none of his files display a blank line at the end```
Keep `t.forum_id = ' . $forum_id . '` as the first condition so you can make it so the "AND" is at the beginning of `$limit_access_check`
It's not there... I went and check my backups... It's in the backup... Something happened...
Anyway, new commit with those comments.
@riadhchtara You misunderstood what @Senky meant. He was not referring to assigning the string 'backtopost' to a variable, but rather the return of request_var(). Because you use request_var() twice, it is better to assign its value to a variable, such as $backtopost, and then use that instead of the function call in the other two locations.

Second, because this is going into the 3.1, you should be using the phpbb_request ($request) object instead of the request_var() function.

Finally, $backtopost and $back_to_post are IMO named too similarly so people might get confused when trying to follow the code. Perhaps choosing a different variable name for one of them, or even doing the logic ternary-style on the U_APPROVE_ACTION line.

For instance, first, do:
`$backtopost = $request->variable('backtopost', 0);`
And then,
`'U_ACTION_APPROVE' => append_sid("url", "..." . ($backtopost ? '&amp;backtopost=1')),` (I abbreviated most of that to save space).
And finally
`meta_refresh(3, $backtopost ? $post_url : $redirect);`

Does that make sense?
The luvit dns module does have isIP functions to figure out family type.
Spațiu după virgulă, please.
{'form': form} (spațiu după : ) și sparge-o pe două linii, te rog.
I can't predict any circumstances when neither working fb config nor fallback one will be chosen. Still that doesn't mean this would never happen and shouldn't be handled.
Yes on having this function, but I think the name should be different.

I was borrowing from Ruby's ActiveRecord terminology here, where `find` actually finds only one record by the primary key, `find_by_*` means find one record by a certain column, `where` means find all that match a certain criteria (given in a hash), etc etc

This would map most closely to `where` I think. I'm also totally open to not following that convention, but just think the names of these in general should be "nice".
Is this emitting multiple messages so that both explicitly namespaced events and the generic handlers can capture it?
Это тоже не правильно по логике. Объект класса UnitErrors сожержит информацию о полях, которые были неверно указаны для баннера. Это по идее должно было использваться при отображении какие поля неправильно указаны в форме. Соответсвенно использовать $this как раз правильно
Тут нужно убрать DEFINER=`root`@`localhost`. Тогда запрос будет выполняться для текущего пользователя.
> Ещё нужно будет дописать событие для генерации кеша статистики и работу с ограничениями по кликам.

) Её тоже событием в бд раз в сутки. Сейчас не могу точно написать, там нужно подумать как сделать,чтобы к существубщей записи только  прибавлять статистику с момента поледнего обновления, а для несуществующей считать всю что есть.
Use _string.Concat_ please
What is CURRENT? and given that it is set true one line above this, why do you have to check 
```js
if (CURRENT)
```
Ah, okay. I think to get one-to-one working, we also have to put the <one-to-one...> tag in Drug, but we don't want to do that and modify core OpenMRS. So, for now, keep many-to-one, and add a comment the it should be one-to-one but don't want to modify core drug mapping files.
Hm, my comment isn't coming, let me try again.....
\< property name="drug" unique="true' /\>
NB.  I had to add this because my tests were going nuts. The web of imports ended up importing matplotlib before iris.tests could set the backend to "Agg".

Please elaborate on your cartopy alternative.
A result of the now sorted attributes in *_xml_id()*.
Master:
```
$  py test_merge.py TestColpex
('altitude', 'm', (('positive', 'up'),))
('forecast_period', 'hours', ())
('grid_latitude', 'degrees', ())
('grid_longitude', 'degrees', ())
(0, 'm', (('positive', 'up'),))
('model_level_number', '1', (('positive', 'up'),))
(0, '1', ())
('surface_altitude', 'm', (('source', 'Data from Met Office Unified Model 7.04'), ('STASH', STASH(model=1, section=0, item=33))))
('time', 'hours since 1970-01-01 00:00:00', ())
('forecast_period', 'hours', ())
('grid_latitude', 'degrees', ())
('grid_longitude', 'degrees', ())
('time', 'hours since 1970-01-01 00:00:00', ())
```
Branch:
```
$  py test_merge.py TestColpex                            
('altitude', 'm', (('positive', 'up'),))
('forecast_period', 'hours', ())
('grid_latitude', 'degrees', ())
('grid_longitude', 'degrees', ())
(0, 'm', (('positive', 'up'),))
('model_level_number', '1', (('positive', 'up'),))
(0, '1', ())
('surface_altitude', 'm', (('STASH', STASH(model=1, section=0, item=33)), ('source', 'Data from Met Office Unified Model 7.04')))
('time', 'hours since 1970-01-01 00:00:00', ())
('forecast_period', 'hours', ())
('grid_latitude', 'degrees', ())
('grid_longitude', 'degrees', ())
('time', 'hours since 1970-01-01 00:00:00', ())
No handlers could be found for logger "tests"
```
This was code while testing. I didn't want to think about garbage collection until necessary. This has been fixed.
Add a comment like:
/// Only set for the special case of an aggregate function result used in an "ORDER BY" clause.
/// This TupleValueExpression represents the corresponding "column" in the aggregate's generated output TEMP table.
Good point. I neglected to revisit AdHocSP after making the hard changes elsewhere.
Please send patch to vivagraph author.
@fahad19 afaik, we were discussing of changing Config/settings.yml to Config/settings.json,  and I wanted to pretty format the saved file.

In any case, should we get this in, I think we would need to have it as a public method somewhere.
No, 1.23 million is correct for English. See "Plurals of numbers"  in http://en.wikipedia.org/wiki/English_plural: 'When modified by a number, the plural is not inflected, that is, has no -s added. Hence one hundred, two million, four score, etc."
What's with the hashes (#) at the end?
Not sure where this import came from... I've just killed it.
please use ``{@inheritDoc}`` for all methods already documented in the parent class or the interface (eventually adding something if there is a need for an explanation about the way it is implemented). It makes it easier to maintain than duplicating the phpdoc (we don't need to fix typos in 10 different places)
Christian you reverted the code back to the pluginAPI but this is not correct.  It is in the dotmarketing-config.properties now and should be retrieved that way Config.getStringProperty("escalation.job.java.roleToEscale", "com.dotcms.escalation")  the second String parameter is the default 

Please fix this
I think we don't need pry in this PR. :heart:
A more interesting case is what happens if you run `configure`, *then* edit some-pkg.cabal and then run `cabal build`. It may be necessary to add a check for this so that `configure` is re-run automatically.
Можно без квадратных скобочек. http://docs.python.org/tutorial/classes.html#generator-expressions
Без них код чище будет.
aca el espacio
GitHub considers this diff outdated. Replied in the main thread.
This does not make sense. See above. Also, I don't think we need to explain what a density function is by using a term like "relative frequency". Right here, that is arguably more confusing than helpful. Therefore, it should be enough to write:

    @return \f$ f(x) \f$ where \f$ f \f$ is the probability density function of a normally
        distributed random variable with mean \f$ \mu \f$ and variance \f$ \sigma^2 \f$
Remove "testdb=# "
Isn't there possibility that either of left/right has no sample_id?  In that case, do we still need to check with success()?
Since i am not familiar with the SRF details, my question is:I find all the functions, such as pg_ls_dir and array_agg, with returning setof in pg_proc.sql didn't use PG_TRY for that. Those two functions you mentioned are very straightforward,  when they will raise error. Thank.
At some point I ran into a subtle bug where ``_k`` was an iterable type, then found a recommendation to always specify the tuple of arguments.  I cannot find the reference now.
This function should not be inlined.  Implement it in the cpp file.
Space around operators are good for you
You should add tests for IE and Chrome and other user agents. Less because we want to support them and more because if this code is used more generally, then other user agents shouldn't cause it to throw errors.
`fjord.search.index` shouldn't be importing anything in fjord and these don't require anything in fjord, so I'm not sure where the circular import would be. What was the circular import problem?
Don't remove this in the same patch please. It's a leftover on purpose but I'd rather completely remove it from every file than partially.
Ahahahahah. Now you know why I thought it would 'probably' work. ;)

>  dalli needs to work independent of Rails


Roger. I thought this was the case, but I glanced at the code and thought that other things made it a hard dep...

Give me a moment.
Got it, thanks! Have a fix in the latest commit which simplifies the formula quite a lot.
The changelog may look more than what has actaully changed, since I basically nested the `if` blocks such that we try harder all the options. Before it could have been that `xcode-select` points to an empty dir and then later all breakes. I have tested theses changes in all possible conditions with all possible settings of `xcode-select` I could come up with. I really want to get it right. If you read it _not_ in the diff, it also reads more clearly.
J'ai rien dit, on est pas sur le serveur. Cependant, on a plus de perso dans cette classe mais seulement un joueur qui contient donc, le perso et la position. Pas besoin de plus du coup.
Yeah, I see where you're coming from. I guess I mean that we should be able to remove this now that the bug has been fixed?
I have big plans for this guy! This is where we'll have real APIs for configuration, including connection pools, response caching, cookie management, etc. Currently all of that stuff relies on statics.
This algorithm is probably not correct. It should in fact return the past segment following //gpii.org/ - and not the entire URL minus the final segment. Recall that we anticipate terms of the form "  "http:/gpii.org/ontology/A4A-2.0/display/screenEnhancement/genericFontFace" - see our piratePad discussion at http://piratepad.net/NhEWxhwGyT - in this case, the name of the ontology should be considered "A4A-2.0" - as always, the "semantic web" style other URL material is just fluff
This looks like it adds a different bug. This is insufficient to remove deleted security groups from the cache. That's the full behavior of CachedMap.put(). If someone clicks on a stale security group link, this code will fail to correct the cache, so then that security group will remain on several other screens for several minutes.
`@_execWithOptions`?
I think this can be simplified a lot: there's no need to add/remove listeners depending on focus.  Here's what I suggest:
* Remove everything from the body except the `element.bind("keydown", handler);` line
* Rename this method to bindKeyHandler()
* Rename the handlers methods that use this accordingly.  For example: handleSelectionFocus() -> handleSelectionKeydown()
Same note on `width()` here--can eliminate the parseInt..
Should add a comment explaining this adjustment
I ended up keeping the context menu definition for project, but I added some logic to open() so that the context menu doesn't appear when there are no items. 

BTW, I changed the working set so that right click selects, but the project tree doesn't do this yet (I guess we can treat this as a bug). This is likely necessary when there are menu items for the project area.

Ty

-----Original Message-----
From: Randy Edmunds [mailto:reply@reply.github.com] 
Sent: Wednesday, June 13, 2012 7:57 PM
To: Ty Voliter
Subject: Re: [brackets] Context Menu API, partial implementation (#1012)

> +                    clickedSel =  editor.coordsWithinRange(pos, sel.start, sel.end);
> +                }
> +
> +                if (!clickedSel) {
> +                    editor.selectWordAt(pos);
> +                }
> +                editor_cmenu.open(e);
> +            }
> +        });
> +
> +
> +        $("#projects").mousedown(function (e) {
> +            if (e.which === 3) {
> +                project_cmenu.open(e);
> +            }
> +        });

There are currently no project panel context menu commands that operate on the current selection, so we don't need to do this now.

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/1012/files#r982102
I have a suggestion for simplifying this dialog a bit. As implemented here, when searching all files, the dialog looks like:

> Find in Files: [__________] in all of brackets (use /re/ syntax for regexp search)

When doing a scoped search, the dialog looks like:

> Find in Files: [__________] in brackets/src (use /re/ syntax for regexp search)

As an alternative, how about adding the scope to the prompt label instead. Global searches would look like:

> Find in project: \[___________\]  (use /re/ syntax for regexp search)

Scoped searches would look like:

> Find in **src/brackets**: \[_________\]  (use /re/ syntax for regexp search)

Bolding the scope folder/file will help draw attention to the fact that this is a scoped search.
If you have two of the same selector in a file, I think this would only let you select the first one.  Seems like some work to fix, though, so if you want to spin this off as a bug I'm ok with that (just let me know when you've filed it).
We don't; should be cleaned up maybe.
Sure, I'll do this tomorrow.
does this need to be public ?
Please move the javascript into assets. I've already responded to a very similar diff here https://github.com/gaye/rideshare/pull/17 with that comment.
Kiepska nazwa klasy
Might be tidier as a ternary expression
Use full path imports.
Have you written tests for this? What about code coverage tools?
You should trim whitespace before you assume first position is the minus.
Adjust this so it aligns with the input
This looks wrong, since the DB deals with base-10 IDs.
L'indentation...je chipote :-)
ok, floating point precision changes can certainly lead to a difference of the old and new results, even with the same seeds, ok then!
You can probably get away with only allocating it once for the whole loop :)
You're right. I'll pull it out.
Can you please remove that trailing whitespace?
Same as above with regards to keeping view layer and style layer separated.
I would still show it in the list if not readable, just don't link to it.
Change this to 120.0.0.1 for security reasons!