labofsahil/github-event-dataset-2012 huggingface dataset:
Analyzer function

I think I ironed out all of the bugs by now. I also added an OSType.class (enum) because we're detecting operating systems in multiple places now.

by the way, OSType has extra (unused) functionality so we can have stuff like windows 7-specific errors.
Sync/Fidelity addition; misc bug fixes

This should be my last commit to this branch until we figure out a better method for development; I'm all for forming an organization.

This adds Sync/Fidelity functions to talents, including the scaling.  Don't put too much truck in the values or the scaling; I had trouble balancing given the current state of the game.  I also reworked Bioenergy to work differently.  Preliminary results suggest that Sync/Fidelity are much more important than Bioenergy, since the latter currently recovers so quickly.  Think about short term spamming vs. long term tactical skill use.

I fixed the knockback talent to work without throwing an exception (it wasn't checking actor names) and changed ATOMICEFF_GAIN_LIFE to only show a message if health is actually gained.
Video preloader
Fixing problem with links inside of < > brackets in safe mode.

There's a test case that better explains what was going wrong.

Also, a fix for an issue introduced with the logic inversion in commit 35930e0928e19...
Bug fixes and formatting

Fixed name selection
Fixed HP tracking
Added else statement to door selection
Line breaks added for readability
Fixed access callback for User Map and Donation Map local tasks on donation forms (#313)

See https://www.assembla.com/spaces/jr-springboard/tickets/313 for details

User Map and Donation Map forms were not properly protected. Added new permissions to control access to these forms.
GPMONGODB-226 Gracefully handle sub-types in embedded collections.

* Dropped unnecessary use of _embeddedClassName, using existing _class property instead
* Handle subtypes with one-to-one as well as one-to-many.
* Add test cases for the changes.
add collective.langdet

automatic language detection of uploaded files
Goodie to show days between two dates

If only one date is listed, then will tell days between current date. 'inclusive' keyword to include last day in count.
https://duckduckhack.uservoice.com/forums/5168-plugins/suggestions/2701570-days-in-between-calculate-date-differences
Skel/Template (draft) and a few minor helpful changes

Comments in commits.

Added a bake skel/template

Added a create() method for Forms (to set the defaultModel on the extended FormHelper)

Added an addCrumb() alias, for more simple core helper upgrading (and more closely follows conventions)
Implemented currency formatting for the entire framework, there is now a helper in Handlebars that will do this.

You can now use {{currencyFormat varName}} in handlebars and it will
format it as a currency.
Jbpapp 8331
Updates to *Manifest classes to work Site-wide & Minor changes to DuraCloudObjectStore

This is a larger pull request of 10 commits. The code has been well tested.  Trying out how the merge will work into the main codebase.
Fix issues with Manufacturing OrderSaga test

Hi,

A couple of proposed fixes

1) The tests were throwing an exception because the MessageCreator property in the ExtensionMethods class hadn't been set. After a few different attempts, I settled on the Test.Initialize() method being the best place to set this.

2) The tests were then failing because the ExpectSend wasn't occuring(instead, Bus.Defer was being called) - so changed the expectation in the tests to be ExpectTimeoutToBeSet.

Tests are now running successfully (for me at least :p)

Sorry about there being a lot of commits - still getting used to Git...

Also, I've already put in the request but against the develop branch - not sure if that was the right thing to do?

Cheers

Richard
v0.4

It now does everything as advertised... login & display thru DNApi, shout from local.

There's also an added dependency: libjson-perl, which is necessary for Perl <5.14 (mine is 5.12ish, I think 5.14 is not stable yet or something).
[Profiler] Improve the profiler panel design

Using ajax to explain the query on the main page
Added the ability to specifiy where the gradlew wrapper script is located.

Used radioBlocks which don't have inline, so I had to move some of the parameters to a POJO to store them. The alternative was to muck with the DescriptorImpl.getInstance, or really make the jelly convoluted. If the inline for radioBlocks is rolled out, this plugin could use it, but then it would be tied to a bleeding edge version of Jenkins. This keeps the jelly, the DescriptorImpl, and the UI clean.
Vaelastrasz the corrupt fixes

Hello everyone,

I fixed few things for vael:

corrected vael's cleave spell id and timer
corrected vael's burning adrenaline casting order (mana user, mana user, tank, ...), removed bug where no other vael's abilities were used when there was no mana user (return at line 291)
I fixed yo code!!!  ;)

