labofsahil/github-event-dataset-2012 huggingface dataset:
[DON'T MERGE] Stronger use of CacheCache

Analysing #949 i found out that most of the time (~98%) is spent for the sequence cgData.loadCache() + cgData.storeCache() for each cache retrieved by the search (by viewport).

This search returns only rudimentary informations about each cache. Information that is not worth to be saved in the database BUT must be stored there because of the architecture of c:geo (intents use the geocode and not a cache object e.g.).

Instead of really storing and loading each minimalistic cache into and from the database (what is very, veeery, vvvvvvvvveeeeeeerrrrrrrrryyyyyyy slow) i tried to use more often the CacheCache - with success. The in-memory-db cache is now controlled fully by cgData. The advantages are:
- less DB operations
- better overall performance
- smaller DB. Only fully detailed caches are stored now.

Please have a look at the changes in cgeoapplication and cgData. These changes could be merged into the market release to solve #949.
As further steps i will move CacheCache into an inner class of cgData and will have a look at ParseResult (if it is still needed).
Loose the regex requirements for recognising functions that are to be available in a remote proxy.

    Line 328: Allow multiple spaces (or tabs) before "@remote" on a single line
    Line 329: Allow you to put "@remote" (or other metadata) on the same line as a function declaration.
Only measure the start and end time of an OFMessage if the processing…

...me service is enabled.
added new bounded tests

added three new tests - one for testing within a particular range (inclusive), one for testing if an output value is greater than or equal to a test value, and another or testing if an output value is less than or equal to a test value
Atualizado pcre

Signed-off-by: Matheus Macabu <mkbu95@gmail.com>
Followup fix for: Create a 'Manage Templates' admin page for the hl7query table - HLQRY-11

Followup fix for: Create a 'Manage Templates' admin page for the hl7query table - HLQRY-11
RFC: Custom block plugin system

This is just a proof of concept to try and get the ball rolling. The render definition for wool has been moved out of textures.py into a seperate file, which you can then add back in the config file using customblocks = ['wool']. This tries to render red wool using a custom texture, which I placed in /tmp/wool.png - you can download the file I used from http://sucs.org/~tswsl1989/texture-test.png

If you try this, then the texture will be visible, but very dark and hard to distinguish. I'm not sure if this darkening is the issue with the other wool colours.

Suggestions/comments?
Retrieve the driver classname when using the cloud environment

Retrieve the driver classname when using the cloud environment
Add Travis CI build status and Gemnasium dependency status images to the README

Looks like you're a rbx-2.0 build and a mysql2 dependency away from going green!
fix invoke_editor calls in edit-command

edit-command always reloads the file specified after it is edited.
Implemented directives for event and access log rotation

Can I also get push access?
Итератор для провайдеров данных.
Add an iVar to track the search mode before/after quicklooking

The `savedSearchMode` iVar is used to return Quicksilver to the right search mode after the Quick Look panel is closed.

This has been changed following the discussion on the dev groups.


P.S. I remembered to change the pull request so this merges to quicksilver:release :-)
Java and C++ snippets

Here are some Java snippets that I find handy for output and comments. Most of them are inspired by the default ones in Eclipse. There are also four C++ snippets in a similar fashion. I believe others can find these useful too.
Rails 3.1.3 was throwing error, path was not being added to sprockets load path.  This fixes it.

/opt/ruby-enterprise-1.8.7-2011.03/bin/ruby /opt/ruby-enterprise-1.8.7-2011.03/lib/ruby/gems/1.8/bin/rake assets:precompile:all RAILS_ENV=production RAILS_GROUPS=assets
rake aborted!
couldn't find file 'flash.js'
  (in /u/apps/can_send/dev/app/assets/javascripts/application.js:13)

Tasks: TOP => assets:precompile:primary
(See full trace by running task with --trace)
rake aborted!
Command failed with status (1): [/opt/ruby-enterprise-1.8.7-2011.03/bin/rub...]

Tasks: TOP => assets:precompile
(See full trace by running task with --trace)
Added authentication to TMongoWire

Hi,

First of all thank you for your work on this library. I needed authentication for my application so I decided to implement it.

I added an Authenticate method to TMongoWire class which requires the user to provide the database to authenticate for, an username and a password.

Because authentication needs MD5 I added an MD5 library by mfichtner@fichtner-meyer.com to the repository. I altered to library so it doesn't cause any collision for users already using the library. It only exports one method MongoMD5Hex which returns an hexadecimal string representation of the hash. 

Kind Regards,

Fred Oranje
Update from eskerda/owncloud-submedia-app
Merging sailthru:master into bbartholomew:master
Uncaught exception fix (InsertLink/Image)

Small update to fix uncaught exceptions as reported on issue #138

This fix simply gives a focus to the editor's body before executing createLink or insertImage functions.
Homework for July 9

I'm finished with the homework sections, but I might play with the code some more and see if I can figure out some stuff on my own. Maybe.
Failing test case for #514

Ref #514
Fixed some translations + added new strings to german translation

Fixed some translations and added new strings to the german translation propertie
Added HotKey feature

for link, format and wai-plugin; manual hotkey testpage; updated guides;
Redesign?

Hi Tim,

Thought it'd be fun to give the howtonode.org site a bit a re-design. Only did it for my own kicks so no love lost if you don't merge, but thought it'd be silly not to let you know about it.

