labofsahil/github-event-dataset-2012 huggingface dataset:
Multiple digits this time...

This commit has a *very* simple implementation to get the first 2 tests passing, and has a new test for multiple digits.

Again, try to focus on doing the simplest possible thing that can work, *even if it's ugly.* If you come up with an ugly solution that is simple and easy to understand, go ahead and commit it as a working version of the build. You can commit a refactor separately before ping-ponging back to me.

When you're done w/ the implementation (green) and refactor for this new test, feel free to write an extra test or two to handle more than 2 numbers, but make sure to write a test for newlines before kicking it back to me. This should be the next example that I will need to implement: `"1\n2,3"`
LPS-19114 Modify URL to match GZipFilter ignore
an update to the last pull, simplified so existing implementations won't have to be changed.

an update to the last pull, simplified so existing implementations won't have to be changed.
Backwards compatibility with Python 2.5

collections.Mapping doesn't exist in Python 2.5. Since it's only being used to
check if an object is (essentially) a dictionary, setting Mapping = dict is a
reasonable shim for backwards compatibility.
Remove json dependency

Started, but not finished
Derp - double lines ;)
updated spec file to include provision data queries

add data/provision_data.ddl and data/provision_data.rb to their respective packgages
bugfix to pass redirect_uri when initialize SC client
Comment layouting

Not perfect yet, but moving to right direction.
Save the current view (if dirty) before reloading

Thanks for the handy plugin. Only thing missing for me was that I had to save, then do cmd-shift-r. I've modified the plugin so that if the current view is unsaved (dirty is sublime terms), it is saved before the reload is triggered. Now you can just do cmd-shift-r on an unsaved file and it will be saved and Chrome will be reloaded.
Simplified the Map.get_blank_map() method to a one liner

Using python's ```[0] * 5 = [0,0,0,0,0]``` functionality, we can avoid two nested loops and generate the array in one go.
Oneworld

If there is only one world, this makes it so the world selector isn't shown. It's like magic!
translation of zh_TW, Traditional Chinese

translation of zh_TW, Traditional Chinese
Deprecated features in druntime

Stop using 'length' inside index expressions and local variable shadowing.

This clears the way for turning these old deprecated features into errors.
Issue #4 - Refactor SituationQueryBean

As discussed a while back, I'd like to propose cleaning up the service alert query API in the TransitDataService API.  To recap the discussion, the SituationQueryBean would be refactored to match the OBA / SIRI / GTFS-realtime "affects" clause model, where the query mechanism for alerts uses the same style of affects clauses as the alerts themselves.  This pull request implements the proposed change.

I know it's been months since we last had a discussion around this, but there are some bugs in the onebusaway-webapp admin interface for editing service alerts that would be resolved by this pull request.  However, it would also break any existing users of the SituationQueryBean interface, so care must be taken to modify any downstream clients after this change.

Thoughts?
Improve Augeasquery agent

This PR improves the Augeasquery agent:
 - add an 'augeas' application to ease requests;
 - add match values in output (when available);
 - add a 'transform' parameter to add mappings on the fly;
 - add documentation in README.
o-f-t-r-t-a
Performance and rough functionality improvemetns

By switching the implementation over to use dicts the queueing has been made twice as fast.

I also added prototypes for filtering out albums by length and by name although they remain commented out because I don't know how to make a preferences dialog.
fixes poco + sound, videoplayer and videograbber working

also ofAppNoWindow + TARGET_NODISPLAY for boards without graphics output like the beaglebone
ACP is missing some language entries

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4256

+
`SN_ADDONS_ADDON_EDITED`
`'SN_ADDONS_ADDON_EDITED_ERROR'`
Load older posts earlier than user hits bottom of the page

Older posts should start loading earlier than user hits the very bottom of the page. It should start at least at 200px before bottom.
Textbox overflow on copy/paste of bigger text 

It is problem with `.sn-us-mentions`. It's height needs to be updated along with updating height of the textarea.

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4307
Remote AppHb bit, left in as an example
Исправления для форчана

1. Исправлены стили вкладок
2. Исправлено http://iichan.ru/b/res/2355532.html#2368825. Там простой null-проверкой `post.Msg` не обойтись, а проверять `msg.parentNode` каждого поста накладно, поэтому я просто засунул код в `try`.
3. Исправлен постинг
Add Gotye's "Somebody That I Used To Know" to songs.

This is a great video (and a fucking awesome song).
Reject empty string for required field

-Update the validation code to reject empty strings for required fields.

-Add test case to publish to tests publishing a document with an empty string
for a required field.

Fixes #169
drag now has outline

warning: the outline is drawn onto the map instead of on top, so if you go to far with the drag it will keep the outline.
Adding new rim header custom headers

Fixes Issue: blackberry/BB10-WebWorks-Framework#333
Updating dialog to swallow insecure resources

This solves the PR: https://przilla.ott.qnx.com/bugzilla/show_bug.cgi?id=188971
adds dual 3.4 and 3.6 support

I added the switch for this to work with 3.4 and 3.6 and also paired it to fix the icon for 3.6.
Oversatt manglende tekst linjer
Column width adjustments and JSLint formatting

Adjusted lines to fit inside 80 characters, made minor adjustments to formatting based on JSLint messages (specifically the eval from c9.io )
Updated formatting of instructions code block in Readme.md

