labofsahil/github-event-dataset-2012 huggingface dataset:
Generate transforms experimental

Changes required for the transform to be generated as part of the kernel. Tested on the buildbot and passing all tests.


A high-level overview of the changes made:

* The vector field Coordinate is added to the state so that it is
  extracted from the Fluidity state in the generated code.
* Factory functions for the Jacobian, its determinant and its
  inverse have been added - these generate the UFL representation
  of J, detJ and invJ.
* A string preprocessing pipeline stage has been added that 
  post-multiplies each form by detJ. This incorporates the 
  determinant into the code that is generated automatically.
  grad and div are also re-defined so that they give the code to 
  perform the computation of the transformed derivative by 
  multiplication of the values at the coordinates and the inverse
  of the Jacobian.
* Femtools is now queried for quadrature and shape information. 
  This is used to declare automatic arrays inside kernels for the
  quadrature weights and basis function values.
* The code generation module has support for building 
  multi-dimensional array initialisers for the basis functions and
  quadrature weights.
* The transform_to_physical kernels are removed. Instead of 
  passing detwei and basis functions to local assembly kernels, 
  the kernel receives the Coordinate field as a Coefficient 
  instead. The Jacobian is calculated in the quadrature loop as a
  function of the coordinates and derivatives of the basis 
  functions in the standard way.
* The KernelParameterGenerator has been removed completely as it 
  is now superfluous.
* A partitioner has been added to the pipeline that finds the 
  maximal subtrees of a UFL expression tree that contain the same
  indices and is rooted by a single node. It denotes these by 
  inserting SubExpr nodes at these points.
* The code generation in the ExpressionBuilder class has been 
  changed so that it generates an expression for each SubExpr
  partition, and accumulates its value into a temporary variable.
  The Local Tensor expression is computed from these subexpression
  contributions at the end of the loop body.
* Support for generating code for representing ListTensors has 
  been added - these are represented by a name that is a function
  of the indices by which they are subscripted.
Add 2.10/2.11 and metagallery support
LPS-25836

Applying the change to web form portlet
Adding two more handles to mug.

Just for debugging and testing purposes. No biggy, I suppose.
Fixed issue with borders around indeterminate progress bars.

A *small* bug appears when a border is applied to the progress bar. If the value of the progress bar is set to zero, the progress bar increases in width slightly, until the value is changed to a non-zero value. 

NOTE that I didn't change progress-polyfill.min.js, since I wasn't sure what you used to minify it (I used Google Closure, and it was very different, so I thought I'd ask first).

You can see what I mean here:

http://www.useragentman.com/tmp/progressPolyfill/test1.html

In order to fix this issue, I changed line 161 from:

if(progress.position !== -1) {
 progress.style.paddingRight = progress.offsetWidth *
(1-progress.position) + 'px';
}

to this:

if(progress.position !== -1) {
 progress.style.paddingRight = progress.clientWidth *
(1-progress.position) + 'px';
}

(Note the change from offsetWidth to clientWidth).  When this is
applied, the jump in width doesn't happen anymore, as seen here:

http://www.useragentman.com/tmp/progressPolyfill/test1-fixed.html
changed sort method of "Aktuell im Kino" and "Demnaechst im Kino"

... to most recent.

Before it was sorted by visits.
Fixed issues with last_ip in client tables

Last IP wasn't populating correcting in tables, this fixes that
Improved NuGet creation

Leverage NuGet-functionality to fetch project metadata (version number etc) from the project files and also to sense dependencies through the dependencies already added through nuget.

Build files in buildscripts folder are obsolete except for the build.boo file (but I didn't remove them). I added a new task to build.boo to build the new versions, build.bat nuget2
Changed docs to add info of how to add script to site
Multithreaded background scraping

I think it is enough tested now ;)
Appended an empty option element to the select element that holds the me...

...dia list so that it can actually be selected and the uploaded image will show up
[i18n] > ready for translations in wp-content/languages/edd/edd_etc

Based on pippin's edd_textdomain added this new textdomain option
Misc fixes

A few fixes/improvements:
- remove unnecessary require statements
- in amendResponse(), remove unnecessary self variable
- in amendResponse(), don't do any rendering if there are no additional results
- in amendResponse(), re-open the menu if it is visible with no items
- when amending results in the dropdown, fire the *response* event (jquery ui 1.9 compatibility)
- fix removeFromCache()
- document dependencies
update-tag-value-handler-26

gridaphobe: **approve**

Pick up changes from object browser version of tagValueHandler.js. Updated README.

Fixes #26
Add pbrisbin's Haskell HTML template syntax plugin.

This plugin consists of a set of syntax files for highlighting the various HTML
templating languages in Haskell, for example Hamlet, Cassius, Lucius, and
Julius.
Delta robot motion optimization final

re-requesting for recovery from a major git disaster.
Hotfix/project without translation

Sending to 'develop' this time.
Minor changes in resources
Korean Translation

Hello. darcyclarke.

I made a translation to Korean.
It was very interesting jobs to me.
At the first, I focused to translate.
But some questions is not fit in Korea situation.
If you agree to add questions fitting to Korea situation, I'll do make.
Please look through my suggestion. :)

Best Regards,
Songhun.
add alias option for filter

The callback filter could already be used to filter on fields not directly associated to the admin class. The added alias option allows to also use the default filters and requires less code for basic filters.

Example:

```php
   // CommentAdmin

    protected function configureDatagridFilters(DatagridMapper $mapper)
    {
        // ...
        
        $query = $this->getDatagrid()->getQuery();
        $alias = $query->getRootAlias();

        $query
            ->join("$alias.post", 'p)
            ->join("$alias.category", 'cat')
        ;

        $mapper
            ->add('category', 'doctrine_orm_string', array('alias' => 'cat', 'field_name' => 'name'))
        // ...
        ;
    }
```
Added some few features, like error page, start page.. made some few fixes..
Add getting rid of Win9x support to deprecate.dd

A try to change deprecate.dd according to D-Programming-Language/phobos#406 in spite of my poor English.
Yikes! I know @sdesai isn't the biggest fan of Widget's `UI_EVENTS`. Hopefully we can come up with a better stort for this in 3.6.0.
I'm commenting on this because you asked me to!
For these and other plugin READMEs -- if this is going into the master repo, this instruction isn't needed; also, this misses the plugin version installation part of the process.
>I agree with you about what OnMove is supposed to be. If you'd 
>prefer calling it NotifyMove to follow the pattern in WindowSystem, 
>that'd be fine.
>
>There is no commanded Move() at this time (actually WindowSystem has 
>MoveWindow but it's a nop and is not called anywhere I could find).
>I don't think calling Move() would be appropriate in response to 
>OnMove() as the user/OS already moved the window.

yea, MoveWindow was what I was thinking of for Move()

>The problem here seems to be that different OS don't generate the 
>same events for the same actions. For example, Windows doesn't send 
>a move message when creating a window, only while/after moving it. 
>Maybe there is a way in OSX to "eat" the first move notification 
>after creating a window?

possible, let me looking in the docs as to what it says 'should' 
happen and if I can guarantee to always get the move after create. 
One issue is all this happens in SDL where I have zero control.
Let's use 20 here and update the |var small = Math.abs(diffX) < 20;| to be <=

sorry for the back and forth :/
What about removing container?
This will not be executed at all if assertion is turned off, which is often the case in production.
I believe so, attr_accessible is a really basic thing that should be included in all Rails apps no matter what. If a developer doesn't have some sort of attr_accessible/attr_protected method in their models then they need to read and learn a bit more about Rail.
Why not @volatile?
Doh.

Thanks, that's why code review is so useful :-)
it's fine to change - but do it in master not v0.6 - i don't want any hidden problems to pop up
Perhaps mix it separately based on the amount. Such as `1` mixed with `.5` at `50%` results in `.75`.
Here you expect that A is the the dtype it was expected, but the check at lines 16-19 check only the run time dtype. So you should use the run time dtype here too(or assert it after line 19.
Thanks. Thats solve my problem. Now I can do brew search.
Oh yeah, I'll commit that post haste...
Looks good. Will you also add Scalacheck properties, and update the relevant test cases?
are these error codes something VMware define?  If so, probably worthwhile adding to the javadocs the link to them.  If not, I'm not sure we should be creating them without discussing with the team (and also javadoc'ing)
Should be fixed in ac56d68. Thank you for the report!
Yeah I noticed.. Thanks again for the comment.
yes, i mean x1,x2,y1,y2
Same note as line 318
Probably a bug in 1.0b1. I'll install it on linux if I can't track it down on windows.
`ms` is the shortcut extension for `mustache`.
Just thought I'd say this. I am jamesm-sitegen, Changed my nick :)
Ugh. I know you're right. But... ugh.
good call, this should be a log.warn
Nice `.skipNulls()`! Guava never ceases to impress.
thank you, more scripts will add soon )))
I'd rather either keep the bracing around the else or move the comment in the end of some line (``else`` or ``return -1``).
Though, this isn't a big deal
This commit wont allow core to compile on Linux. Will try to fix and post here!

Anyway, you are doing great here...
Same ( Because of it I can't get warlock spells working like Demonic Circle, it requers fake aura that must be placed in that table..
appease the peoples
server keeps crashing with it, like http://pastebin.com/Kvt40A8C . But i guess its just sth with our server, as noone reported it yet :/ At least i havent seen.
good, but in commit appeared typo :) renew commit please, and retry (this - also partially usable).
nombres de archivos aqui tambien, porfa, todo minusculas, y si es de world, char, o auth correspondencias para cuando añadas nuevamente el sql de las frases y loot.
rendes angol mondat
Hey, I'm not sure if you already knew about the [vim-ipython](https://github.com/ivanov/vim-ipython) that ships with IPython itself, but perhaps it already does what you're looking to do.
sorry typo: "friendly_id slugs should not class with eachother" should be "friendly_id slugs should not clash with eachother" ;)
Here... You'd make this
return opener.top.restoreSession()
>>>>> "Karl" == Karl Palsson <reply@reply.github.com> writes:

    Karl> Uwe, what sort of things could I do to check the stlinkv1 here?
    Karl> try and write a program to SRAM that's greater than 0x1800 or so,
    Karl> see if it still works?  --- Reply to this email directly or view

Well, when the blocksize was greater then 0x1800, the STLINKV2 stalled. Only
unplugging and repluffing helped. I guess some internal buffer overflowed.

I guess you will see similar behaviour...

Bye

-- 
Uwe Bonnes                bon@elektron.ikp.physik.tu-darmstadt.de