Cheers
Management commands

I fixed a few typos and added more functionality to the flag management command.

New options:
* --percent, -p [int]
* --superusers
* --staff
* --authenticated
* --rollout, -r
* --create

Updated --everyone and --deactivate to store their values in 'everyone', if you don't pass either of these, the value will be set to None (unknown in the admin).
Задание первой лекции

Исправила код с помощью http://jslint.com/. Возникло несколько
трудностей, с которыми я не смогла справиться. Посмотрите, пожалуйста.
Most recent merger.
Update advanced/debugging/wiener_filtering.py

I'm not sure,but I think it should be changed to work under scipy0.11
Исправления по замечаниям.
Energy projectiles now move instantly, leaving a straight line from their origin to their destination, like real energy weapons. 

Welderbackpack now has a proper icon.
Stun rounds can now be reduced or blocked by armor
Added more update_clothing() calls
Fixed strangling non-humans.
Blob is now an announcement only 67% of the time.
Map updates.
Invalid HTML problems

You have a few places with invalid HTML; these changes should fix that.
minor changes

Add a command to toggle "enabled" setting via quick panel.

And adds some minor changes to follow package style.
Basic Saving Product Creation,Updation

This commit contains basic saving product's creation and update from Client app.
Russian Language

Add Russian language file
my presentation for apr 2012 #2
Use 'past' and 'future' date validation between multiple DatePicker fields.

Adding suggested enhancement #358, Use 'past' and 'future' date validation between multiple DatePicker fields.

First time I do this branching stuff and pull request. Hope I'm doing it well.

Julien.
Added the possibility for the user to add the 'onShow', and 'onHide' options, which are functions that execute before and after the popover is shown

Hi,

I thought it would be nice to allow the users to add a function that is executed right before the popover is shown, and another one that is executed right after the popover is closed. For example, in my project, I had to perform an AJAX call to display the content of the popover, so a function like this was crucial.

I implemented these changes, by calling the onShow parameter function at the beginning of the show function, and the onHide parameter at the end of the hide function.

Best regards,
Andrei
Use AppState instead of Session.

Since resources are normally application-wide, they can be stored in AppState instead of Session. This allows the script manager to work even when session variables are disabled in web.config.
Fixed is_plugin_enabled() was deprecated by elgg_is_active_plugin()
Buy art

* [Stripe button docs](https://stripe.com/docs/checkout)
New Member, Hello!

Added myself to the student roster and re-alphabetized the student roaster so will are all in order by first name.
DNS and IP entries added to subjectAltNames x509 extensions


Added accessors for ips and dns_names to the object, changed 'to_s' accordingly. Confirm my assumptions in the specs, though, I'm not an expert on this.
Minor update to Revision3 to handle SD and 480p videos which are not available and fix for CNN-News

It appears that some old Archive content is not available in all three formats. Therefore, in these scenarios, it will simply choose the lowest quality available. 

i.e. SD -> 480p, 480p -> SD

Also includes a fix for the CNN-News service which is required by the Channel
Pull Req: Remove BBX-Emulator and exclude node_modules in maven artifact 

- Removes blackberry-webworks/BBX-Emulator submodule and it's references
- Remove executable which starts desktop webview (bin/bbx-framework)
- Jake build scripts no longer copies node_modules to target folder where maven artifact is generated
Pull Req: Added low and critical entries to battery events.

This pull request has implementation of feature blackberry-webworks/BB10-Webworks-API#6

Another pull request for doc created and can be found here: https://github.com/blackberry-webworks/WebWorks-API-Docs/pull/101
Enable wikipedia river throttling

A few small enhancements for wikipedia river.
Retry pull request...

- Set ES version to 0.19.3 
- In ServletRestChannel#sendResponse(...) the contentLength is only increment if prefixContentLength or suffixContentLength are set. 
- ServletRestRequest#uri() also includes the query string if available in uri.
Group the reports with other Time Tracker modules

Try to group the reports module with all the other Time Tracker modules.
Jellybean

Fixed resolutions to include anything MDPI on up. I also removed No Wallpapers because CMWallpapers already provides it. More info in my commit messages
a few small errors

Hi guys, here are a few nits in the web pages.
IDENTITY_INSERT error on Table with no IDENTITY column

Hi.  I get the following error when using the :odbc mode with a table that has a primary key but no IDENTITY column.  

    IDENTITY_INSERT could not be turned OFF for table [TheTableName] (ActiveRecord::ActiveRecordError)

is_identity? seems to return a Fixnum with a value of 0 or 1.  The issue seems to be caused both values evaluating as true.  is_identity?.nil? is always false regardless of whether is_identity? returns 0 or 1.  The issue was observed on Ruby 1.9.3 running on Windows 7 64bit connecting to a local SQL Server 2008 R2 database using the adapter's odbc mode (needed to get integrated security working using the Windows user account the ruby process was running under).
Document that MongoDB implementation is incomplete

Let's document this fact to save folks time messing with this
if they use MongoDB.
Fixed missing semicolon on line 164
Guys, please make sure you don't remove the last suffix from application.css.scss again. This may not be relevant in development, but it kills the asset pipeline and makes everything go awry in production.
LGTM. One change before I accept this, can you return your call to `this.raw` to enable the ability to chain?
It may be a private method, but it's also reusable -- so it should definitely be documented. However, if there is the possibility of setting a flag so that this docstring doesn't show up when the API docs are generated, we should do that.
It also breaks the ability to merge in another branch and manually discard all of its changes.  (I.e., you can't commit if the merge result turns out being identical to the previous HEAD.)  That's even more useful.

And I don't think there's a way to toggle "allow-empty" without popping up the log-edit window *first*.
(nejen) ternarni operator se vyhodnocuje line, takze se z nej vykona jen patricna cast; a fetchWord se nezavola
V masteru by nemel byt force. Respektive by imo nemelo byt v release vetvich, coz master je release pro 2.1....
Thanks for the heads up! Do you need me to make a new request?
`msg_channel_` should be deleted in the destructor.
get it reviewed first
Looks fine, apart from my two comments.
Please use  acceptable default values
1. наверное здесь также предполагалось использование "!"
2. скобки здесь явно улучшат читабельность (разделят где начинаются аргументы метода)
I also added some files of the Lua Script editor, that I'm going to implement.
;) thanks!
百度翻译“我的作品”，就是MyWorks
what's the difference between this and javascript_dom_paste ?
`-fheinous-gnu-extensions`?!?!?

That has to be the most epic name for a compiler flag that I have ever seen.
(Wrong class name)
The python style guide mandates that the high level description should only be a single line.
What's the scenario that this is covering?
You're clobbering the ret value that we intend to return here -- you don't want to do that.  Since this is about logging something verbose why not just call *kafs_verbose() with a constant string like "out of memory" when the asprintf() fails?
use Options for these vars, and then lines 45 through 51 can be rewritten:

lastOfflineException orElse lastNormalException orElse lastBlackHoleException foreach { e =>
  throw e
}
Thanks, +1 for you.
I don't know how to proceed, I am sorry. I thought I had commit rights so I could help you out, but I don't on this project. (Let's just wait until one of the owners wakes up :) )

As an idea, use `pod spec lint QuickDialog.podspec` to test the spec :)
This later became `flipsign`. Will try it.
Yeah, I'll change it and it should work fine without the hack.
yeah, but we are moving into the front end stage soon, so you could start helping out there.
Just think about it
That one is picked up as well; seen it work :)
What? Lifebloom does not have charges.
Dispel removes lifebloom stacks.
>у меня сегодя в почте 4 письма с воплем "мобы проваливаюЦЦА!". лучше пусть сервер вообще не запускается чем, стоПицотый раз объяснять...