7b830dc6 	is the one that makes the fixes. I think I left a "break;" out of one of the switch statements though
[#38591771] Adding documentation for group registration widget.
geocaching.de -> opencaching,de; neue Teamliste
Created Maven build file
Chinese (Simplified) translation

Base On DoubleX's translation. [http://bbs.gfan.com/android-3680376-1-1.html]
Done by crhan[[@ruohanc](http://twitter.com/#!/ruohanc)]
enable numlock on as default

for get this working the package numlockx have to be add as a dependency of mdm and installed.
Fixed all validation issues found.

This was surprisingly easier than the abstract template, which is still not fully valid (even with phpDocumentor/template.abstract#4).

I haven't found any problems resulting from these fixes.
Fix an issue in Timeline

When the first argument of setInterval or setTimeout is not a function, for example, "alert('hello')", there is an issue in the "after" hook of HookSites.window_setInterval and HookSites.window_setTimeout as userData is undefined.

The complete fix might be adding this type of argument into Inspected target Timeline.
Just what the commit says.

old method:
  win.CreateChild<dt::GuiButton>("button1");

new method:
  win.AddChildWidget(new dt::GuiButton("button1"));

new method is more like node:
  node->AddChildNode(new dt::Node("node2"));
  node->AddComponent(new dt::MeshComponent("mesh1"));

also changed all the 
  dynamic_cast<dt::GuiButton*>(widget->getMyGUIWidget())->eventMousButtonClick
to
  widget->getMyGUIWidget()->eventMouseButtonClick

because GuiWidget has these events, too.

tested on my end, and it seems to work without any errors.
Fix a tiny bug in the docs (spotted by pathogen's doc loader)
(#12189) F5_Monitor provider : Fix bracket typo in template_destination

http://projects.puppetlabs.com/issues/12189
24016797 pulldown is failing at the pooled plate stage
Updating unit tests to fix a broken test case in next

woops.
Swatches: corrected path to referenced image

Fixes Issue #28
Need to check for empty param before setting $sec

PHP dev servers (like mine) are often configured to 
show all errors and notices. The original code displays 
an "undefined offset" notice for commands that don't 
need a second argument.
Added some activity details

new activity method and associated classes for deployment
Update 'master' branch from dabapps
tests and features : clean commit of PR #17 with 1_1_X

Hey @joshmcarthur,

I needed a clean merge  #17 and #15 to push my features into production, so I decided to quickly merge #17 with 1_1_X branch and tests are green yay !

I am sorry if you were already working on it.

Please let me know if I can help,
Stephane
One more test

This test also shows some problems.
Some cleanup, and some safety queueing.
Static library build script that works with Xcode 4.3 

Script works with Xcode 4.3 where "Developer" directory is gone.
Changes! (features + performance)

Bunch of random changes, namely:
- Makes backgroundView on KKGridView do something. Fixes https://github.com/kolinkrewinkel/KKGridView/issues/55
- Adds +indexPathsWithNSIndexPaths: convenience method to KKIndexPath
- Miscellaneous speed improvements found with Instruments.app, primarily:
-- Not calling _defaultBlueBackgroundRendition until a cell is pressed (as opposed to when a cell is created)
-- Using ivars directly within KKIndexPath, instead of going through setters/getters
-- Object pointer comparison in KKIndexPath's isEqualToIndexPath: method
-- Initializing arrays to a capacity, so that they don't have to start growing as quickly. The capacity value is arbitrarily related to my project, and can be made higher, if needed.
rescue BSON serialize error when set :continue_on_error option.

When mongo bulk insert, I hope ignore bson serialize error.

    require 'mongo'
    data = [{'$invalid_key' => 'value'}, {'invalid.key' => 'value'}, {'key' => 'value'}]
    coll = Mongo::Connection.new['testdb']['testcollection']
    coll.insert(data, :continue_on_error => true) # Error


This pull request is rescue BSON serialize error when enable :continue_on_error option.
7017 omexml link

See #7017 and https://github.com/openmicroscopy/openmicroscopy/pull/37
README typos

Spelling/grammar
detect linux api env
replace javascript with html

@alimd
salam man ye bakhsh az file sldieshow.html folder 05 Back & Next JS ro ba java neveshtam fek mikonam intory behtar az ghabl bashe.
Make sure to remove BOM from file on compile

Whenever hbs is creating pre-compiled templates, we need to make sure the BOM (UTF-8 header) is removed or it'll create weird strings in templates that pushes content around.

This will also fix edge-cases were readFileSync will not return a string if file wasn't found.


See https://github.com/joyent/node/issues/1918 for the work-around used.
Engine controllers descend from common base class

This is a simple change that provides a common base controller for the AuthorizationsController and TokensController allowing for simpler manipulation of api-wide filter chains.
support for short_open_tags=off
Update export/import

Based on experiences exporting a smartqueue I've updated the data structure when exporting and importing nodequeues:

- Added owner, reference and subqueue_title
- Removed subqueues. It is a calculated value.
- Do no set add_subqueues if nodequeue has an owner. It is assumed that the owner will be a smartqueues module which should maintain subqueues on it's own.

There's also a few whitespace fixes.
NH-3004: DriverBase.RemoveUnusedCommandParameters removes all parameters when UseNamedPrefixInSql = true and UseNamedPrefixInParameter = false

Unit test and fix for https://nhibernate.jira.com/browse/NH-3004
Kav Latiolais - Better Together: Building Scalable Real Time Collaborative Apps with Node.js

Thanks for putting this together gents. Look forward to seeing all the great talks.
Add volatile-highlights mode.

Hi,

I discovered this gem a few days ago and think it fits perfectly with the live coding environment.

It highlights any text that's been changed by operations like undo, yank-pop, etc. until the user presses another key. It's subtle, but turns out to be surprisingly useful.

The quickest way to get a sense of what it does is to go do a few edits, undo them, and then _redo them_. Each redo will show the text that's just been changed.

It's nice because it gives you an almost subconscious cue about what your last editing command did and also lets you check exactly what changed in places where e.g. your undo affected multiple lines. There are a fair number of editing commands it works with; the author seems to have already added in most of the commands where it makes sense to use it.

Thanks,
- Jeremy Rayman
jeremy.rayman@gmail.com
Menu

Here's the GNOME 3 menu style in Firefox!
I made a small fix to your jquery-autoload fork

I discovered that some browsers appear to have small differences in how they report the height and scrollTop values, so I added 20 pixels to the comparison, ensuring that there won't be a problem triggering the autoloader. 

Thanks very much for making the initial fix; I hope you find my change useful!
Support standard markdown extensions

Fix for Issue [Support standard Markdown extensions](https://github.com/wheelibin/Markdown-For-Visual-Studio/issues/1)
Added two doc links
Fix images breaking on submission errors

See [ticket #62](http://mozillaignite.lighthouseapp.com/projects/86804/tickets/62-flow-bug-with-image-upload). This goes with the simpler, if slightly less elegant, option of clearing out the image upload field rather than devising a way to save it somewhere more permanent and keeping it across requests.
small compatibility changes

Hi, please find two fixes which throw errors in some browsers.
Translation Trait uses language code instead of language names

Language names are used in other other places, such as Translation module.
Added directory namer for custom directory structure

References issue #9

@dustin10 your thoughts on this implementation?
Add type-safe checking for createSequence where large values (BigInteger...

...) are defined within double-quotes. Otherwise a GroovyCastException is thrown.
HTML Sanitization

* Added HTML sanitization in the image viewer.
* Added gallery level titles and captions for imgur.
* Fixed a bug that kept XHR from working in Safari.
Chat Service 0.1

Barebones chat is now supported
Article 'How to force routes to always use HTTPS or HTTP' translated.
Overflow menu doesn't get disabled in action mode

The overflow menu (if any) can still be activated on 2.3 and below while in an action mode, unlike the native behavior. I've got a sample up [here](https://github.com/veeti/ABSActionModeInconsistency) and a 2-line fix that I _think_ works properly on this pull request. (I'd double-check that it doesn't break everything, though.)

Thanks for the great library!
SQL exception translation enhancements
0.3

This branch will become version 0.3 of pollit, it is only partially completed.

Here are some of the newer bits this version will/can have:

 - anonymous voting
 - multiple choice selections per poll
 - limited multiple choice selections per poll (ie: pick 3 of the following)
 - south migrations
 - better views
 - forms
 - more tests
 - more docs

**Ideas/suggestions/code/anything, are welcome**
fix a small bug in the generation of the new_hist.pdf histogram
WiFi bcm4325 driver activation

Everything in one commit - this should work, tested on my CM7.
追加しました。

宜しくお願いします。
documentation for self referencing data loader

see propelorm/PropelBundle#114
Валидация URL с учетом указанного порта

Добавили возможность указать порт при инициализации
LPS-25059
only load if request uri matches user-new.php

Any comments on whether this is the best method to handle this?
Use qwebkitplatformplugin.h from webkit trunk

Replace qwebkitplatformplugin.h with the original file
from webkit trunk.
Rename deprecated classes

This renames the deprecated classes to their new names.
Forgot to include nested slide example in the demo directory...

I think you would want to have this in the demo directory.
update to remove undefined from filter bar. 

did an additional check for undefined for local storage values. been a
while since this was released, and chrome is a rapidly evolving beast,
but this should take care of things for now.

thanks for the incredible extension!
Updating documentation, re-adding years, months, days, etc

* Updating documentation to reflect earlier changes in 14121e6.
* Changing string parser to use regular expression.
* Allow user to set variable in settings to allow use of years or months.
* Re-adding years and months to durationfield string parsing.
* Adding tests to timestring parsing.
Edited show and person pages
Added support for Haml

Hi Seyhun,

I added support for Haml for layout and themed generator.

Regards
Thomas
Add .scss as a default css file in sublime settings.

Hi, just thought it would be useful to put in .scss (for sass) as well for the defaults.

Cheers,
-Tak
Fix NPE when the aggregation value is null
Got lib/display out of lib/io (GridIO)

This is a proposal for a slight modification of the GridIO interface. In the old days the io-lib was more or less independent for the display-lib (more more than less) but GridIO used the Grid class from display.  With this slight modification it is seperated ance again, as the Grid object is instanciated outside of GridIO. Have a look at it, keep it or throw it away.
deleting messages & migration for heroku

redirect back after deleting messages makes more sense, and migration needed to be fixed for heroku.
Fixed relative/overflown positioning

Issue #235 only fixed bottom* positioning, and actually made top* worse.

This patch is a comprehensive fix for all relative/overflown positioning.
It includes a demo file for positioning, so that you can easily test and verify that all positioning modes work correctly.
Added "Disable" and "Transparency" strings.
Display compose_to only if profile, fixes #819
Shouldn't this be "< 0"?
Shoud be `The default resurrect icon`
CalcNotIgnoreDamageReduction ???
This Function will "not ignore" talants like http://wow.tltgame.ru/aowow/?spell=53376 (effect №3) ---> Paladins with http://wow.tltgame.ru/aowow/?spell=31884 will be ignore 50% of resilience ...
Is this correct ???
(если можно ответьте по русски =))
Docs don't match function
Pokud nelze primarySequence detekovat, není lepsi docasne mit public, abych si to mohl nastavit, pripadne dalsi setter?
Hey, so have you tested if custom stylesheets that you include in `css_paths` in blog posts get rendered on the homepage? If you get it working please tell me. :)
Sure. I don't really know how though. Any advice ?
Also, Cyclus fails to run unless I set the CYCLUS_OUTPUT_DIR env variable.  I think instead cyclus should have some default behavior if the env variable is not defined - instead of sigsegv.  Also, If the cyclus.sqlite file already exists, cyclus terminates.  Instead, I think it should either overwrite the file, or generate a new unique filename for the output database.
@stof https://github.com/Behat/Behat/commit/dd79797c748fd82e903627d3a8eff8f04cf37ed2
Needs work but it's going!
One more thing, would be great if you could style up the CCV page
Regression-test?
The ```mode = 0777``` line might not do what you think it is doing. In python, a leading zero means the number is octal.

    >>> print 777
    777
    >>> print 0777
    511
    >>>
use "box" or "queue" instead
My apologies for the confusion
Oh right, that will match Mountain Lion, won't it? Was trying to future-proof.
On Thu, Jan 26, 2012 at 2:58 PM, Nigel Pearson
<reply@reply.github.com>
wrote:
> /usr/games?
>
> Not on OS X :-)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/MythTV/extras/commit/ad019a9978239459eb207cad1a9e208c92248bc9#commitcomment-906220

heheh, pretty sure that was an expansion for $PATH at some point.
why the name test_gradgrad_inc ? specifically why gradgrad?
The --assertf~= function is giving incorrect results. I checked its logic by testing it with integer! arguments so it appears the logic is correct.
So the layeroffset is a new item in the layer?
Maybe it would be better to have layeroffset as a property of the layer item such as here http://pastebin.com/VUbRhHzD
Yea - The `Minion_` seems redundant to me. I would prefer to see `Task_Abc`.
Yes, that's true. I didn't actually test the regex. I'd rather the regex **didn't** match those.
hi,

i tried several aproaces:

this happens if i try to import falc files:
unreadable file: 11 Logos.flac
header parsing failed
unreadable file: 01 The Shakes.flac
header parsing failed
unreadable file: 02 Amplifiers.flac
header parsing failed
 etc etc

Theese files where in my library before and beets read them fine then.




On Fri, Feb 10, 2012 at 4:54 PM, Adrian Sampson <
reply@reply.github.com
> wrote:

> @fixxolas: Can you be more specific? (Do you have a traceback?)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/sampsyo/beets/commit/72659975a2c800e2a8fc840f575b90e5fbd0e9ff#commitcomment-954086
>



-- 


Mvh
Nicholas Amundsen
@dkubb actually the missing method was added to rbx-head *yesterday* that's why travis is a bit behind :)
It might make more sense to have num_examples be a property of the iterator rather than the dataset, in case we make an iterator that can change the number of examples in the dataset (for example, an iterator over image patches of an image dataset). But I don't think this future concern should hold up this pull request.
Maybe say "this string is marked as ASCII-8BIT, Ruby's binary encoding" or something like that?

I only bring it up because there's also "US-ASCII" which has different semantics (only allows bytes 0x0-0x7f)
ТЗ в папке TECHDOC\ (ранее не ту папку указала)
> The question is what happens when you call "isKindOfClass:" on a pointer that's not an object? Any ideas? Anyone know the difference between calling isKindOfClass: and using object_getClass()?

Just tested it and it fails to link (it seems). Will work on a fix for this.
I think you mean `next if g.directory?` here; otherwise it returns early and I get false positives.
Just that I don't know whether Invertika uses them. This would give them some time to adapt.
I'll correct it!
Good point on the size_t.  I forget that too often---although the next set of warning flags that I turn on will make me fix these mistakes.  ;-)

Also, it's a count not a number of bytes and it should be compared to 1.  I didn't get that bit wrong.  The reason that I put the XXXrcd comment
in here is that I'm not entirely sure what to do.  I'll go with your suggestion for now.  I haven't looked much at the surrounding code to determine
what makes the most sense.  KRB5_RC_IO_UNKNOWN will cause the whole authentication to fail, right?  This is likely correct, although there are some potential downsides with, e.g. partially written or corrupted files that I think that we should consider in more detail in the future.  I.e. how do you recover from this situation, etc.
Whenever I see a `1:`... structure, I consider the possibility of a `1:0` bug (I've found one in base R, even).  Is it absolutely impossible for `nrow(plot_table$layout)` to be less than 2?  Also, since the index is repeated (after looking at it, it looks like it moves the last row to the top), why not pull that out?  Or, should the second one reference `plot_table$grobs` rather than `plot_table$layout`?  You would probably be better with `c(nrow(plot_table$layout), seq(length=nrow(plot_table$layout)-1))` or `c(nrow(plot_table$layout), seq.int(nrow(plot_table$layout)-1))`
To avoid this hack you can set ```OpenLayers.imgPath``` and add

```<link rel="stylesheet" href="js/openlayers/theme/default/style.css">```

to the ```<head>``` of the page.
I'm not sure what you are trying to do here.  Are you putting the controller in a block?

Honestly, there should just be an invisible block and a normal entity goes on top of it.
Good! Thanks.
Wow, I'm amazed at how little agreement there is on what to call this. I've seen pushFront, enqueue-left, enqueueHead, and others. I'm pretty sure the standard name in CS is `enqueue`. Removing from the front is maybe `unqueue`, but that is not as standard.
I'd much rather have `shift` and `unshift` than most of those names. But they don't make a whole lot of sense to me. In fact they kind of sound opposite; `unshift` seems more likely to me to remove an element. Compare "please unshift this request" to "please enqueue this request"? In fact "shift" is used in parsing, where it means basically "push". So that is confusing to me.
@JeffBezanson, I find myself in this rather odd situation:

```
julia> @assert !isless(-NaN,+NaN)
assertion failed: !(isless(-(NaN),+(NaN)))
in error, /Users/stefan/projects/julia/j/error.j:5
in anonymous, no file
in run_repl, /Users/stefan/projects/julia/j/client.j:22
in _start, /Users/stefan/projects/julia/j/client.j:134

julia> !isless(-NaN,+NaN)
true
```

Hmm. Also, all the tests pass, including the new ones testing exactly this.
me too @markstory. But I thought as $path is a parameter it would probably make sense to be a bit more fault-tolerant. But, of course, it's ok as it is.
https://gist.github.com/1565195
https://gist.github.com/1565199
Незнаю после чего но вот какая мудрость вылезла https://gist.github.com/1949724
https://secure.gravatar.com/avatar/457439dd610042a46972f25d1c8b1019?s=140&d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png
Было удаление 1920, 1921, 1922, 1923, а вставка 1820, 1821, 1822, 1823.
Удаляли одни значения, а добавляли другие, поэтому возник конфликт по уникальному ключу.

Заметил, что много файлов было поправлено следующим образом: в самом конце добавлен энтер, мегаправка.:)
?con mil diablos!
seems good. for gomlins - not need change faction to creatures with GOSSIP flag.
found another issue with this commit, my startup log is spamed with things like 
2012-01-07 20:08:19 ERROR:Table `creature` have creature (GUID: 300063 Entry: 34944) with `MovementType`=0 (idle) have `spawndist`<>0, set to 0.
Currently theres like 64k entries like this, different npcs. In database the npcs are of course ok, movementtype 0, spawndist 0. WIth reverting this commit i dont get the errors. Does it happen at r2 too, and should i create an issue ?
If something should be done, it should be the capability to map Ruby methods to backing keys without assuming the method name is the same as the key.

    field :foo_id, map_to: "foo-id"

It's better if we mangle method names ourselves than if Mongoid did it for us automatically in a way we cannot control.

But again, this is only if something actually needs to be done.
don't want to use Deferreds?
Uh! Do you have an example using this AudioObject?
Sorry,we have to revert this commit for the moment because not everyone has cryptokit >= 1.4, and i can't manage to integrate it well with our installation_helper for the moment.
ez nem hülyeség?
Or like, not. This didn't seem to help.
At least for the more common alternate line polarized displays e.g. the LG Cinema I have and the Zalman Trimon I've used.
Glasses free autostereoscopic displays like the Nintendo DS and the very expensive lenticular displays may fit the existing vertically alternating code although I've been told the Phillips lenticular at least takes image + heightmap style depth input.

My interest is TV so I'll probably be making a side by side or top and bottom split version at some point.
Oh, and use 4 space tabbing. No tabs please, but spaces (you can probably configure your editor to do that for you).
You should use `assertCount`
Just `/search`. I was thinking hard about that and decided that we will not use leading `/`. One character less in URLs and it makes browser have a better download filename when users are saving a page.
Конфигурация не найдена
Ah, [there he is](https://github.com/ogx/Calculation2D/commit/e0f04a593576d71a32b7a7b0dcf851b530e6d086#L1R44) ;)
I can add attribute definition class whith prperty like allowExpression and during parsing check does this attribute support expressions
if it works, be glad to see this in.
Won't work on Firefox™.
"with a request attached" or "that have a request attached"?
I did notice that if you set this in the default group, a test fails.  While this solution does not make the test fail, would you want to make that test in user_spec not dependent on the default setting?
Oh yes friend, why not, I want to implement it in my server too. Thanks for answer.
What about using a null object here instead of returning nil?
wil says these are already in for `addons` even though I don't see how...in any case it means they should be for both. I agree with the tests needed
前端略嫩了些，看来你还真不是文艺青年呀～ 建议暂时不宣传，内测用，等寒假有时间了咱们改一下，包括和其他几个项目的整合，厚积薄发
Shouldn't we return -1 here as well, as this is an error path?
Interface -> Imeplementation
In that case, would it be better if it was read from the config file?
uint32 packedGridPos = packTileID(y, x);
is there only once...
where can you see the other one ?
You're supposed to add the bapu directory to the build includes. Bleh, I'll merge it anyways.
Please add .rb at the end of the file, to make it clear that you're pulling in code in your directory, and not a gem.
This was a bug, right? Safe to remove.
I've been meaning to tell you that there is a flow in doing this. With libxml you are not getting the entire block contained in a tag, but only until the next entity or max 1000 characters.

I think it might improve performance to always output an attributed string for an entire block (like the entire p) and possible construct it in a way that you can add the hyphenation wothout having to change the classes. Possibly with letting you pass a block or subclass the steing builder.
add MRML as the first subdirectory. Order is important, Logic and Widgets wouldn't be able to access the MRML cmake variables from a clean build.
Btw, it would be great if you delete the the inner build (directory Slicer-superbuild/Slicer-build) and build it again ( to make sure it works)
Also non pushed typo fixes and testing the git GUI
Large code change.
This must have taken a while I assume?
I uploaded a new package libnfs-1.2.0.1~git0804e67. So we can reenable libnfs.
No. without autostart.sh also freeze. Really strange.
If I simply delete this Line the Error still Exists.

I have some Time in the next Week, so I will work on it.
Did you mean this to be in the pull request?
@agcolom - I just tested that page in Chrome 15/Mac and the 3 wide navbar looks ok. Is there a specifici window width I need to resize to?
FYI... there's an invalid em value on Line 48 in css/structure/jquery.mobile.core.css.
Sorry for getting back to this so late. Got really caught up.

I just looked at the test files, and it's going to be a pretty heavy edit. The test files are all assuming that the test database is SQLite (hard coded in cgi-uploader.config), but the error is only applicable to Pg. So we'll need to somehow work this out.

Any ideas?
Still need to build the main JS file and commit these changes there.
Thanks Mike. That fixed the problem
Just to note: this bug was only in the master 1.7 dev branch, not in the 1.6 release branch.
I am cherry-picking this commit into master.
I've never tested the performance, but the documentation on LinkedHashMap states that it is better than using a treemap, so it shouldn't make a big difference to before. My only reason to change the code was to get a limitation to the cache, and yes, we should also use that concept in the caches cache. That one would be even more important. Can you do it? I'm just on the jump to work...
Thanks for revealing this. The script doesn't include this, but I think this can also be done with DB script. I'll get back to you with an update for this.
might want to use QString::trimmed() instead (better semantic)
http://www.musicpd.org/doc/protocol/ch03s08.html

so that you have to enter a password when you go to the website in order to have write-access.
Ah ok, so I can simply change : to #!/bin/sh and then let the code do the rest as it was before, thanks!
is this already fixed? I can't find the fix in the repo :-(
Did the smart thing...just copied Sergios fix (his commit 701de8d6a3e3b93448ecaf8bf5aa778aaecaaec4 ) for select to fix multi_select...works for me now.

Regards
Michael
Whoops. Mind you, you're one to talk about compile errors. *friendlysniping*
@LihO if you see reports in issues there are many crash reports so I can confirm it too, unstable :/
break; missing?
These spaces are required to make a valid SQL comment! Use dash dash space then you can comment!
А вот это говно нужно вынести в tests или examples.
or comment
@nesquena are you sure about this?
Sounds great, can you make a commit/pull request?
ehm, no because I'm still not good on git. Can you do it, they're just one lines? :P
ok, made it 55% main net, 50% testnet.
It's merging two diverging 'master' histories.

Don't be deceived by Github's presentation, it's not actually committing the same thing again. E.g. if you launch gitk, you see that the commit doesn't actually add anything, but combines two histories (those with 26d44 and 5c41b as their heads).
Those SVG files were for the gradient fallbacks, for browsers that support SVG but not CSS gradients (IE9 and Opera need it).
My thinking is this. Get the hello_monkey client working:
https://www.twilio.com/docs/quickstart/php/client/hello-monkey

So I have a helper function that now generates that - except that the html doesn't work. Seems to be the same but I can't verify that the token is correct - so write a test suite that checks the token is generated the same.

So then I walk through the PHP script that generates the token printing out the subcomponents that go into making up the token and check them line by line. Getting the Erlang token component generation to match that of the PHP to make sure that I am getting all the padding/hashing encoding stuff working right.

This change is part of that plinky-plonky/make it all the same :(

Another approach might be to generate a token in Erlang and then push it through the unpacking routine in PHP and make sure it comes back to the original Erlang structure?
I love this approach, tag-> process
Wow :D
Awesome.


![Who's awesome](http://who-is-awesome.com/who-is-awesome.jpg)
Zmiana wielkości liter + spacje
The screen shows acore, but it's Contacts(People) that is dumping to logcat. 

Getting ready to pull down the new stuff and give it a shot.
you can not translate things this way...
wtf? you just broke it. I did a sniff myself, adjusted the packet order (race before gender) and tested here, it worked like a charm.
shouldn't this mozilla.org?
argh yeah .. i guess we can add some BC hack in there.
Drive-by mini post-review comments. Please fix what you can, and try to avoid doing again the ones you can't.
Lazy @wicz, I can't believe you pushed an icon like that. Shame on you :flushed:
Now we have a wiki :)

Relevant content, would be anything game-related. If you followed the previous game, we also had a wiki, which someone kindly filled it. (I cant remember who was it though..) Anyway, we can worry about the website later
layout.html extending layout.html? This causes "RuntimeError: maximum recursion depth exceeded in cmp" I think.
#bikeshed: No need for the "." between the "X" and "alpha"
Thanks, But this is a bundler issue if you want it to append a command to run bundler when missing gems are found. Does not make sense to put this in every sub project that uses bundler.
Binary files should be excluded from the repo. You would need first to create a *.gitignore* file in the root of the repo with something like this there:

	assets
	bin
	gen
	libs
	.settings

	proguard.cfg
	project.properties

And then possibly `git rm` command should be used. 

Generally, only these files should be in the repo for Android Java Project:
 * res/*
 * src/*
 * .gitignore
 * AndroidManifest.xml
 * project.properties.
Hey Alan. Can you not add whitespace all through the code? Basically every line of whitespace change that you add means that we need to wade through each line manually to look for differences. Was the line changed from logic changes or from whitespace changes etc? The best approach is to separate logical changes from whitespace changes. In this case though its just a bunch of spaces being added. If someone sent the change separately I'd reject it unless all of the code was modified to have the same whitespace and really there isn't much reason to do so imo.

Is your editor doing this? If so can you turn it off?
These single line if statements aren't python best practices and woudl be confusing for a reader.
Need some sanity checks in case the config was not set up properly
That function is reserved for internal core use only. Same goes for _doing_it_wrong().
*Always* add `@return void` if there's no return value.
hence also why i think this should not be handled in the request, but in the server object itself.
Let's change to # Replaces placeholders for missing expression values in NetCDF with 'NA'
For some solver libs,  one can allowed to allocated a rough memory space, say sub block wise. The default argument value may be used  for such cases.
Regarding your suggestion about raising an exception in `TestPipEnvironment.get_pkg_version()`: wouldn't that prevent usage of that method for editable packages, since they wouldn't be in installed in `env.site_packages`?
No need to call `slice` here.
"handle" should be "handled"
About the tests, I have no time now to write them, so merge it without them, or wait I have a moment. Anyway I tested it and it is working ok.
I think this was already resolved with Neb's commit, I would take these down (other than the antiban) but I don't know how
I don't understand what the MultiNodeSpec has to do with OSGi bundles.
Have you've verified the bundle metadata and the generated poms?
`Try` ?
Why Terminated(_) ?
No, I mean, it's in the ...camel.internal package, and is private[camel], since it's for the ActivationTracker, put in the ActivationTracer companion and make it private[ActivationTracker]? The name "State"/"StateName" doesn't make sense to me if I just see it in ...camel.internal
I think that is problematic from the command line script
We could, but I couldn’t come up with a good name which was not `Resume`, and that one is obviously taken. Dispatch within ActorCell is also a bit more concise like this.
Ok I'll do that then. whatever floats yr boat ;-)
I thought we had the suspend count in a var? (I thought I saw that somewhere in the PR)
I didn't remove the nodes since I can't know in `beforeShutdown` if the test has failed, and removing the nodes just makes the barriers succeed.

The compromise right now is that if any node is in a barrier when a node is disconnected then that barrier and all subsequent barriers will fail.

If I leave the code as it was, then every MultiNodeSpec will print a number of ERROR messages when the test is shutting down normally.
hmm it may not fail.  should test that files wo perms have right error msg.  that's what uses #deny
Okay, so:
if pkey is composite:  'SELECT COUNT(*)' 
otherwise =>  'SELECT COUNT(mypk)'  

Is it what you were thinking about?
Just to add 2c re user concerns: I do **not** think revisioning for groups or orgs data is needed though recording *activity* for reporting purposes may be. These 2 things should be distinguished and I would imagine that reducing the amount of revisioning we do will significantly simplify code.
I would call this is_sysadmin, but I don't think we have a style guide for names
I think there's something in `ckan.lib.helpers` that might do something very close to this already.  Probably worth checking (@tobes will know) if it's applicable here.
I think we should bind the :first codename to the initial user in setup, then reset the codenames.  Anything else could break when we change things later.
http://www.merriam-webster.com/dictionary/yea
Definition of YEA

1
: affirmation, assent
2
a : an affirmative vote
b : a person casting a yea vote
 See yea defined for English-language learners »

LOL
Well, actually in it's current state it broke things as it incorrectly encoded URLs. In theory there are places where we could even url encode url options seperators like ; and & or the path seperator / (which I missed btw. in my change above: will fix that up too). Anyway: we need to do this right and take for granted that it *could* cause regressions, imo.
m_filePos doesn't mean the same thing when reading. When writing, it means "How many bytes were written upstream between 2 pauses".

I should create a more speaking variable, but I find it a bit silly to add 8 bytes just for readability.
free(0) is totally safe and has been since 1999.Sascha Montellese <notifications@github.com> skrev:In xbmc/windowing/egl/EGLNativeTypeAmlogic.cpp:

> +  (*nativeWindow)->width = 1280;
> +  (*nativeWindow)->height = 720;
> +  return true;
> +#endif
> +return false;
> +}
> +
> +bool CEGLNativeTypeAmlogic::DestroyNativeDisplay(void*)
> +{
> +  SetCpuMinLimit(false);
> +  return true;
> +}
> +
> +bool CEGLNativeTypeAmlogic::DestroyNativeWindow(EGLNativeWindowType nativeWindow)
> +{
> +  free(nativeWindow);
Can we be certain that nativeWindow is never NULL or do we need a check here?

—
Reply to this email directly or view it on GitHub.
Tab instead of whitespace between "return" and "string".
I guess that was a rebase artifact. Thanks for noticing.
Is that a good idea either? Or should we maybe think about unifying that behavior, so that the data in the catalogs is consistent with what is presented to the user through FUSE?
Thanks, I just started to read the batch from yesterday.
I took that regexp from http://code.google.com/p/periscope/source/browse/trunk/periscope/plugins/SubtitleDatabase.py#40 . I think year isn't optional as its used to check if we got a movie or something different when we dont know what is it ( when you cant find out using the file name ). I fixed that unnecessary escaping in new comlmit. I dont have any idea why | is repeated there.
in my opinion it didn't, file names using camelcase are windows-style (think of `/home/joe/` versus `C:\User\Joe\`) and is less shell friendly because it requires additional key strokes. just like the `Server.md` file, which is as a side note utterly deprecated.
This selector must accept a single argument.
Unnecessary newlines. Prefer terse code.
This style of conditionally assigning variables is rather cryptic. We try to stay readable throughout core, so these would ideally change to `if` statements.
This is timing sensitive. Use `process.on('exit', ...)`.
Good catch. I added the name feature before I added the RDD fetching, so I didn't think about it at that time.
This is the line that seems too long
This looks great but please add spaces before the curly braces you use (I've noticed them missing in a few places, such as this one). Also, did you double-check that BeforeAndAfter runs the "after" part after every test instead of after the whole suite? There used to be two versions of it.
could you spell out what "it" is here?
D'accord, mais quand le save fail tu veux retourner du html si c'est ce qui est demandé par le client.
I think you are misunderstanding the semantics of `from mod import ...`. It does *not* do `import mod`. In most cases, doing `from mod import ...` will not put `mod` into the namespace.

```
[scratch]$ cat foo.py
from os import path
print os

[scratch]$ python foo.py
Traceback (most recent call last):
  File "foo.py", line 2, in <module>
    print os
NameError: name 'os' is not defined
```

When `from mod import ...` is done inside of a package's `__init__.py` file and `mod` is a module inside that package, then `mod` gets inserted into the package's namespace as a *side effect* of that module being imported. When modules inside of packages get imported (no matter where the import comes from), it gets inserted into the package's namespace, as long as no one is doing anything slightly weird. We wouldn't have to change any other instance of `from mod import ...` outside of `__init__.py` files. Certainly, no code outside of `__init__.py` files should be relying on `from mod import ...` to actually put `mod` in that namespace because it doesn't work that way.

But sometimes slightly weird things do happen. Code analysis tools will often want to import modules directly and thus miss some details like making sure that when executing the `__init__.py` code, it should be as a package and not just another module. Now it's fair to say that those tools should be fixed to be more robust, however, I think that it's also fair to explicitly import the names that we need instead of relying on a quirk of CPython's import implementation. I don't even know if it's documented well anywhere. It might not work exactly the same way on other implementations.
Ah, got it. A note in the commit message could have clarified that, or maybe if I was smarter ...

I'll just go ahead with the commit.
Well, the trick is that for one-file builds it has to be declared static, in multi-file builds it has to be declared extern in a header, and in builds of third-party packages it needs to be looked up via the famous array-of-pointers API structure. I'm pretty sure declaring it NPY_NO_EXPORT in a .c file plus putting it in numpy_api.py is the right way to automatically achieve all of those things. I found it useful to look at the generated __multiarray_api.h to see exactly what was going on.
No, NOCOPY has a very different meaning from KEEPORDER. The latter means that the order ought to
be kept (hence the name) while copying, while NOCOPY means that the data should not be copied.
无论如何都返回 200？
ну там, где if (cut) проверяется
так тогда он не будет видеть репозиторий
вырезал отсюда. выложу на вики потом
I think we can call `is_options_required?`

About the `include_blank` I mean, if the user want to set the `include_blank` to `false` we should support this. I know that this will generate invalid HTML but setting the value to `true` if the options is required always will make the users confused when they are trying to set the value to `false` and it is not working. Maybe we can only document this behavior.
Since we are not using the `current_config=` method this should be only `attr_reader`
In first, I removed it. But I got test failure.

```
  1) Failure:
test_0020_all stored attributes are returned(StoreTest) [test/cases/store_test.rb:139]:
--- expected
+++ actual
@@ -1 +1 @@
-[:color, :homepage, :favorite_food, :phone_number]
+[:color, :homepage, :favorite_food]
```

If remove one, the following lines aren't executed.

https://github.com/rails/rails/blob/master/activerecord/lib/active_record/store.rb#L88-89
`config.action_dispatch.default_headers.clear` would clear them. Rails has many defaults, we don't list them all on `config/application.rb` for a reason. Documentation solves the problem of knowing what is on by default and how to clear it.
We cannot use `as_json` here, since `serializable_hash` is used for both json and xml. You should probably be overriding `serializable_hash` instead, which is intended for that.
No problem, always good to have code review ;)
These lines are now gone.
I think it is  good to make explicit the database structure.
Doesn't `decrement_counter` only decrement by 1? Can you change your test so that it has 2 records before setting the association to `[]` to see if it passes correctly?
I maintained the 1.8 hash syntax that was present in the file. Do I update the entire file or mix the two styles?

Op 20 okt. 2012 om 23:28 heeft Rafael Mendonça França <notifications@github.com> het volgende geschreven:

> In actionpack/test/template/template_test.rb:
> 
> > @@ -47,7 +51,7 @@ def my_buffer
> >      end
> >    end
> >  
> > -  def new_template(body = "<%= hello %>", details = {})
> > +  def new_template(body = "<%= hello %>", details = {:format => :html})
> Use the Ruby 1.9 hash syntax
> 
> —
> Reply to this email directly or view it on GitHub.
>
I guess this one could be rewritten as:

```
case AnnotationInfo(TypeRef(_, _, List(arg)), _, _) =>
  Some(arg.typeSymbol)
```
``'None'`` --> ``None``
Exact, it will be fixed in next commit.
Is there a reason to use this over ``qapply``? It seems redundant to me.
list(Union(self.args)) should be able to do this.
put all of these in if __name__ == '__main__'?
see below comments re: test names
It's a nice-to-have... but being able to test with discontinuous data that doesn't form just a horizontal line would be nice. I won't withhold a signoff without that functionality, though
Ok I will change a bit the desing to do that
why is this test repeted?
Params['pid_dir']) already exanded the dir. 
Concider using  Params['pid_dir']) directly instead of using PID_DIR
no, nothing else really requires it, but it is useful to have this information for testing
have a reason
I think these annotations aren't right, or at least "center" is confusing.
I find 2 variants used throughout the docs now. Which is better in this case?

:func:`db.printShardingStatus()`
:dbcommand:`printShardingStatus`
yay moar jquery!! ;)
Using loose comparisons (==/!=) would seem to be more robust to me.
This requires a bit more consideration then.

What would happen if config names were changed to reflect class names, such that external code could convert from class name to config name?
If the test fails does the user remain in the database? Will this result in subsequent test runs failing?