Institut fuer Kernphysik  Schlossgartenstrasse 9  64289 Darmstadt
--------- Tel. 06151 162516 -------- Fax. 06151 164321 ----------
This test will fail if the default height and width change. Throw in a height and width for good measure.
omg, javasript has infected my brain. Fixing to ``getattr()``
A global rule discovery module, which calls the various methods and applies the "business logic".

Mandatory Tests
Step 1 - IP -> hostname regex
Step 2 - nmap generic (ex. common open ports)
Step 3 - SNMP generic (ex. sysDerc, location, admin, etc.)

Optional Tests based on the previous results or seed files (ex. from GLPI, from flat file,  from corporate database extraction/query)
Step 4 - nmap specific (specific tests to figure out what exactly is running)
Step 4 - SNMP specific (run a SNMP custom discovery module)
Step 5 - WMI queries

Putting it together :
Business rules to manage the flow through each step for each type of template.

Application to the configuration automatically through management rules
Presentation of the results (what can be supervised) in the skonf UI
 rule based
Hi Joseph,

Thanks :)

That's actually the first thing I tried but it meant a lot more changes all over the place. I tried the metadata approach in parallel and it was more elegant... Plus I want metadata too :)

BTW, I merged it with your new changes but not sure the regular functionality is now working, more testing to do before I push. 

Wout.

On Jan 25, 2012, at 4:20, Joseph Gentle<reply@reply.github.com> wrote:

> I really like the idea of having a 'shout' message to do normal communication to everyone on the document. But I'm not convinced that should be mixed with metadata though. Maybe we should add another kind of message in the wire protocol:
> 
> `{doc:'hello', shout:<JSON>}`
> 
> Which broadcasts the message to all clients which have the document open. I can see that being useful for a bunch of things, including a document-wide chat window.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/wmertens/ShareJS/commit/59e438139cd3e0417dc9c9577f671f91575ca45b#commitcomment-898897
Oh, perhaps another idea to fix this debug configuration limitation is to change install_qt4_executable() to pass the plugins into fixup_qt4_executable() plugin argument instead of calling install_qt4_plugin() itself and passing through the libs argument.

For example:
install(CODE " ....  pass debug plugins into fixup_qt4_executable() .... " CONFIGURATIONS Debug)
install(CODE " ....  pass release plugins into fixup_qt4_executable() .... " CONFIGURATIONS Release RelWithDebInfo MinSizeRel)

Then install_qt4_plugin() would only ever be used at install time instead of configure time.  Maybe that can be simplified?
Ah why not yes.

case ...
    $, -> 
    $; -> 
    and so  on
:) Tanks. What about an option to play only bg-music when an video is available :D

AND VERY IMPORTANT: Increase ScreenVersion from CScreenOptionsVideo and add/update changelog for this Screen.
Should work... maybe because it's in the if?
Sorry, but I took the scripts from Signfinder repo fork. If you say me real author, I renamed commit name.

P.S. Sorry for my bad english.
@ahodesuka this commit does more than just add the feature `Auto noko to replies from the main board when not in dump mode.`
IntelliJ IDEA alerted me to this one. If these are among the problems you mentioned in #21, we indeed need to take action once #17 is in.
What is the intention of this commit?
Agree.  We can improve on usage.  It's investing to note that we use templates as a developer convenience, but all of our HTML documents are cacheable.  Really this function could become subsumed by an option to res.render() that we could send upstream...
cm7 i believe, i had it laying around.
MD5 sum is different.
We're dealing with this shit right now

 E/Adreno200-EGL(  169): eglLockWindowSurface: invalid memory type for fd=15 offs=0, flags=0
 E/Adreno200-EGL(  169): eglLockWindowSurface: failed to map the memory for fd=15 offs=0
so, what would this do? (im a noob, but something pulled me towards this feature)
I really fell in love with the style `return callback(...)`. It is considered a good practice to always return a value from functions, even if they are asynchronous. Also, when you call the callback you really want to signal that you are retuning, so it is a __really__ good habit to always use `return callback(...)`
Really? You merged and closed the pull request yourself: https://github.com/whimboo/memchaser/pull/9
btw, mapnik memory datasource bindings in node-mapnik are a lame shortcut. I need to change them to simply expect a mapnik.Feature object. I get frustrated with the time it takes to expose objects and pass them around (compared to boost python), which is the poor excuse for why the js mem datasource bindings don't work like this from the start.
My reasoning was that the test is still checking that the value has
changed, because its

   1. Setting cached_main_body_text_folded to nil,
   2. Doing something that ought to populate the cache,
   3. Checking that cached_main_body_text_folded is no longer nil.

On the other hand this change is also partly an admission of defeat,
because I could not find any way to make cached_main_body_text_folded
reliably be nil initially. Do you have any suggestions for that?

Robin

On Tue, Jan 24, 2012 at 10:02 AM, Seb Bacon <
reply@reply.github.com
> wrote:

> I don't understand this fix.  The intent in the original test was to check
> that the value of `cached_main_body_text_folded` had changed.  The line
> changed here shouldn't be setting the value but testing it.  That was the
> intent anyway!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/sebbacon/alaveteli/commit/3efcaad4342a345713d8ba90dce6e4df0d73e4c3#commitcomment-895580
>
The changes in treebank.py are actually not small: rewrote corpus reader so that it no longer depends on NLTK, added code for punctuation re-attachment.
Same concept, g_list_position can be quite expensive, take care of your precious cycles :)
Are your test changes pushed?  I didn't see anyplace to add a test.