Strangely the line breaking was not functioning using just indentation to indicate code. Manually added tags to preformat code in order to allow copy-pasting.
Updated CodeIgniter files to 2.1.0 (Nov 14, 2011 release)
Collection new_label translation with i18n_catalogue

Allows this:
```yaml
generator: admingenerator.generator.doctrine
params:
  model: Avocode\TestBundle\Entity\Album
  namespace_prefix: Avocode
  bundle_name: TestBundle
  i18n_catalog: AvocodeTestBundle
  fields:
    songs:
      formType: collection
      extras:
        new_label: album.songs.new.label   #will look for album.songs.new.label in AvocodeTestBundle.xx.yml
      addFormOptions:
        type: \Avocode\TestBundle\Form\Type\EmbedsongType
        allow_add: true
        allow_delete: true
        by_reference: false
```
change vim-ruby to use a more current repo
Remove Google Maps API key.

Google Maps API v3 no longer requires a key; omitting the key makes it
easier to get the visualizer up and running quickly.
Update 20-files-present-and-referenced to ignore .git and .emacs.backup

Self-explanatory :-)  I can't think of a scenario where these files should ever be referenced (and .gitignore is already on the list).
Fail in load a bitmap data compressed by RLE4.

Fail in load a bitmap data compressed by RLE4.

This is test code:
https://github.com/kntroh/RLE4-bitmap-test
hoers K Hansson
(#11959) Use Puppet.warning instead of Puppet.warn

Previously, Puppet.warn was used in lib/puppet/cloudpack.rb if the
remote Puppet agent certname couldn't be determined. Now, Puppet.warning
will be used instead.
Adds docs for DMD pull requests #1099 and #1100.
fixing setup for tests

removed require_onces for old autoloader
moved lib outside of features folder
ignored vendors folder
set right autoloader on phpunit configuration
Update NaiveBayesClassifier.php

Query optimization.
Increases speed.
tesat

asdfasdf
Build changes for node packager
Added the ability to use pre set preferences

Added the ability to use pre set preferences, haven't done images as graphics isn't really my strong point.
Trying to wrap my head around this one...
Great that the code is committed but what's the point of leaving the comments in?
following DRY, since we now use this same block of code twice, consider moving it to a partial.
I had gotten used to `out` as well, but I think in this case backwards-compatibility is a better argument.
а в сообщении об ошибке так и осталось 70?
server/upsd.ch mods are to be discarded
upsc mods are still missing
I'll try to fork a branch to complete it.
Nice website :)
@DouweM He did not exploit anything. Please realize that.
That leads to this list:

``` text
#	deleted:    req/binject/ext/binject_c/binject.bundle
#	deleted:    req/chipmunk/ext/chipmunk/chipmunk.bundle
#	deleted:    req/ftsearch/ext/ftsearchrt/ftsearchrt.bundle
#	deleted:    req/hpricot/ext/fast_xs/fast_xs.bundle
#	deleted:    req/hpricot/ext/hpricot_scan/hpricot_scan.bundle
#	deleted:    req/json/ext/generator/generator.bundle
#	deleted:    req/json/ext/parser/parser.bundle
#	deleted:    req/redcarpet2/ext/redcarpet/redcarpet.so
#	deleted:    req/redcarpet2/lib/redcarpet.so
#	deleted:    req/sqlite3/ext/sqlite3/sqlite3_native.bundle
```

So adding .bundle and .so should be good, I think.
I'm compiling node 0.6.10 against libev 4.11 (original shared system lib).
The patch needed by node was included in libev 4.11, so it does not need to be forked any more.
Me, late ?
It's far from perfect but at least we can move the Zeta Components release forward.
Why can't I like a commit?
is this deleted anywhere?  The whole class wouldn't be needed if the job fired a GUI_MSG_UPDATE when done.

After all, the job is disposable, in that we don't care about progress, and the job itself doesn't contain anything useful that we need to keep track of.
Nice, thanks you.
Could you add some comments regarding the "JMX-friendly optimisations" ? That's useful to know if someone where to try making the concurrent version of the RetransmitTable. Especially, it would be useful to know if there are other cases in which strict consistency is not required.
And... fixed!
oups... yes you're right.
Indeed it would. I hadn't seen that.
Shouldn't we rename these to something like cvt_l_s_ (underscore at the end) as with ins, ext?
fool me :<
aye aye
Hartnäckig das Ding.
Hello,
Could you review you patch related to mailstream_cfstream.[ch]?
mailstream_ssl.[ch] also uses int16_t and has no build issue on win32.
Best Regards.
Does the "/" here work on Windows as well?
Not sure why you removed this, but it breaks 'manual' triggering.  Without it the tip only shows up on the second and subsequent calls to show().
It sure does, it still doesn't hurt to think about. In this case a it would've actually uncovered that the addGame branch does need not to be adapted in the same way, a thing I didn't think about initially either sorry, I just noticed it since I saw the changes is in "addGame" and the comment does not seem to make much sense in combination with that. That is since if a game was detected we know that the plugin should be present. In case we can not load the plugin directly after the detection anymore, something is really fishy. It might make sense to think about erroring out in this case. The current behavior is harmless of course.
Looks great. The obvious question: Is there any way to test this at all?
This kinda fucked up the database read, doesnt look like a db session can be opened in an include header then accessed from the main file.
SEJeff, I think I am going to side with Seth, besides, as I recall, you generally argue against this kind of magic (you argued against autorequires), that the user state would not define homedir management, yet it is magically added since we assume that the inclusion of the ssh_auth?