ужас.
I think thats not related to issue, because i have fix only the warning. Iriskin say that compiler make automatic brackets. If we have the following logic
a || b && c
compiler add intern brackets
a||(b&&c)
The `return decorator` is still needed, but the else isn't.
Thanks for caring about the PowerPC port!
Good point, I was mainly trying to follow the current convention and not break anything :)
I think this condition statement should be like this.
Is it an typo? or Just as your intention? (I mean like -> (!leftC.view.hidden || !leftC.view.hidden)???)

```objective-c
}else if (leftWasHidden && rightWasHidden && (!self.leftController.view.hidden || !self.rightController.view.hidden)) {
```
This seems to be the location of bug mentioned before.
Reproduction steps:

1. change any value besides Thickness and Length
2. watch the reseed reseeding to different values than before

This might be to do that there is no reseed done on loading of xml (just a guess)
In addition to this modification i want explain the reason of this hack.
The problem was the impossibility to use the function convertToDatabaseValueSQL in a custom type on insert.
Did you try stock Snes9x 1.53 or just your tree? I think if you check you'll find the jump noise is missing in vanilla Snes9x.
With regards to the checksum - this is what SNES9x gives me back - according to Maister, SNES9x pre-strips the header before giving you back the CRC - so I assume this will suffice - 

"DIDDY'S KONG QUEST" [checksum ok] HiROM, 32Mbits, ROM+RAM+BAT, NTSC, SRAM:16Kbits, ID:ADNE, CRC32:006364DB
We need some way to confirm that the server has gone offline and then restarted, otherwise we'll proceed to the post update page immediately.
It doesn't have to be an int counter though.
alignment of the code is a bit messed up here. looks ok on my computer. Any solutions?
This is dirty, since it can cause multiply defined symbols if some header takes winsock in before these. The warning needs to be removed in a cleaner manner.
Oh, yes, sorry, it's weird I didn't get a notification or anything, sorry!
Apple deprecated OpenSSL in Lion; this prevents the build from failing when OpenSSL headers are #included.
also \@ see <a href="doc link" />
@felipensp we need to care about these todos... Here is supposed to retrieve the types of a list.
We can use the getNumArgs() method to know how much template args the type accepts.
Oops; my bad. Sorry about that.
Dear Clemens 

I just tried to merge your branch and changes into spreadsheet. It did not work. What branch shall I pull to merge in your changes?

I am getting a merge conflict for:

lib/spreadsheet/excel/writer/worksheet.rb

how did you update and commit that file?

Best
Zeno
Edit: Started with rrouse's fork (link below), and made this change to octopus_shard in abstract_adapter.  
https://github.com/theymaybecoders/octopus/commit/d9e9141423876fe70bb879607dbaae913b525438
https://github.com/LearnBoost/mongoose/commit/475368e5be8b91d9d03df487e61fc1d9908edbfa
check nulls and watch format
I think it would be preferable to avoid the existing public object altogether, so that it can be removed when the deprecation expires.
just an observation.. maybe this is better:
    LPC_GPIO1->FIODIR = LED1 | LED2 | LED3 | LED4;