Refactoring makes sense. How about we make a few larger issues in addition to this?  Also need to refactor all the legacy ::instance() calls and event responders.  I'll take these on.
This does not look right to me. Why are ldap and mysql handled in different ways?
A workaround is to change back to git:// and remove the prebuilt fetch. Then  git clone https://github.com/CyanogenMod/android_prebuilt.git -b ics

it's an extra step but might help till the issue is resolved
or here...
This is sick. :D keep it up.
we should also removed line 78.
I just looked at the one on the server and it does nothing with the target.
Sorry, but will have to dig up my old local copy (haven't updated much of
anything on the server for a while).

It *should* be passing the target to each of the three callbacks.  Also
have a delegateTouchListeners around here somewhere that replaces the -
this - object with a delegate element.  I'll try to find that.  There's
about six months worth of extra add-ons, functions, etc. that I never
posted as I got tired of doing everything to this myself (while most of the
rest of the world churned out dubious plug-ins for dying libraries).  I
guess nobody liked Google Code (or things that live longer than a year).  :)

Thanks!

On Sun, Jan 15, 2012 at 7:04 AM, Paul Connolly <
reply@reply.github.com
> wrote:

> I'll have to look at the example I wrote tomorrow (it's in work), and test
> to see if I can resolve this properly. It's most likely just a case of not
> using the fnEnd signature properly on my end. (I assume el would play the
> role of initialTarget?)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/gibletto/My-Library/commit/f11ea1fe7f729a649cac003c482cae4e4f16f950#commitcomment-868487
>
In plaats van een hidden-form kunnen we dit beter in de sessie zetten - lijkt mij.
I started thinking. May be it would be best to return application/octet-stream for unknown types, and, by proxy, for files without extensions?
Should build-depend on (target) readline
У меня дома смогло загрузить банд только по https. Быть может это мои личные настройки guthubа повлияли...
It's considered bad practice to embed the name of your repo in poms by the Maven community, so we follow these guidelines. Set your ~/.m2/settings.xml with the info. Everything should be described at <https://community.jboss.org/wiki/MavenGettingStarted-Users>.
No, let's keep it that way for now. I'm going to tear it out anyway after next week (I.e. #728)
No specs failing right now.
11 view specs are pending.
It was a test! No one downloaded it! Trust me! Its gone, Check my other commit
IE and Opera will load the image immediately from cache
i also liked that one too. are explicit elses nonos?
$_SERVER is not a constant
Ah! @rosarior beat me to it. I should now delete my local zip support branch. (Now that you have saved the world!). :-)
I don't really need these directives anymore, but this is just an example of using absolute paths. Also the directives are still needed if one customized `g:clang_auto_user_options` and removed `gcc`. If you think these lines can mislead someone, I can replace them with some other include directive with absolute path.
I think you forgot to add cloudyshelf.subscribers to the commit
No, just fancy.
Another thing:

in system.c (audio_update function), soundbuffer sample order on Wii/NGC is right channel first then left channel, make sure this is also true on PS3,X360 or you will got complaints about reversed stereo
Assertion failures fixed in 7a261ff9b86164d8e9fc75845efe93b8792e17be - thanks for quick reports
The string externalization architecture in Aptana Studio is on it's 'development' branch, which right now is targeted for the 3.1 release. We try to match up the branches between the rubles and studio. 'development' is the next major release branch, so to get a hold of that Studio build you need to be running nightlies: https://wiki.appcelerator.org/display/tis/Changing+the+Update+Type
return;

;)
thanks, it could take more commits:P
Okay... I'll keep watching this then...
I would love to see it work in my phone...

thanks again for your help :)
listener.onThrowableCaught(event) might be a better method name.
If you go to the admin section in github you can press the "Test Hook" button which will rebuild :)
Ah nice. Ok, so we need that other list to be accessible here (see previous comment), so maybe a getter? I always prefer that as they're easier to deprecate or if we decide to put validation on, we don't have to chant the caller here.
Looks fine to me.  Does it actually work?
Can you please help me with the source for cm9?
I just need to know where to get it, and how to fix it to be ok on HD2,
And I know that on sense roms BFS works fine, only on cm7 we have root problems!
And I have found what was making all the problems in my kernel :)
Now all ok, I will test more and update GIT! :)




Best regards,
Yuri Shpovlov
CCSE, VASCO-CE, McAfee ePO, JNCIA-IDP+UAC, eSafe-CE
System & Security Engineer | Computing Service Department | Northern Branch


-----Original Message-----
From: fhasovic [mailto:reply@reply.github.com] 
Sent: Tuesday, 17 January, 2012 23:15
To: Yuri Shpovlov
Subject: Re: [desire_kernel_35] Added missing genlock.c (just in case) (65ede45)

2010q1-202 is sourcery. And I find my current build extremely stable and ICS friendly ;-) I have built ICS CM9 ROM from source and customized it from tytung's cm9 and my tweaks ;-) And never had any su problems...