This makes the user state non definitive, and crosses scope in a way that I would call unpredictable.

Also, Salt has been made with the goal of being accurate to the definition presented, and that hidden operations are minimal.

Also, we don't have anything that manually creates home dirs (at least we should not), we use useradd to do it, so the expected platform behavior executes
kann mir das für alle art von medien gut vorstellen,
sollte man nicht aus den augen verlieren das thema

und das 1.4 prob sollte auch zu lösen sein, 
geht nichts gibts nicht, kennen wir ja :)
What's the different between " and ' ?
sindra summons bomb trigger only in LOS.
```javascript
this.matrix.multiplySelf( matrix );
```

I don't think that works like that. Got the feature to work by multiplying the other way around (like in `updateMatrixWorld()`).

I've tested the code a bit and seems to work, but the rotations aren't right when you rotate more than one axis. `getRotationFromMatrix()` may need some work.
I think that copying JetClassObjectTreeNode from JetClassOrObjectTreeNode is not a best decision here. For example canRepresent can be reused after weakened first parameter to PsiElement. getChildrenImpl methods in both classes are also look very same.

Can you call getObjectDeclaration() and reuse existing code somehow?
It builds now. Now I get a run time error:

```
/home/jeff/src/julia2/julia/lib/libarpack.so: undefined symbol: dzasum_
```
Oh ok. The regression seems unrelated to using system libs because it happens even with the compiled libs. You can check it out on beowulf.


-viral


On 30-Jan-2012, at 10:49 PM, Stefan Karpinski<reply@reply.github.com> wrote:

> Well, it's central in the sense that it's not a feature you can leave out. If PCRE is compiled with the wrong options or doesn't support the same options as the ones we expect, regex support will just break. For example, we definitely need Unicode support turned on. I have no idea why the hashing tests would fail with this change. Strange.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/284919152afe72f6b7caa3949a0e894248e4a662#commitcomment-914875
why that, now our custom gameobjects and creatures are exactly behind UDB or YTDB entry
Not because of instance bug, just to verify that missing map don't cause high cpu load.
raynar is correct about cpu and i think maps were server side only files
Crash on Rev 1605 after ".reload all"
detail crashlog: http://paste2.org/p/1890525
Yes, IMO P should be visible.
If there are always 3 items to put in, why not have 3 gapfills?
So you'd recommend I separate the code into separate 'put' and 'putImpl' parts similar to 'to' and 'toImpl'?
It's good that we finally credit him.

I am wondering about why he is listed under the list of developers in the engine team though. I can't seem to find any commit of him under the nickname there nor under his real name, so I am not sure whether he actually was part of the ScummVM team in the past. If he was that's of course the right thing to do. In case he was not I am wondering why for example the guy who contributed the initial gob engine is not listed in the gob team, but DanielFox is listed in the engine team now.
Ma justification était : "On repasse en basic flat model pour la segmentation : sinon on peut pas taper à l'adresse 0xFFFFF000 par exemple :/."

Je ne suis plus trop dans le bain du coup j'aurais du mal à me prononcer là.
You won't need this. In fact, there is no OSGi service providing ProtocolSessionContext, so injection will fail. Instead, contexts are created in createWonesysProtocolSessionContext() method, inside the test.
The unit tests still need updated and checked as well as updating the docs and minified version.
Rod's the expert on the restoreSession stuff. Would like to see him weigh in here, since this seems a bit unusual.
I was overreacting (as I always do when PRECIOUS BYTES ARE WASTED!!11) but the HTML for the favicon is not needed in this case, since it’s in the root of the document. See http://mathiasbynens.be/notes/rel-shortcut-icon if you’re interested enough.
The shebang is not useful for "lib" python file, because they won't be run, but it's just a detail ;)
VisAD IOSP dependency version is derived from current tds version, not VisAD library version.
can you open a pull request, so it's easier for me to track these and not forget about them? thanks!
Revert the change to the above line, its not related to Infinispan 2lc and it will break Hibernate 3.3.x.
r+ sweet looks hot!
should delete extra print
We haven't implemented clone cells yet. Actually, DimScape is on hold
for a while because we are doing some work for Ted Nelson that takes
priority. If you'd like to implement clone cells, we would like that
-- and I would at least be willing to help you with figuring out how
to implement them up to spec (the Xanadu specifications are pretty
confusing at best).

In terms of video cells -- there's a handful of problems with them,
ranging from the limited set of formats supported by gstreamer to the
differences between the qt-phonon and the kde-phonon. I have had them
working to a great degree, but even then I always had problems with
the colour pallette. It's a nice feature to have, but it is certainly
not fundamental to a zzstructure navigator, and the DimScape
implementation is at best a toy.

I'm glad to see some interest in the project, and I'll forward this to
the guy who did most of the work on this project.

On Mon, Jan 30, 2012 at 7:37 AM, miquelet
<reply@reply.github.com>
wrote:
> Hello,
>
> I don't know if I'm writing this in the correct place of github, I'm completely new here. So, my apologies if this is not the place.
>
> I'm interested in zzStructures -since I've read of it from an article of Ted Nelson, I think it's an amazing field to explore- and I've found your applitude Dimscape first at YouTube and then at github. I have Dimscape installed and running under Ubuntu 11.10.
>
> Well, it works, my congratulations! But I can make it work only with TEXT zzcells, not for video, and in fact this is not an important issue for my approach to zzStructures. But I've tried to setup zzcells with multiple zzlinks in the same dimension -which requires zzcell cloning- , and as far as I can see this feature is not still implemented in Dimscape. Am I wrong? Are you planning to implement that in a short term?
>
> I have only preliminary skills in Python and SQLite, so I still don't feel able to collaborate, but may be in the future I will.
>
> Regards
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ccs4ever/Dimscape/commit/bc9531fe866dde5ffed0177dbf9da5c12a67d06c#commitcomment-914046