This is a first stab at including support for my new pyepg app which attempts to do a better job of providing listings data than XMLTV. Specifically it has significant improvements on how the data is identified.

Currently I've bodged this in to use a non-existent exmltv format (I see this as a stepping stone format, it will be xmltv-like with some extra fields). However I think to do this properly I need to rework the entire grabber/epg code.
Thanks! My bad
`$singular_values` is not defined. Guess this should be `$singular_rules` instead (2 x).
what is this for?

what do you want to do with that?
`lstrip` is not the right method. 


    S.lstrip([chars]) -> string or unicode                                                                                                                                                                                                    
    Return a copy of the string S with leading whitespace removed.
    If chars is given and not None, remove characters in chars instead.
    If chars is unicode, S will be converted to unicode before stripping

Look at these examples:

    >>> 'Foo Bar'.lstrip('Fo ')
    'Bar'

    >>> 'cGZjLXNpdGU6MTIzLWFiYw=='.strip('Basic ')
    'GZjLXNpdGU6MTIzLWFiYw=='
Awesome. Thanks for the quick response!
The name means "log level", but the variable is used just for one purpose
Need comment here.
I don't understand.  Are you setting balance to 0 if the patient owes something?  Need comments.
Da haste recht
Yes, but the current upgrade code doesn't have a UI for choosing. We have started keeping all the old versions in GitHub though.

Back to the case in hand, I'll talk to UnaMesa about the CORS issue.
Mark, why this project is discontinued?
This feels like a cleaner alternative. No if statement.  Now using shellescape() to correctly wrap arguments that have spaces in them with double quotes.

Note: shellescape() needs to be called for each argument rather than the whole makeprg statement so that it knows which spaces are argument delimiters and which spaces are supposed to belong to an individual argument.
There is no any reason to use a form here. We are already in another form, and form in another form is not allowed.
We should probably change this to point to a URL with more information about the project.
is returning true necessary? does it just have to be not falsy? maybe...

``` ruby
unless self.rubygem = Rubygem.find_by_name(gem_dependency.name)
   self.unresolved_name = gem_dependency.name
end
```
No my manifest is not the same as this one and yes I did edit it in my github repo manifest for AOKP but thanks for suggestion since I already did that?  Your quoting things that cm members use, good for you you can copy and paste.  Roman has the final say in what manifest I use and what works for me?  I don't think so.  Why not just make another branch at least for people to use if they have issues?  It doesn't seem like that big of a request but you act like I'm asking for a million dollars here.  I'm done responding to you, obviously Roman can not even comment on his own commit discussions so what's the point?  I offered my forked manifest to others having issues if you don't like it too bad?
Nope - didn't know that.  I use ',' for my leader. :(
Unrelated to this (specific) commit, but can you try to keep the subjects a little shorter? They're hard to read, both in the Git log and here.  https://github.com/blog/926-shiny-new-commit-styles has some more details.
But...But...But...

Why didn't my IDE push this...RAGE
and cap deploy:setup ?

On Thu, Jan 5, 2012 at 11:53 PM, David Y. Ross <
reply@reply.github.com
> wrote:

> Very strange.
>
> $ [14:51:34]> ls
> Capfile app     conf    config  lib     modules public  run.sh  stop.sh
> test
> $ [14:51:41]> cap --version
> Capistrano v2.9.0
> $ [14:51:44]> cap setup
> the task `setup' does not exist
>
> Further, I don't see in the recipes how you tell play to put it's logs
> there...
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/dyross/play-capistrano/commit/97154f1c9ea0d7d276875b2f0c92fd6ac40ecdc8#commitcomment-841969
>
Thanks for this - looks like it has solved my issue really nicely!

Much appreciated!
This last commit is actually fairly important, see how everything decomposes into these tiny methods? Everything is very clear and we have defined an interface where you do not need to know the details of the anagramlist object in order to use it. It is becoming beautiful.
Max Health is a terrible doctor. 

... >_>
Python 2.4 is not available on travis-ci.org and won't be. Please see [the list of available versions](http://about.travis-ci.org/docs/user/languages/python/) in our documentation.
On 8 March 2012 18:16, Ross Patterson
<reply@reply.github.com>
wrote:
>> Unfortunately that's a skin template rather than a view
>
> I believe the AT base_edit form looks up a `@@at_base_edit_view` view in the TAL.  Maybe that could be used.

That's too late as IPubAfterTraverse is called before the template is
executed (we need to set the correct skin by that point too.)

Laurence
Did you mean targetNode?
Sikker på at denne funker? Oppgrader-boksen ligger nå godt synlig helt nederst på siden hos meg i splitter ny Chromium...
@Dreadii who said that dynamic LoS is only useful on DS and RV? you have been missinformed if you think that way
i didn't expected to add this on main repo, i am shocked.
@Ussmaan beacuse you will extract vmap info for go, so with this arenas like dalaran sewers who uses dynamics go will workfully in the near future :D (and also without mention that the others file have been modified). so if you dont reextract them, the core will not start properly.
Fixing what? because I think I did already, didn't commit jet
This closes #16.
Just keeping the nodeitems and the arcitems as fields is sufficient. The nodes and arcs can be accessed through them. This is a kind of code duplication since you duplicate the list structure :-)
Yes thats a good point. Thank you for pointing it out. Coincidentally I was looking at the Fragment documentation this morning(http://developer.android.com/reference/android/app/Fragment.html) and realized that I need to do my fragment UI stuff in the onActivityCreated method.  :)
How can I make it stop inserting spaces before the `:`s and just align the values? I've tried `"alignment_space_chars": []` in my syntax-specific settings and it has no effect.
If you change this line to `var $container` the problem should be fixed without other changes. The problem is $container is referenced but never defined, so it falls to the global scope.
I don't believe that main_app.url_for is correct.  It looks like it should be main_app.refinery_blog_post_path(@post.prev).  I'm not sure is there is another way to do this so I wanted to get your comments before posting a fix.
Oh! If you tried to send it as an attachment or something, it didn't take. Feel free to file an issue or e-mail the logs to me at zwaldowski@gmail.com.
No No No, we need more boops!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
> wow ! x30 speed up with brain grids?

Only if they are very noisy.

G
Ah yes, now I can see it thanks!
That's ugly. If possible can you not just detect in the click whether keyup fired just before it.

This issue is that in opera both keyup and click would fire. So gaurding againts it in the click handler is a more elegant solution.
indeed. Thus, using ``TYPE_OPTIMIZE`` would not ensure your pass runs after the optimization :)
gibt es eine Methode oder ein helper, mit dem wir das Datumsformat anpassen können?
Apparently this worked, or at least I got the right link after my test feed refreshed!
It might be not an event thread when calling getActiveScreen. Use invokeLater or acquire an even lock when accessing to ui from outside of event handling or drawing code. 
See the *PageManager.java* for reference.
Shouldn't this be 

```js
self.commands.push({type:'query', 'db_command':db_command, 'options':options, 'callback':callback});
```

?
Hi,

Maybe you could use this little sweet-heart over here:
https://github.com/ggozad/jarn.jsi18n

On Wed, Feb 15, 2012 at 6:24 PM, Leonardo J .Caballero G. <
reply@reply.github.com
> wrote:

> Yes I know but this is temporal solution check out this branch
> https://github.com/collective/collective.upload/tree/jquery-i18n it
> trying to add support i18n to a script jquery maybe can help me!!!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/collective/collective.upload/commit/2296bd92fb2a53d6ed79e2917d9dcdf5a36be802#commitcomment-969968
>
Ok, I think we have enough textures... Please no more O_O
I'm reviewing this right now. Hold, please.
Make sure you're null-safe here.
Зачем threadList хранить в MVar?
There is a problem:

http://subir-imagenes.es/images/sinttuqlq.png
i really don't understand... putting usb in the systemui process, breaks what for what build?
in my testing, nothing seems broken

it is important to me, because i hard try to test before put something to the mainline :)
but if my tests are compromised, i had to know and to find workaround ....
Hey, make sure you fork my system_vold also since it has the code change necessary to execute the volume swap.
The terminology standardised on in WordPress is update, not upgrade.
renamed to CompositeAnnotationSourceManager
Not a good idea to be sending output here, could break lots of things, esp. downloads. Tie into the `[view, page/default]` and `[view, page/admin]` hooks and add the PRE directly before the end HTML token. This means you wont capture delayed, but I think that's OK for phase 1. 

These metrics should eventually be put in the page as JSON so we can style them/build a debug bar, etc.
I think some of the logic in this class and `AuthenticationErrorMessageResolver` can be shared via an abstract parent. Structurally, they seem very similar. The construction and declaration of `MESSAGE_MAP` and `resolve` are potentially the same with minor variances.
Good. Will do.
Same with scala.collection.Seq() then
But they don't do the same. So won't work
The point is not that "flow method compose flow-expressions", but "flow-expressions can be composed", which is in CS jargon simply used as "flow-expressions compose". See analogue: "applicatives compose".
I think it would improve understandability anyway to use Terminated(`reader`)
ok, yes, I just added it because it was treated like that in `convergence`, but all that will be revisited in the other ticket
You mean 'updateConsistentHash'? Misspelling.
So if I'm not mistaken this should only be an issue with long version like `x.y.z-RCX`, and `cross CrossVersion.full` which is only needed for Scala Milestones and RCs. Doc for normal releases will look funky I think.
public API? seems like an utility method that should go somewhere?
ah, I see now what makes re-appearing from UNREACHABLE a challenge; was this not supposed to happen only upon DOWN? @patriknw ?
So you tell me there is no Santa?
this makes this harder to read imo
Did not work for me. I don't have vtk installed on my laptop, so I can't
run the examples.


On Wed, Dec 12, 2012 at 8:01 PM, Eleftherios Garyfallidis <
notifications@github.com> wrote:

> In doc/examples/nii_2_tracks.py:
>
> >
> > -**This example gives a tour of some simple features of dipy.**
> > +We will show how to generete streamlines with Dipy. First we will reconstruct
>
> @arokem <https://github.com/arokem> Sorry, I mean to run the examples.
> And/or generate the html? Did it work for you?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nipy/dipy/pull/103/files#r2402321>.
>
>
Do you think new_authz should have a function for updating the config permissions, or we should have a logic action function for it? Or are we happy exposing this dict? If so maybe add a docstring to the dict in ckan/new_authz.py.
I don't like these continuations in a string can you not do some thing more like the following even if it is ugly too

str = '....' + \
   '...'

srt = ('...'
           '...').format(..)
should work too
This section is really solid. It gets the point across without being too verbose and I think once the docs page is up it'll finish up hammering the point home.
I thought about using the recorder, but I'm not sure what the "tags" are that you mention.

I went with this choice because as far as I could tell (which may be wrong since I didn't know about this tag feature), I would have to iterate over the lines in the recorder and search the text for something that looked like a failure. Such text would of course be subject to change since it's just a message for the user. The mocking I did is very explicit in identifying that an error happened and was reported to the user, irrespective of what text is displayed.

Thoughts on that? Perhaps this tagging concept is something that would be useful here?
Same as above. I could see storing it as a static value, but we'll also want to have it as a publish-time option.
Забыл скобочки, если больше одного оператора у внутреннего оператора, скобочки ставим.

Второе, функция glob:
Возвращает массив, который содержит совпадающие файлы/директории, пустой массив в случае отсутствия совпадения или FALSE в случае ошибки.
На некоторых системах невозможно отличить отсутствие совпадения и ошибку. 
Это конечно жесть php, что выдаст неизвестно =) Но наверно лучше делать так:
if (empty($files)) - тоесть, если не равно FALSE или array(), в остальных случаях должен быть пустой массив.
Maybe we should add a line to this test :
slugged_text = slug(u'Álvaro Justen'.encode('latin-1'), 'latin-1')

just to make sure... it's an encoding we'll be dealing with a lot...
Same concern here. Expected a worker to start a request, but would have done nothing
why not also pick up the `ec` implicitly by currying the argument list?
There is no ``only`` argument but a ``with_context`` argument
No ivars please! Switch to private properties. (No need for `@synthesize` either....)
But I'm not 100% sure what these constants mean. I know it's a speed, but *why* is it 0.08 is my question. I'll still pull them out if I can....
@jwilling The reason I set it in the drawing method was so that the "private graphics renderer" would be isolated from any subclasses interested in drawing. What we *could* do is check if the properties were "dirtied" and if they were, then set everything again.
I think we are considering `err.code` the preferred API
Yeah, that's a good idea.  exit should make nextTick a no-op.
You shouldn't use ``assert``, because it will get removed when byte-compiling with ``-O``. Use instead

    if not isinstance .... :
        raise ValueError("`n` should be integer")
`addOutput` should be used for this dialog, otherwise it cannot be added as bread crumb.
I have change it back
You're right! Changes coming
@ joshuap We don't use ```and``` . Can you change it please?
also, s/so that YAML Parser/so that the YAML parser
That seems to specific a test.  I'd rather avoid impl details tests and go for properties the api should abide by.
That might take an expert hand to wire it into the fast track mechanism. Let's do that separately from this PR.
Could you propose a wording? I thought I have it covered with "This margin is stripped '''before''' the arguments are interpolated into to string."
So is the optimizer now happy with one-branch switches?
If you aren't going to implement the global show=False for the test runner yet, at least add a little blurb to the docstring explaining why this is here.
Now that I'm reading on, I'd really like to ask you to use `expr` and `pattern` instead of `x` and `y`.  I'll cope with the current version nevertheless, but `x` and `y` is just too generic to make reading comfortable.  Feel free to disregard this remark, however, if you think I'm overly picky.
In fact, why do you need this? Shouldn't basic provide these methods?
Is this heading towards a regression? Tuple is just a container and presently will hold anything that is sympifiable and it presently supports the has method.

```
>>> type(Tuple(f)[0])
<class 'sympy.core.function.UndefinedFunction'>
>>> Tuple(f).has(f)
True
```

And with the current changes you now get

```
>>> Tuple(f)
(f,)
>>> _.has(f)
False
```

We should put a deprecation warning in has to check for non-Basic args so we aren't allowing the error (to find it) pass silently.
The `not isinstance` block will never be run.  `list` either returns a list or raises an exception. It never returns something that is not a list.

Also, why does it need to be a list in the first place?  It looks like all you're doing is iterating through the elements.  That will work just the same if a tuple is left as a tuple.
This should imply real, not integer. Consider
```
x = sqrt(2)*i
x**2
```
x is imaginary and x**2 is real, not an integer
should I make it a class method then? Yes, that seems the right way to go about it...
You should change the style of the docstring to match the other comments in sympy.physics.mechanics. See the inertia function as an example (or really any of them).
r_factor appears to be defined but not used...did you intend to use it around L195ff?
I figured so...
You didn't reply regarding the use of  `@threaded`.
Can you add the `sympy.` back in? I know it does not really matter, but I don't see why we should break the pattern either.
It is not a float, it is a function returning a float. Check the "coloring" example notebook. Another docstring already explains all this, just add in parentheses a reference to it (either Plot or Series).
Ok now I see why collection
Oh cool I didn't know you could do this with `assertRaises`!
typo? ", ,"
s/instantaneous/instant/
`display_name` under `'`
Instead of `request.get_host()` which is based on client input, please use `settings.SITE_URL` which also defines the protocol.
typo? i think you meant environment?
here too!!
I probably forgot to remove them.
I tried this, but it does not seem to work.
Please discuss that on the RFC. I already have, if that helps.
Missing ^ and $.
Then again, this doesn't really reduce the length of this line.
Note: this ID needs to be unique, so include the session key or ID in it.
Is configuration from this file merged with some other configuration?
so the problem with this approach is asynchronisity of node. the reason we nest callbacks is to ensure that we have an object before doing the next steps. this approach above is error-prone because you cannot guarantee that `users` is set before you continue to do the processing at line 14. you have to nest these callbacks.