--- 
Reply to this email directly or view it on GitHub:
https://github.com/dorimanx/desire_kernel_35/commit/65ede45e82f25f1cac54b361fcabe32e810ab049#commitcomment-875891
I like the second idea, maybe `handleConnectionLimit()` in alignment with the other `handle*()` methods in `peer.server.ServerProtocol`?
Yes, I noticed that... but it seems this is not picked up by the hierarchy, hence my thought this could solve it. Can't test it right now, gonna see what happens at work tomorrow.
We don't need a setter for version, it's just straight up set in this file, and gets overwritten each time. This is just so we know what config people are using.
One future feature for this plugin will be Amazon S3 storage. This should probably not be included with that in mind. Can we remove?
I was just adding an issue about the failuers in json.dumps when dealing with datetime or decimal data types. Your commit seems a suitable solution, Ive' found that 

    from django.core.serializers.json import DjangoJSONEncoder
    json.dumps(results, cls=DjangoJSONEncoder)

also works fine.
Also Matt, the problem with html:true is that it allowed for the user to do script injection and run unauthorized code on the page. Which isn't a problem for students because their events don't get shared. But it is a problem for class events because if a teacher put some malicious code in an event description and we allowed it to be run for each student. They could collect passwords or session information. Which would be bad.
I'm not a Git expert yet, but it appears this plugin has had many issues resolved on a separate branch? https://github.com/padawan/mt-plugin-field-day/network 

What's the rationale for not taking the latest one and working from that? Just curious!
The CM OC method didn't work on my kernel , i think we should turn back to the First OC Method (i don't know who have found that method), I know the first OC method, i will update my source in the afternoon , i have to take a nap, cause i have played badminton for 3 and a half hours :D
@TomGiordano I know you are really busy but could you please take a look at my above request?
actually the bigger question to this would be, where do i find the location to write to? In the specs we merely get passed an int for status, now if this is the address or not, Im not sure. If it is then do we just pass it 0 as syscall.h mentions?
Related to WorkItem: 3563
Comment must be dropped prior to merge. 

You guessed right. I doubt that recursive resolution of the body producer function was necessary at all. Dropping it simplifies things. So vote +1.
Adding a comment..
Теперь, используя html5-отправку, можно постить одинаковые картинки: http://iichan.ru/b/res/2003279.html#2211904
Is this suggestion just for clarity purposes? Or is there an issue with allowing the client record to asynchronously upload? As far as I can tell, the `ClientUploadDelegate` would abort the session even if it was in the next stage.
OK I will try.
I'll revise this a little after merging. I think the message is that if compatibility with schema.org does not matter, then something like this is another option.
Why are we disabling plugins on enable?
This should be:
return new google.maps.LatLng(obj.latitude, obj.longitude)
One possible reason is to allow the elements already enumerated to be GC'ed, although this only happens when the original enum is also GC'ed, which is probably not the common case.  Good change.
Use Yoda conditionals as per WP standards - `'' !== $meta`. Also note the identity, instead of equivalence.
What do you mean? Should I delete /*sp*/ if it is not used?
It's assumed that if you call `next()` with a parameter, it's an error.
No need to HTML paragraph tags either, YUIDoc puts them in by default.
It seemed like a convention when reviewing some of the other deferred/promises libs out there, but the field is much broader than I'd initially found.  I'm taking jQuery's implementation as strong convention for the sake of interoperability (or near-interop).  jq has a promise() method to return the promise, though I suspect it is implemented as a method because it accepts an object parameter to decorate that object with the promise API.  @juandopazo has asked for this same support, but I'm on the fence.
I am nt sure this is correct. the services webapp does have its own top.jsp, right? It shouldn't reference the default UI's top.jsp?
I think that the same (new properties) should be done for the cas-management-webapp module until we decided what to do for the management UI.
Oops! That should be `render($raw = false)`. I fixed it in a subsequent patch.
already fixed on Dev
NPE here?
The flow method acts as the delimiter of flow expression composition 
vs 
The flow method acts as the delimiter of flow expression compose
Yeah, we have already talked about that. If there is not a ticket already, add one.
I have added this feature
Moved all the detail to set.rb, but we still load it now in the main renderer file.
This is needed for the Setting based attributes to work and kick in the special patterns.

I need a different trait for the patterns tests because this one isn't really a pattern, it is specially handled in the Account methods.  And, we should have a way to test packs in an orderly way and I can test with the :sol pack prototype without poluting the wagn repo
SImplified api. just pass through the hash.  I didn't see the need to add :user=>user to the hash.
Added ... thanks ;P
Missing backquote after InteractiveLauncher.

Mhh, maybe you could try to generate the doc to see errors in your module:

    cd kivy
    python doc/autobuild.py
    make doc
orbitual -> orbital
OK - I made some stuff to empirically make sure that SNR conforms to
that definitio, up to some pre-specified tolerance, by adjusting the
variances. Also, refactored the whole thing a bit:

https://github.com/arokem/dipy/commit/5bbb183af9c312aefdf519368213ec7f96ee4225

What do you think?