-- 
--
John Ohno
http://firstchurchofspacejesus.blogspot.com/
Nice work :-)
Yeah, I think it makes sense. Most client won't care about this, maybe they want to expose the state (varz or logs).
Added test case 1 - this fails with uncorrected code. And works with corrected code.
Probably better if the comment read "Added to ensure Joomla 1.7 compatibility" since it worked then.

Thanks for fixing this so quickly, Rouven.
I wrote the translation to this as "någon har redan gjort anspråk på den koden -- kanske var det du?". I really don't think "paxat" should be used, the point of the translation should be to make it as easy as possible to understand. "Paxat" is slang, and will probably be understood by less people than if you used an actual recognized word.
% (prefix,)
Lovely change.
Added the ability to use Less instead of CSS with components.
Less files should be placed in the Less folder instead of CSS.
Add the following to your _config.xml file to start using less.
\<display\>
      \<property name="useLess" value="true" /\>
\</display\>
och nej, det borde inte vara `(...).to_s`, för det fixar `number_to_currency`-anropet som står nedanför alla dessa kommentarer.
euh ... c'est ... surprenant ... tu devrait travailler avec madeleine vu qu'elle a déjà commencer à faire qqchose avec les template ... là ça me semble un peu très beaucoup bancale ...

Et puis, je dis ça je dis rien, mais il faudrait p-e commencer par avoir qqchose dans index.php (point d'entrée du site), non ?

Ensuite concernant ce fichier, je doute que tu l'ai testé avant de commiter : permission_test(...), ça ne marche pas ... il faut $sys->permission_test(...) ...
my changes where the initial ones when working with the original neo4j repo (not the ractive branches). Those changes are just about upgrading the libs used.
So, i am on ractive now, but nothing pushed yet.

@ractive thanks for adding the collaborators ;-)
Instead of using a span, wouldn't an "abbr" element be more semantic?

```php
<abbr class="required" title="<?php esc_attr_e( 'required', 'woocommerce' ) ?>">*</abbr>
```
Thanks @ahodesuka
And there's the hide subject option @Mikenter
Nitty nit: new line at end of file.
Impressive indeed.
It seems that Cloud Engines & Medion both simply modified this file. The CE is dev:14336k(boot),-(rootfs)

Should prove interesting to sort them out.
The checks work. The reason it's confusing is because there are so many different permutations that `BlockController::install()` can return.

If you don't have a db.xml file (*and* you don't declare the `$btTable` variable in your block controller), then the install method returns object `$r` having `$r->result == true`, which passes validation just fine.

If you don't have a db.xml file but you *do* declare `$btTable` in your block controller, then the system will look for the db.xml file but it will be missing, in which case `BlockController::install()` returns FALSE (not the object with ->result and ->message), and that triggers the first branch of my validation check (the one that shows "Error: Block Type cannot be installed because no db.xml file can be found. Either create a db.xml file for this block type, or remove the $btTable variable from its controller."

I think the comments of the `BlockController::install()` method need to be updated to explain all the permutations, because right now it just says it returns a boolean, which is very wrong.
ya which part trying to kid lunches and type on a droid 8P
Where’s the difference between resources/core and resources/test_app ?
From your description it sounded like this was being handled in Ember, so that being the case I think it would be better if you did it.
It would probably make sense to add support in Eris while you are working on the code.

I haven't updated all the scripts yet though, so expect the latest server code to spew errors, and send out data in the old format for the meantime.
Well, since we already subtracted the mean when computing each covariance, it is still not 100% correct but should be close assuming we have enough samples and the mean in both segments is the same. 

I think the really correct solution would be to also store E[XX^T] and E[X]E[X]^T in the covariance object, since the covariance is given by

cov(X) = E[XX^T] - E[X]E[X]^T

we could also rescale E[X]E[X]^T when "adding covariances".  But this would make things more complicated and break comparability with MNE..
Too bad. With this new commit, there are more and more false click events when scrolling ;))
We do that to use a custom template. Our websites are mostly in french and the template from capistrano is in english, so we rewrote it to have a maintenance page fully in french.