eg. 

    users = User.find({}).populate('team').asc('username').run (error, users) ->
      throw error if error
      if @request.params.format is 'json'
        ...
Please add that information to a comment.  We really should circle back and do it the right way (once the right way is available).
I added the extension explicitly to trigger the error if they try to access something else, but later on I added a verification of the extension, so I can use this structure now. Should be ready in the next commit.
the file should be ``tests/Behat/Gherkin/DumperTest.php`` to match the structure in the code
Please don't include commented code.
Why not use `LastIndexOfAny()` ?
Aaaaaaaaaaaaaaaaaargh...  I'll check it again.
Remaining code does not need to be in the ```with``` context.
This stuff is already handled in the load API PR.
I meant this whole table is out of date.
There are other conditionals further up in the while loop which include a ```is not None``` test, but it is not done here. I was running the ostia meaning process when this occurred: https://github.com/pelson/iris-data/blob/master/generate_from_data_zoo.py
Aww, 4 characters.
Can this be in the grey zone? It's 31.5 mph :smiling_imp:

Happy change, of course, but sometimes it would be nice to have neater code for the sake of a few chars.
Typo: refactor
please add it to the settings-test as well
<code>.exists()</code>
Use " {:r} " instead of " '{}' "
Please delete these lines (2154-2160) as they are commented out - thanks!
My first reaction reading this was 'why not just write a functional view for this one, it's only 4 lines'. May be worth pointing out that (for example) this code with 405 to a PUT request.