On Mon, Jun 25, 2012 at 9:27 PM, MrBago
<reply@reply.github.com>
wrote:
>>
>> +    # SNR = var(signal)/var(noise) => var(noise) = var(signal)/SNR:
>> +    p_noise = np.mean(p_signal/snr)
>> +
>> +    if noise_type == 'gaussian':
>> +        # Generate the noise with the correct standard deviation, averaged over
>> +        # all the voxels and with the right shape:
>> +        noise = np.random.randn(*vol.shape) * (np.sqrt(p_noise))
>> +    elif noise_type == 'rician':
>> +        # To generate rician noise, we add two IID Gaussian noise sources in
>> +        # the complex domain and combine them together:
>> +        noise1 = np.random.randn(*vol.shape)
>
> Ya, the std(rician) isn't exactly sigma, it's pretty close to sigma, especially as sigma gets small (snr gets big). The exact form is on Wikipedia.
>
I think there will be tests failing due to this because of bugs not yet fixed in CKAN, I can make a list of the problems but need an hour or so
I don't understand these two classes. Bizarre way of doing namespacing.
Also missing gettext call
That would be a bug.  I'll add a test case and fix downstream.  We don't actually have use cases for the deeper cases yet.
This code is moved outside the mail methods, in setting up the parameters.
They are already implicitly static final (It's an interface).
I only added lines in the same format previous lines were defined in the original code and I want to make minimal changes.
不要加多余空行
I think you must define a default value for waypoints in case you don't pass any waypoint in the options hash.
Please remove the spurious space before the `bool`.
I would simplify it by defining a method ``getFixturesDir`` in the child classes instead of having to call ``$this->getFixtures`` in each child class. What do you think about it ?
Okay, I'm not sure about this one ... for me, both is fine.
Matei can you be more specific what the problem is in local mode? This is really just to garbage collect the dead workers. If you are running locally, it will work just fine if you have multiple workers on the same host, it will just decide to garbage collect them all at once.

If we rely on a timeout, what would it be? In my experience people will leave nodes off for a long time (e.g. if they are swapping out hardware, sometimes for days)... and they'd want to keep seeing that node as dead until they have a replacement.
Remove all of these //, I guess these are left-overs
This looks strange to me.. I think its kind of "standard"to have the word Exception in Exceptions. So maybe we should call it SeekAheadNoBackArrayException ?
It sounds reasonable to keep the information about this around, I think making things more consistent from Python would be nice. Probably the way to do this that best fits the patterns of nditer would be to add a new internal flag in nditer_impl.h, say NPY_ITFLAG_SCALAR, which is set exactly when output_scalars was 1. Then, a function (maybe NpyIter_IsScalar) to query the state of this flag would be needed, analogous to NpyIter_IsBuffered. The Python exposure in npyiter_multi_index_get would then check this function and return an empty tuple if its true.
да, лучше убрать тогда. для того, чтобы фиксировать историю, можно и нужно использовать комментарии к коммитам, а в коде должна объясняться текущая версия
I'm getting a failure in rails test suite using sprockets-rails from rails repo, can you verify if you are getting that error also?
Feels odd having a comment above this class and another one next to it. Could you include this in the main commentary?
I didn't get this test. If you use absolute path it will not generate the cached asset?
I made this https://gist.github.com/e108d17ca5d9ae97135b most of them are used from Base.
What I'm not sure about is Migration, Migrator, Schema, SchemaDumper, SchemaMigration they are used in rake tasks so I don't want to load them when doing rails s, so I guess we shouldn't put that in eager load. Is that correct?
Ya, that is pretty annoying.  We can get around it by defining a private `default_queue` method:

```ruby

require 'minitest/autorun'
require 'thread'

class DelayedMailer
  attr_reader :queue

  def initialize
    @queue = default_queue
  end

  private

  def default_queue
    Rails.queue
  end
end

class TestFoo < MiniTest::Unit::TestCase
  def test_init_calls_Rails_queue
    test_queue = Queue.new

    mailer = Class.new(DelayedMailer) {
      define_method :default_queue do
        test_queue
      end
    }.new

    assert_equal test_queue, mailer.queue
  end
end
```
sorry, i copied from case_sensitivity_test.rb,  i've need to remove this line.
Of course, since you're unconditionally wrapping it in a Some, you can go back to map and just result in ((tpe, Right(choice))).  However the difficulty of seeing through all these layers is why Option[Seq(Either is undesirable.
I'll see your reassessment of code reviews and raise you a SIP-should-go-into-master-when-done-done-and-not-a-second-before.
I have a lot of ideas in this area, confcall later this week?
typo: `reify`
As in ``RectSphereBivariateSpline``, you can change this block to

        if not ier in [0,-1,-2]:
            msg = _spherefit_messages.get(ier, 'ier=%s' % (ier))
            raise ValueError(msg)
let's stick this under require 'time', maybe I have some ocd?
flare_spectrum is not in sunpy... but it's in your branch.. how come it's not shown here??? :-S
_print_diagram?
Two issues
1. I would align the right column
2. This does not generalize if we want to start putting handlers in different places.
move L171 between L166 and L167 and use pargs in your `all` statement?
We can kick it down the road but you must have read the likes of http://stackoverflow.com/questions/5426754/google-python-style-guide . Have you heard something else indicating that GvR's attitude (http://code.activestate.com/lists/python-list/373223/) has changed or are you just prognosticating :-)
参见上面关于if else的新评论。

另外上面还有三条评论你似乎没看。
I **did not** make the change to `password_hash` in the new PR because of the issue I described above.
Follow Yii code style:
- remove extra spaces around operator keywords.
- place code block brakets at the new line.
sourceWriter.commit(treeLogger) does this, it also closes the PrintWriter and commits to the context.
I chose `mail` because PHP has a function named `mail`.
No need for leading backslash in use statements.
This should probably be `!defined(TARGET_OS_IPHONE) && !TARGET_OS_IPHONE` - gcc complains about dereferencing undefined macros in strict mode.
I went back and forth on this, and made sure that QUnit captured test runtime both inclusive and exclusive of setup/teardown, but I don't think it makes sense to *display* both. The former better represents a test's total cost, and I finally settled on it because that is the information I find most compelling. It may be that my use case¹ is unusual, though, and I'd love some other input on this point.

¹ Trying to minimize execution time regardless of association with a specific test callback or per-module setup/teardown
The tests fail on this one.
that approach would allow us to reject explicit assignments of < 0 via REST interface, too
What does S3 have to do with this?
seems a little weird to request `/en-US/dashboards/revisions?locale=de...` Any chance we could pass the locale input param as the locale url prefix instead? then it would be available to the backend as request.locale
Oh oh oh I missed first part of your question. It probably would be best to tie the method into the TraceKit object so it wouldn't potentially override a then-different _.has method. One sec
Do you mean just add line breaks in the conditional or split it between multiple if() statements?
Same here as above, should not be 0 but $config['default_style']
This isn't a MOD so inline `Start`/`End` comments don't really make sense as they aren't used anywhere else.
phpbb 3.1 requires php 5.3.2, the old version checks should be taken out.
Yep, but C++11 standard already exists so it's only a matter of time for compilers to implement it.

Also, but lambdas & closures + std::function is a great feature that was implemented for gcc 4.5: http://gcc.gnu.org/gcc-4.5/cxx0x_status.html

I would better say that gcc 4.5 is minimum requirement.

For now I will remove this this line. But I hope this decision won't be a counter-evidence for using lambdas and std::function in the code.
Yes, exactly. I thought it might come in handy. Does it look like a good or bad idea to you?

I intentionally kept the invocation original callback for uncategorized messages as is (with one argument), but the more I think about it, the addition of categories might interfere with the old approach anyways.

Maybe the whole block could be simplified to

```js
case 'message':
  var category = message.category;
  if (category) this.emit('message:' + category, data);
  this.emit('message', data, category);
  break;
```
General question for @nathanleiby , should we use separate controllers in MVC for extjs, or combine them all into one?
yes, along with that, we talked about this before... the screener encounter should contain an observation on the concept (doctor to visit at OPD) (or the like) and have the value as the doctor name (free-text) or provider-identifier and use that to show count-of-patients-waiting-at-doctor
No this is something to setup for later... This is for updating deployed modules automatically.
So we create a repository that will host the new module omod and create a update.rdf. When the OpenMRS server admin logs on, they will be able to see that there are updates available for the module and they can press update and get the newer version of the omod. We dont worry about this now... but after we have our first deployment done :)
As this is not in a factory-specific module, I'd suggest *_dereference_factory_args*.
Style fix, no spaces around the `if`
多个attr合并为一个吧。
Do not add print in tests enabled by default.
better yet:

```
SNMP_V3_AUTH = (
    settings.SNMP_V3_USER,
    settings.SNMP_V3_AUTH_KEY,
    settings.SNMP_V3_PRIV_KEY,
)
if not all(SNMP_V3_AUTH):
  SNMP_V3_AUTH = None
See the comment in settings.py
Typo: s/desiderd/desired/
We should just remove `measureit`. It's not used anywhere in the codebase, and without guppy it is useless anyway.
Didn't the lambda syntax/feature fail on 1.8.7?
yes, that is what I am doing :)
And just so I understand - you need to run your server as root, listening to port 80 to test this locally?
Also document the return or remove it.
We just have happy and sad--so we can ditch the idea check here.
Whenever someone posts to a page and you handle it, you should redirect them. Otherwise if they refresh the page, it causes the action to happen again and that's bad.
Yes, it could be an instance method version and `if mecab_support?` would be OK here. However I reused the class method in the instance context to avoid duplication.
New style:
`build.include? 'with-metis'`
Oops. 
I removed this line. Cannot think of a reason why I put it there (anymore).
Why not using `forEach`here (or maybe more appropriately `filter`)?
^Indicate if the comment is a top level comment.
Done. Hopefully it will be clearer now (though a lot longer :))
The diff sucks, but I basically just added lines 75 - 82 and removed the require that was around the code below.
Isn't this redundant?  Have not tested lately but I think the struct declaration works just like class declaration such that "eleveldb_itr_handle" is already a synonym for "struct eleveldb_itr_handle.  But I could be wrong.
This is problem places
I tend to use `alias` when aliasing methods that were explicitly defined in the same file, and use `alias_method` in meta-programming.
What I mean is `:load-path "mu4e"`.
The load-path part of the recipe should definitely refer to the mu4e subdirectory, which is the path where all the emacs-lisp files are. As for including mu with mu4e, I think that is fine, especially considering the fact that @jleechpe's points out. I have two concerns, though: 

1. The build process looks unix-specific. Is there a way to build it on windows? If not, that's ok,. We can't magically package a non-windows program for windows. But if there is, we should add a windows-specific build property as well.
2. Does mu need to be installed into the user's `$PATH` in order for mu4e to work? If not, what setup is required? Based on a cursory examination of the mu4e code [here](https://github.com/djcb/mu/blob/master/mu4e/mu4e-vars.el#L42), it looks at a minimum the variable `mu4e-mu-binary` needs to be set in a post-init form. Something like this, assuming that the mu binary is built in the top directory of the repo:

        :post-init (setq mu4e-mu-binary (expand-file-name "mu" (el-get-package-directory "mu")))
It may be more efficient to call equalsIgnoreCase rather than lowering the input strings.
But you have to provide the code for all this things you are saying we are supporting!
I'll take a stab at it. 
I'm generally not afraid of large function bodies if there's no reason not to. 
I think some pieces could be moved out into private methods or something just to make it feel/look more manageable.
If you go this route, you should validate that you have ##px, right?
Why only for type === number?
Uppercase please
This code shall be commented
I'd suggest using keys here more like INCORRECT_PIN_LENGTH
just something minor that I've noticed throughout this patch, please put a space between variable, = and value so the coding style matches the one from the other pvr related code.
--with-http_session_sticky_module=shared 加上upstream，改成--with-http_upstream_session_sticky_module=shared
Defining a new instance manually is now more cumbersome, as you can see from all the definitions you had to change to include a boilerplate call to `combine`, which was previously done by the default implementation of `hashWithSalt`.
You're right. It's only three pieces, in the same file, and fairly obvious that it's duplicated. The risk of changing only once piece is fairly low.
Looks like jank
I think he's talking about the a on line 87?
This is probably not an issue, but will ask anyway. If remove_template happens to delete the current template (self), would that cause issue for self.parent.remove?
this is the reason why there is a bunch of protected methods: you can extend the class to customize it.
This should be "By click the" and "agreeing to the terms"
Why not do the same kind of check here? IsJsonP instead of always falling back.
There is an existing method called `libcloud.utils.files.read_in_chunks` which already does what you are doing here.
Well, technically you'll get either a TypeError or a ValueError (the latter if the string doesn't parse).
Is there a way to pass a mask to the constructor?
by default python will return None 
On Aug 20, 2012, at 8:48 AM, jao <notifications@github.com> wrote:

> In bigml/util.py:
> 
> > +def is_url(value):
> > +    """Returns True if value is a valid URL.
> > +
> > +    """
> > +    url = isinstance(value, basestring) and urlparse(value)
> > +    return url and url.scheme and url.netloc and url.path
> > +
> > +
> > +def split(children):
> > +    """Returns the field that is used by the node to make a decision.
> > +
> > +    """
> > +    field = set([child.predicate.field for child in children])
> > +
> > +    if len(field) == 1:
> > +        return field.pop()
> Is it okay to have no return when the condition is false?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Yeah, that's good!
My directory is so cluttered that I use "git status -uno" ;)
Most of the "this" scope arguments are not necessary, except for the outmost Ext.earch(), are they?
Right, thanks :-)
Ah, sorry, I mean I'll fix the `@param callable $cloner` part :)

@stof the point in closures is exactly to avoid problems with dumping or accessing the objects in here... 
Using `call_user_func` (for compatibility with 5.3) in 5.4 is also much slower than just invoking the closure, which is an additional point in favor of allowing closures only.
Yes, it's a little pedantic, but in most cases you are likely to have a bug somewhere in your code.
Just one issue that I see: I tend to prefer config variables that are positive, to avoid having to write code with effectively double-negatives.  `if not disabled` is harder to parse mentally than `if enabled`, and for the other case, `if not enabled` is just about the same mental load as `if disabled`.  So I would vote for changing this one to have the config variable be `enabled`.

I know it will require flipping all the logic in the PR around, but it's easy work.  Happy to discuss if you disagree, but let's go over it before merging...
Great! I'll move it somewhere accessible for you. I think the filter API is the only good place for it.
~~~~ MY BAD ~~~~
Hum, let me pull this down and play around with it during any downtime in my office hours today.
We should return a list sorted by the native lang names. If we do this, no need to sort `LANGS` earlier.
Oh I see. If you look at `SORT_QUESTIONS` above, it uses two values for one of the sorts. So, I figured I'd keep the same API.
Ah, ok. I was a bit unsure about this, thanks.
Actually it'll make it quite noisy and confusing if we do exc_info=True here, since there are certain things we can't easily catch (like roles already existing) that will warn a lot, or harmless things like specifying --dropdb when the db in question does not exist.
Doesn't feel much different to me, but the use of old_email and new_email instead of mail1 and mail2 is nicer imho.
Would it be better to create all the methods eagerly? I could go either way myself.

Not that I offer any alternatives, but Method.equals() is a painfully slow method. It uses toString(). Sigh.
Please typehint the OutputInterface instead

and ``$arguments = array(), Output $output`` is useless as ``$arguments`` is not optional here as there is a mandatory argument after it.
instead of adding a dependency to Doctrine Common in the Form testsuite (which forbids running the testsuite on the standalone component btw as you don't declare the requirement and don't check if the class exists), I would simply use ArrayObject for the test (which suffers from the same issue)
Please remove this alert call
this code is in a ```spaceless``` tag. So you don't need to use ```{%-```, ```{%``` is enougth
it's the same for other lines.
The question is: do we want to allow integers only or would parameters be fine also _("lifetime" is a generic name above)_
15 minute, right?
same question here on the triple braces
I think this is a dependency of omniauth-yammer (prob omniauth-oauth2).

Ideally I'd like to remove it - it's a bit confusing to have two oauth gems. If we can't, NBD.
yes, it's tested
Can we rename this to EntityManager.  It's more standard.
Alright, fixed the issues and per our conversation added a couple of other minor changes I had in development. I've updated the pull request.