This feature is now present in the master branch of capistrano but it wasn't the case when I wrote this file. When the next version of capistrano will be released, I'll deprecate this task.
I think you just deleted all the commit history somehow. The [Network tab](https://github.com/Shadowraze/MagicWord/network) is blank.
nice one, bug was driving me bonkers!
Can we use this tag in layouts? Say if 2 or 3 pages share same layout structure, can this tag be used to form a unique layout so that in different pages can we use the same chunk name? I guess for this, chunk should take only name attribute. 

Will this work?
Hello Daniel,
Thanks a lot for this contribution! It will be a great visual improvement for e-ink users!
We are currently in beta phase, which means AnkiDroid 1.1 will be released when we are finished fixing its major bugs.
As a consequence, improvements like this one will be included in the next version, which will be probably be AnkiDroid 2.0 :-)
Don't hesitate to send more improvement, or bug fixes, thanks for your great work!
Cheers!
Nicolas Raoul
I agree best would be to not have the need to update, need to think a bit about how to achieve this.
Buena suerte y que os vaya bien ;) yo solia coger fixes vuestros pero bueno :(.
I just use the idle defaul setti ng



On Sunday, February 19, 2012, Daifu <
reply@reply.github.com>
wrote:
> It needs 4 whitespace, you have two here.
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/khaing211/g-simulator/commit/73eae3c53e06ca2bb2dd4a803928aad5f374e6c5#commitcomment-982950
>
Forgot to mention, most of this includes whitespace fixes (stupid mixing spaces/tabs).
Again please do not use these presentational attributes in the markup. These should be in the CSS.
An alternative would be to have an explicit Parameters property that is a collection of opaque strings passed straight to the builder for it to do whatever it feels is right with. This would allow things that are not easily parsed by the command parser to be available and could be done as well as the spare inputs support.
Can we please remove that ugly pink selection color? Better to just use the default one in this case.
How is the checkLoggable boolean set from the configuration file?
I'm guessing something like the following would be the default.

```xml
   <configuration>
     <appender name="LOGCAT"
                class="ch.qos.logback.classic.android.LogcatAppender"
                checkLoggable="true" >
          <tagEncoder>
                 <pattern>%logger{0}</pattern>
          </tagEncoder>
           <encoder>
                <pattern>[%method] > %msg%n</pattern>
           </encoder>
      </appender>
      <root level="TRACE" >
         <appender-ref ref="LOGCAT" />
      </root>
   </configuration>
```
Thank you, for that QAston, but just want to help when I can.
As requested by @superfetz on Twitter : )
Found the related commit: 37705e712bec12ffaf412009244e2b9cddba25a2. @jashkenas: do you know why that change was made?
What good is sending methods that you expect your model to respond to (deep in `@proxy.@object`) to the anonymous instance? It has no logic for the attribute. So previous version relying on talking to their models in lazy blocks end up talking to a stupid anonymous class instance with no delegation.
si, toca parchear el .exe xD
By "bs" I'm assuming you mean "breathtakingly sublime" ;-)
Thanks Merrick,

yeah so:

```
app/
- scripts/
  - backbone.js
  - vendor/
    - backbone.js
    - jquery.js
    - underscore.js
```

Merrick's code being `app/backbone.js`, a little shim module to give you direct access to backbone as a module. Could even toss some noConflicts in there to remove everything from the global scope.
Looks good to me, will leave for @pitapoison to merge
Turns out we wanted the old behaviour after all.  Testing this with line 38 reverted.
This puts the scripts ON the device that's mounted, so that IT has chance to say where it will mount, or that some maintenance processes must be run.
Corresponds to build 355.
Most source files changed from 4 space tab indent to 2 space indent.
Introduced isocenter-based positioning and synchronization for CT displays in CompareDialog.
Lots of small changes and bug fixes.
@olegstepura The actual rule depend of your alias. As it depends of your setup, it cannot be enabled by default (it would break things more often than fixing them). But now, people can see it and uncomment it when they need it.
Where's your .emacs? :)
Apply Aura: Water Walking

What's the point of target->ToPlayer()->CanCastWhileWalking(58265);
Also,58265 triggers a glyph of this spell :|
Broke fresh build.

C:/Skyfire/4.0.6 branch/clone/src/server/game/Entities/Player\Player.h(33): fatal error C1083: Cannot open include file: 'NPCHandler.h': No such file or directory
lol – it doesn't save any typing, so what's the point?

also, i'd probably implement these methods on the Integer class – especially if the code were for internal use. `n.factorial.digit_sum` is just way more readable IMO
Oh, you're right.  I forgot to build with `ENABLE_BCM_4329=true` when using `CUSTOMER_HW2`.  brb...
Все верно. readThreadVisib(); вызывается, когда мы находимся в треде. В ином случае callback не вызывается.
if we also add a "domain" annotation we might even can get completely rid of it; btw we might should carry this discussion rather to the dev list
Cannot instantiate abstract class PHPUnit_Framework_ComparisonFailure
Ne sviđa mi se ovo. Nije dobro mijenjati objekt koji je vezan za ORM, a da ta promjena nije namjenjenena spremanju u bazu. Promijenim ovo.
What can't Catalyst::Test do for this test?
I'd suggest refactoring these blocks out into a constructor (which would also give you the opportunity to define a clear return type). I'l type something up quickly.
DooMLoRD, the sysfs fix you are talking about isn't included in your latest (current) acpuclock... is this normal ?
> Given the current reality of how .modulebuildrc mirrors the behavior of the command line and includes the special case of Build.PL, the documentation is handing the user a flawed method of setting install paths. Setting default install paths in Build_PL works just like passing them into perl Build.PL which is the safer way of doing it. 