Really nice example though.
I've done it this way because that's the way it turned out in my code. 

My Person.get_full_address property couldn't be used as a readonly_field, because (as a property) I couldn't give it the short_description or allow_tags attributes.

But I discovered that I could, if I created the PersonAdmin.address_report() method, that I could have a readonly_field that came from an attribute of the Admin class rather than the Model class.

Also, happy to take advice about string interpolation or other techniques.
This probably shouldn't block this patch, but I'm starting to think it'd be cool, and seriously help readability if we replaced all these SQL strings with unmanaged models, and just used the ORM for querying...
These should reside in the api module
The `InstallArgs` type synonym is used only for making argument lists a bit shorter. It wouldn't be very hard to remove it.
Not convinced by the pipe idea you suggested for handling variadic 
inputs to infix operators. I guess the issue is the evaluation of 
operand expressions prior to passing the infix chain into infixOperator. 
One solution could be to pass through the unmodified section of the AST 
and handle in that in the library. However that seems impractical.

I'm still concerned about compile times and that the 
infix-/prefix-Operator setup is going to balloon into a full blown term 
re-writing thing. Exposing/modifying  AST should probably be done from a 
single point __AST__ or whatever.
So I'm wondering whether it would be better to move operator handling 
back into the parser, use top level symbol definitions for custom 
operators and use the afore mentioned Pratt parser to sort it out.  You 
would still get the user defined ops/prec/assoc just tabulate operator 
specs during parsing as well as processing symbol as usual i.e. define 
(**)[LEFT,1000] or define (**, LEFT, 1000) or whatever.

Or do you think the current approach can be sufficiently extended 
without getting messy?
Also, your PRs won't depend on one another: make them in separate branches, we'll take care of merging.
Shouldn't this happen only inside the condition? Or is ensureCapacity supposed to shrink the _numrows field? This will only affect the returned value by super.capacity() which is public.
I strongly recommend to separate error check code and actual logic.  This (and others) is just unreadable.
I don't think the variable should be called openIdResult (this has nothing to do with openId I believe)
I replaced all use of `pathname2url` and `url2pathname` with `quote` and `unquote`. Also in old code.
So, my plan with Input is not to have a LIVE_INDEXING flag that turns on and shuts off live indexing. It'll be always-on.

Plus if you look at the ElasticTestCase in Kitsune, it turns on LIVE_INDEXING for elasticsearch based tests. So even if we don't go with my plan, it'll be on here.
We're using Django 1.4 that has an override_settings decorator that also works on classes:

https://docs.djangoproject.com/en/dev/topics/testing/#django.test.utils.override_settings

Pretty sure that using that reduces your class to something like this:

```
@override_settings(ES_LIVE_INDEX=False)
class TestCase(OriginalTestCase):
    pass
```
Yes, it's a typo. Fixed it.
Good points. How about.

Initialize our connection with a reddit.

The user_agent is how your application identifies itself. Read the official API guidelines for user_agents https://github.com/reddit/reddit/wiki/API. Applications using default user_agents such as "Python/urllib" are drastically limited. 

site_name allows you specify which reddit you want to connect to. The installation defaults are reddit.com, if you only need to connect to reddit.com then you can safely ignore this. If you want to connect to another reddit, set site_name to the name of that reddit. This must match with an entry in praw.ini. If site_name is None, then the site name will be looked for in the environment variable REDDIT_SITE. It if is not found there, the default site name `reddit` matching reddit.com will be used.