That this is working at all is contrary to the documentation IMO (though it's not lacking ambiguity) and common sense (if it did essentially the same as \*, why would we have it?). Configuration options for Build_PL should only affect the Build.PL stage AFAIK, and only the actual command-line arguments should affect later stages. I consider this a bug that should be fixed, not something that should be documented as a solution for anything.

Most importantly, .modulebuildrc despite it's name is a file shared by all Build.PL implementations. Having an entry like this that will work in one, yet fail in all the others (possibly including a future version of Module::Build) is a recipe for pain. This sort of stuff *must* be coordinated first, and should probably be updated in the [spec](https://github.com/dagolden/cpan-api-buildpl/blob/master/lib/CPAN/API/BuildPL.pm) before it's updated in Module::Build.

> Setting default install paths at install time silently overrides what the user might have passed to perl Build.PL and that's just wrong. Furthermore, the install paths are not just used by the install target. In a perfect world they should be, but I'll bet that's not the reality for many CPAN modules. Why tempt bugs?

I agree with you there. If I understand correctly what you're doing, you can use an \* pseudo-action in the configuration file to achieve the same thing in a much cleaner way. I would suggest that as a solution (in fact, I would suggest everyone to only use '*' for most .modulebuildrc stuff, but that's another discussion).

> If you want to discuss changing the order of how command line and modulerc defaults are handled to avoid the defaults blowing over perl Build.PL arguments, open an issue about it.

Sounds like a good discussion for Paris.

> We shouldn't screw the user with examples based on how we'd like it to work. They have to reflect how it works right now.

We shouldn't screw them over in the future either just because we've got issues now. We don't want to break their configuration at some point in the future, or worse yet be stuck with even more contortions.
Here the fix, in my Gemfile I need to specify a group to twitter-bootstrap-rails gem, like this :
`gem 'twitter-bootstrap-rails', :group => :assets`
C'est normal, ça devait disparaitre =p
ü
ä
Bem, uma milestone "completa", pensando em add uma issue nela e fechar só pra dizer q ela esta "completa"
Yes it is! :)
Why doesn't CapabilityContext expose an accessor for CapabilityIdentity instead of keeping them separated?
@rkorytkowski: that was fast... there were some edge cases we hadn't 100% decided on the functionality for, and I wanted get you up to speed on the debate.

Do you want me to code review this, or is this still a work in progress?
Should this be testEditedResults? It fails when I change it though.
This is how you should load config parameters into a UserDetailsService
just python rms.py right?

On Fri, Feb 24, 2012 at 12:17 PM, David Fuentes <
reply@reply.github.com
> wrote:

> @jyung
>
> loop over time instances, data drops, model cov to extract rms data
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ImageGuidedTherapyLab/KalmanLocalization/commit/d7bfc84121086bc3d4562a2f5fee6844c5e0481a#commitcomment-1008902
>
I think it might be better to issues a save before verifying the return value of access_control_allow_origin.  This is just testing that Fog::Attributes is working correctly, which is covered elsewhere.
`local_checkout` and reliance on network resources is problematic - we have it a lot, but I don't want to introduce any more of it, especially since this test is not necessarily tied to git.

Instead, can we put the test package directly in `tests/packages` and install it editable using the filesystem path?
if these classes shouldn't be extended just make them final
We should actually put all these under src/main/resources I think
We're confused about what our homepage is, above it's op2.github.com/PyOP2, here it isn't
I'll add a check.
hang on.  If 64 bit atomic support is not in the list of extensions, then we claim we support 64 bit atomics?  What?

I think this should be:

````python
_supports_64b_atomics = cl_khr_int64_base_atomics' in _queue.device.extensions:
````
This will result in a second SQL query. `$seconduserid = $firstuserid;` should suffice, no?
Doesn't it?  base is the top of the graph.  The items in the graph are not built, but the graph structure is.
This should be written in Scala
It would be nice to have a few words, or even better, an example when this tool is useful.
I don't think that should go here. It depends on OS, workload etc.
great, then it was some other combination that breaks it
I have instead put forwarders into `Act` for these things; this is supposed a DSL and one import should give you all of it.
that makes each of the trees local-only (since these Props would not be serializable); just clarifying my understanding that only the top Worker router is going to be spread out across the cluster
Yes, it's better to be explicit. Changed all of these.
use trySuccess instead of the check-then-act?
I don't like suboptimal solutions when the optimal solution is known and only 5-6 lines of code away
I meant that maybe there is a better spot to log the audiophile mode, when opening the sink for example?
The test class doesn't need to be itself a command anymore
in mri_convert from freesurfer, we can set the output orientation of a file to be any of the following:

 https://github.com/nipy/nipype/blob/master/nipype/interfaces/freesurfer/preprocess.py#L162

just trying to make sure i understand you correctly - this mapping that can be set with mri_convert is not the output coordinates of nifti, but simply a reordering of the data blob.
This won't work if you return IPv6 addresses from HostWithPort.
Maybe a log message would be appropriate. I more or less don't think that state is possible at run time and only took it into account for the unit tests
:+1: I like this line
Maybe mention something or rephrase to say they have to configure Apache to use their provided certificates? What I'm going for here is to convey that it's the SSL configuration for Apache they need to change and that Pulp just leverages that. May not be necessary.
nor, this, which was added recently.  why no conflict??
right, message is wrong
What if the DB server goes down? One has to reconnect, and
socket-level logic won't do.
Pewien jestes? Jak bylo samo to zawsze dodawalo kod js, bo serverTime jest zdefiniowane zawsze.. no chyba, ze ten check_conf to jakos magiczniej robi
I used the log with the prefix in cases where I wanted to specify a throwable parameter. From what I've seen (I might be wrong), the logging methods without log. before don't use the right signature.
I thought we should do this for other accordions like activities and the in-jeeqs as well (specifying the max-heights). We can do that in another commit though.
Do we really need this str_cmp ? Where would we need to use this ? any reason we would keep this around ?
That is not a problem at all. Since I fetch from your repo and merge it locally (not using pull request).
what about ``{{ attributes(foo, bar some_var_containing_an_array) }}`` ? It won't be a ``Twig_Node_Expression_Array``
Unfortunately, since `NSAffineTransform` is an object, and `CGAffineTransform` is a struct, it's not that easy, and last I checked, many iOS devs had the same issue.
Would you be able to actually be able to include that in this commit? Then after this is merged, you can just pull upstream into your other branch. I'd feel better about merging this.
This property should be combined with the one for other animated changes.
Shouldn't we `exit` with error here?
Yes, if someone doesn't want a history and does a `chmod 000 ~/.node_history` then we don't want to complain about it. Are there any use cases where we want to display errors when saving the history? I wouldn't mind it being a only displayed with a verbose flag, but I'm not aware of node having verbose logging at this time?
I think that should probably be written `View -> View All Ignored Variables`. Alternatively you could write: Finally, select "View All Ignored Variables" from the "View" menu. Either way, "variables" should have a capital "V"
Debian and also other distros use php5-cgi, so this check is incorrect.
I'd suggest executing ```"$this->phpBinary -v"``` and looking for a string "cgi-fcgi", that might be better solution, maybe?
This case also needs a test.
woops, I was wondering what level was and forgot to remove it.
I wouldn't mention this at all, and certainly not so prominently. It's implied by the 1.y.z version number. If you do want to keep it, ABI compatibility goes back till 1.4.1 at least.
this assignment is not necessary as you can directly refer to `statement.cache...`.
Just being defensive - it's not a performance sensitive section of code so it won't do any harm. I was thinking of situations like this:

``` ruby
API.scope = :api # or it could be an array, e.g: [:api, :v1]

scope API.scope do
  # stuff
end
```

Without the `flatten` you'd have to splat the arguments, e.g:

``` ruby
scope *Array(API.scope) do
  # stuff
end
```

But if you want to take it out I'm okay with that :smile:
I have investigated, at the moment for my opinion without change is more clear. Some logical law could be applied but the code could became minus clear
Since it is internal I think is better to call `@_mail_was_called`
no need to wrap a second line here
I find I don't like to use i.e. anymore on account of bad associations with IE.
That's right, it's a way to ensure these sort of rewritings are safe:

https://github.com/scala/scala/blob/master/src/compiler/scala/tools/nsc/transform/PostErasure.scala#L48

I can't find the one for `hashCode` right now.

This code should document the reasons for this restriction.
OK, then let's go with `scala.concurrent.context.minThreads`.
The ``*`` here isn't correct, it needs to be omitted.
Python users are statistically more likely to be Vim users (sample of 1 here to prove it): https://github.com/kevinw/pyflakes-vim
needs a fail();
I'm not a fan of this type of test for race conditions. It's way too difficult to maintain.
What is the difference between just expanding the series of the desired power in a chosen variable and your function?

```
>>> binomial_expand((a+x)**.3) == series((a+x)**.3, x).expand()
True
```
I see how it could make sense to make transpose O(1) instead of requiring a full copy of the matrix. But that would be a change of behaviour with (probably) consequences elsewhere, and a test failure would be a good reminder to check what these consequences are. So I think it's better to keep the stricter version (with `==`) for now, and only change it when it's actually needed.
I've removed this.
No, I will reintroduce these lines.
On 10.04.2012 21:57, Sergiu Ivanov wrote:
>> +        lcms = [(i, monomial_lcm(sdm_LM(f)[1:], sdm_LM(g)[1:])) for \
>> +                i, (f, g) in enumerate(P)]
>> +        i = min(lcms, key=lambda x:O(x[1]))[0]
>> +        f, g = P.pop(i)
>> +        h = NF(sdm_spoly(f, g, O, K), S, O, K)
>> +        if h:
>> +            S.append(h)
>> +            P.extend((h, f) for f in S if sdm_LM(h)[0] == sdm_LM(f)[0])
>> +            P = prune(P, S, h)
>> +
>> +    # Now interreduce it.
>> +    S = set(tuple(f) for f in S)
>> +    S2 = set(S)
>> +    for a, b in permutations(S, 2):
>> +        A = sdm_LM(list(a))
>> +        B = sdm_LM(list(b))
>
> All right, this is clear.  You could perhaps mark this with another TODO comment (unless the scope of the comment about better data structures extends hitherto).
>

I wrote another TODO.

> Also, I'd vote for not reusing variable names, like in 479.  It's very likely just me, but I initially thought that `S2` was created from the _initial_ `S`, which consisted of lists.  (Feel free to disregard this remark, of course.)
>

IMHO this is fine...

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1190/files#r667672
>
```

>>> from sympy.utilities.iterables import partitions
>>> [p.copy() for p in partitions(4)]
[{4: 1}, {1: 1, 3: 1}, {2: 2}, {1: 2, 2: 1}, {1: 4}]
```
I have already updated this, I just haven't pushed the updated branch because I have played with `git rebase` a little bit so I didn't want to mess up the comments.