----

Also I think the `reddit` entry in praw.ini should be changed to `reddit.com` to avoid confusion.
Odd parenthesis spacing again ;)
@tfiwm I think you committed this by mistake.
noted above in the TODO list
Not sure whether the DELETE entries are sane by default
Why did you remove these?
This was a bug with the way polymorphic `belongsTo` set their `foreignTypeKey`.  In this case, it would always assign it to the child class (`metafield` in the case of our tests) rather than the type of the parent, like `product`.

I'm not sure how it didn't affect tests before but I think this is the right way.  It's assigning the `foreignTypeValue` (`subject_type` in our test cases) to `product` or `store` rather than `instance.association.model` which is `Metafield`
The `actionTaken` flag indicates if the client code triggered a render or a redirect during the execution of the action. The class also needs to only fire its callback after the action's body has been called and the implicit render started, so this means one extra wrapping `actionStart()` and `actionFinish()` which prevent the callback from firing prematurely if no implicit render takes place. If this extra wrap didn't happen, the `immediateActionTaken` happening because of say a redirect would increment the counter, decrement the counter, and then the counter would be zero, so the callback would fire. Instead we leave an outstanding action which only finishes when all the user's code is finished. Unfortunately, this extra outstanding action doesn't count as a "real", user-triggered action, because it happens every time and shouldn't prevent the implicit render. Thus, the internal flag which says that the `actionTaken` flag shouldn't be set because we are using the frame to track an internal happening. I am ok with the name but if you can suggest a better one that would be cool. I dont think it needs a prefix because this whole class is internal and the option is part of it's own public interface.
Will this cause any problems if explicitly set to `null`?
I reverted this as it was causing weird session connection issues @Raphfrk so perhaps we need to take another look at things. I half expect clients not receiving the session during connect is resulting from a threading issue.
Appending a DIV to ".CodeMirror-dialog span" causes the block-level DIV element to be inserted into an inline SPAN element, which is semantically incorrect.

If you want the error message *below* the input field, then using $(".Code-Mirror-dialog div") should fix it. Or you could change the alert-message element to be SPAN if you want to place the message to the right of the input field.

Note this also applies to FindReplace.js.
Sounds prudent, I'll add it!
Yes, I intended it to be commented out because I thought the browser version no longer runs, and I wasn't sure how this should be done, but felt this was the place that we wanted to (eventually) do it. The first line of the comment is a 1 line comment that was supposed to explain that :)

With all of that said, let me know if you think we need this code and I will try to figure it out.
Seems like we should put some of these things behind the "experimental" flag since we're not using them yet (e.g. the `_onRemoteHighlight` stuff). Also, it's a little confusing to still have the `node()` function (which calls the built-in WebInspector DOM highlighting) since we're using a different mechanism for highlighting--we should either take it out or at least comment that it's not what we're actually using right now.
Glad to hear I'm not the only one who likes to see things aligned.
I think this can be integrated straight into `ext/language` so it's easy for other language to add their own jumpToDefs.
I am sorry, but I do not understand the first note here. I fail to see the place where access to the database tables with self.<database name> is defined, and even if it were, I do not see how to access it when the table name varies (that is, it is not “cards”, but the content of “table_name”.
Something I noticed while I'm in here, I wonder if redirectToAction would benefit from a second parameter that allows you to pass in parameters or request variables such as:

$this->redirectToAction('someAction', 'some/other/data', array('queryStringName'=>'queryStringValue'));
Nope. If the main thread dies I just want these guys to unceremoniously quit.
This whole code is a duplicate of  assertTempIsClean(self)

Please refactor the common code into a separate method and use this method in both places.

Don't repeat yourself.
Please change `m` to something more descriptive, like `face` (or `vertex`, lol).
Please make this Dygraph.requestAnimFrame -- I'd prefer not to pollute the global namespace.
Yes.

On Sep 13, 2012, at 2:54 PM, Cory Zue wrote:

> In corehq/apps/adm/_design/views/all_default_columns/map.js:
> 
> > @@ -0,0 +1,9 @@
> > +function (doc) {
> > +    if (doc.base_doc === "ADMColumn") {
> > +        if (!(doc.config_doc === "ConfigurableADMColumn" && doc.project)) {
> > +            emit(["all", doc.doc_type, doc._id], 1);
> > +            if (doc.returns_numerical)
> > +                emit(["numerical", doc.doc_type, doc._id], {name: doc.name});
> 
> Is the idea making this a dict instead of just emitting the name that in the future you will emit other categories of data?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
I'm inclined to dislike this given that we're not only testing our code, we're testing Google Maps...
I think this todo is done.
For `create_*` my initial thought was to return a new message type for creating values that exist. `MSG_EXISTS`. The reasoning was that if the programmer wants to update a value, they should call `update_*`.
*profile_image* -> *user_image*

I would for size allow: *small* (20x20), *medium* (50x50) or number (of pixles). And then this size would be used in template to resize the image, whatever the `user.image_url` returns (but we pass it also there so that it can potentially optimize the URL based on that).
The corresponding change has been submitted. Now it should be able to handle quoted strings in the combined header. Please advise, thanks.
it might be a good idea to add the event listener on `p.play` instead of `mouseup`, so that if the user presses play with the regular controls, it will still disappear.
remove anchor class