Also note that if we do transition to a more complex class structure as discussed in [this thread](https://groups.google.com/d/topic/sympy/NAsIl5hziC8/discussion), `Dummy` will not be used at all, since anonymous `Object`'s are not supported any more and the same fate is going to happen to anonymous `Morphism`'s.
Possibly, I don't understand to class instance variables well ... but I'd expect that self.class.lens returns nil
Please don't use YaST indentation here. I feel it is really ugly even in YaST. for ruby is convention to use two spaces and no tabs at all. See - http://craigrandall.net/archives/2006/09/ruby-coding-stds-etc/
It is test for "stability" of quoting (in other words: do not overwrite unchanged values) when there is no change in value.

:serialize is what the agent gets from the file (before unquoting and so on)
:prepare_write is what the agent wants to store (after quoting)

so, data flow of the test is read file -> unpack (remove quotes, escapes) -> pack -> compare if the result is same as original file content.
Why do not use <code>ReflectionMethod::isPublic()<code> here?
oops, sorry confused it with ssh variant `git@github.com/...`. all okay here.
Field names are fixed, there is no need to do any lookup.
@zrax the `RemoveTarget` is outside the `if`. This only affects if we set fLight to nil.
What that line does: add certain HTML5 tags that could not be CSS'ed in IE < 9.
I needed it because I used those tags, here it isn't needed.
Is it possible to just override this display attribute for the .marked .field_with_errors case (since it is the only thing that is different?)?  (instead of redefining padding and background color)
Did the return key get stuck?
Is this explosion still required?
To avoid mutability, could this be a constructor parameter too, with a default of "utf-8"?
s/for.*$/to migrate chunks manually./
We can live without nesting. If it takes a few minutes, great, but otherwise no big deal.
django comes before south?
+1 for `-n`, and I’d also add a dash in `--dry-run`
Probably because initially I had them and then I took them out. But I will put them back.
I think I agree, feel free to fix.
The "old" way.
If the function returns false nothing should be done.
Also I think this function could just use set_lang with a custom path?
true and false are not needed here, logical operators in php return booleans.
These are uppercase to test whether the checks for disallowed content are properly case-insensitive.
actually use a branding constant instead of "Cloud Monitoring Agent" but yea
Weren't we taking out the "withPrivileges noPrivs"?
I'd say keep them inside the definitions. That way they're self-contained; the primes are part of the definition of the hashing algorithms. Their coincidence needn't be encoded.
Deze commit zit niet in de tree?
That is just a semi-random number (date & time)  I added to make a key that would invalidate the cache.
consider renaming all "up" to "post" or "create"
From what I was told, the topbar was to be common through all the modules.
Removed. (And refactored the previous lines for clarity.)
Fixed in gh-1119.
These formulae are easier to find in textbooks on quantum mechanics than in textbooks on linear algebra, but here is a link to lecture notes that are quite readable:

http://www.cs.nyu.edu/courses/fall06/G22.2112-001/linearAlgebra1.pdf

the only reason I have not added the link in the docstring already is that I could not figure the author for proper attribution.
The statement is valid as is--we can't differentiate something that is never a number, and null is never a number (or anything else).
Yes, it's a very shallow copy (two pointers). And I agree with you that signature should take a "const TableTuple&". The only problem changing it is that it will cause a ripple effect - there are few other methods called from the insertTuple method which accepts the tuple by non-const reference as a parameter. 
The other possibilities are to change the signature to take the tuple  simply by value since copying is cheap
or, as you noted, simply remove the comment
under this line comment was added
user = kwargs.get('user')
cast layer parameter to integer once, and just == in all comparisons for simplicity
wonderful rewrite... this is how I can learn by example
I couldn't find a pattern, but it wasn't a matter of unloading apps with sticky dirs like kernel.  It can happen for any of the Riak dependency apps.
right, but that's just there to facilitate the tests which tests if unpack raises when it unpacks a non-existent tarfile. it's just an API hack, but I can remove it and find another way if you'd prefer.
i've merged already, let's go current way
I agree with you. I will rename it.
Please add to the comment that the reason it doesn't delete by name is that it uses introspection.
It'd be nice to add some explanation (or a link to some explanation) of what 'MRO' is and means.
'models.User' prefix is unnecessary through the attributes and methods as this path is established by the currentmodule and class annotations
@nashby  ah such a long story https://github.com/rails/rails/pull/6549
Okay, now you're just undoing stuff that is good and makes sense... Check for the default value _in addition_ to ensuring that errors are tracked correctly, lol
It'd be better to always throw an exception here and get rid of the `exitOnError` and `abortOnError` flags.
This actually changes the semantics of the code; we can't accept this without a test that supports the change.
Seems like you're right; the tag filter has a guard for that.
No else need to do. Just taken the non-null one. Already add checking for duplicate.
You can just name the class TestCase instead of giving it two names.
I think `u''` should be treated the same as `None` in this function, a fallback value.
Looks like some cruft slipped in here.
I've updated the script to reflect these changes.
What exactly do these patches do? Why are they needed? If they are critical, have they been submitted upstream?

We need to know this stuff in the comments so that future maintainers can make informed decisions about removing the patches.
Can we get rid of this file please?
I think the base class already sets a suitable timeout?
keep an eye on line length (re: PEP8)
USe dependency injection for javaCache + refactor the modification-time checking out?
Make that e30569fa7d8776a77a9348ad1886aa599f755cfb
use a counter for each of these? Then you get predictable sets: 1,2  1,3  1,4
yuck?
Unlike most of the tests in this file, this belongs in libcore and not OkHttp.
既然不会忽略那就更应该用`~/.Renviron`啊，修改程序主文件是非常非常坏的坏习惯。不可持续发展。
@dsaff do you think we need "since" tags for internal classes?