labofsahil/github-event-dataset-2012 huggingface dataset:
Update resources/ee3/lang/pt_PT.xml

Updated to the latest US Lang file

EDIT: You can ignore this, RedRoserade already made this.
Correção Professor e Varias Adições

- Correção do erro no cadastro de professor
- Adicionado toString em todas as entidades!
- Correção toString da classe IntemGrade
- Correção toString Telefone
- Ajustes nas constraints de disciplina e periodo: Valores de entrada!
Update Configuration for CB 1493+


Update for:

 - new Configuration Usage (FileConfiguration replaced Configuration)
 - Boss Max-Health exception (Now gets max health from entity) 

Note:
 - Added own Logger: seen in console: [MobArena::WARNING] ...
 - Added spanish version (by lucasvilla97)
Slight bug in pardon command handling.

Fixed a duplicated entry for player-pardoned in the localization file which meant the console printed a message saying the source had been pardoned, instead of the target. I've renamed the option to player-pardoned-by to distinguish between a message to the source, and a message to the console.

Also added a reply (player-pardoned) to the source if the player has been successfully pardoned, before if the player was successfully pardoned the source was not told, which lead to some confusion.

Hope this Helps,

-Keith
added links to all, new and bulk submissions

added all these to studies/index and a bulk link on the study workflows page
Espresso 2 compatibility

Hi!  Just a quick fix for php syntax highlight in Drupal files in Espresso 2
Whitespace and PEP fixes.

I highly suggest adding more space between lines, as jumbled up text and commands can be very confusing and hard to read and interpret.
LPS-30512 schedule fields don't make sense for default values
LPS-29892 Fix journal sql for HSQL
LPS-30870 Make RSS logic consistent (we will apply more changes later) Please read description -->

FeedType is now: FeedFormat + FeedVersion (this is also consistent with ROME standard)

@brianchandotcom, we needed to use useModelValue=false becuase the column in feed entity is called feedType when it actually refers to a feedFormat. do you want us to rename the entity column?
858289: Rename the desktop file to subscription-manager-gui.deskstop
Apparently I have no push access here
$rel_canonical_path in den controller packem
added overwrite_user_agent property to manifest
Image regex

fixed #61 by adding an image regex when creating a new project challenge
Correctly colour output of wrapped lines

After a step passes or fails, the intention seems to be that it should change colour. E.g.

**before step:**
```<grey>When I do something</grey>````

**after step:**
```<green>When I do something</green>```

This works fine except when the step description is longer than the width of the terminal, where instead of changing the colour, a new line is added with the intended colour, e.g.

**before step:**
```<grey>When I do so
mething</grey>```

**after step:**
```<grey>When I do so
mething</grey>
<green>When I do so
mething</green>```

Definitely a problem in Windows/Linux, not sure if this was a problem with OSX's terminal app.

This pull request contains a patch to resolve this problem. It does however cause the functional tests to be sensitive to the size of the terminal they're running in (if the terminal is too small, a lot of the assertions about escape codes output by the tools will fail as they rely on the the assumption that no wrapping occurs). 

Not sure what the best way would be to fix this so that they can run independent of the terminal. Two solutions I can think of:

1. The tests could make decisions on what escape codes to expect based on the current terminal size
2. The tests could actually create different size terminals to run the functional tests in with sizes that match the test assertions.
A quick fix to make sure the eventtimestamp field is always the same length

I added a "0" to dates less than 10 to make eventtimestamp consistent.
framework for looping, random and sitting NPCs

reworked how default anims are applied and random anims reapplied to
idling NPCs, framework for NPCs to have & sit in chairs working, NPCs
just need to be added to _sitNPCs in initNPCs.sqf to init with and sit
in their own chair.
Made HTTP call to SaveAction service on Mobi App sync call
Optional MongoDB support for IonAuth

This pull request introduces an extra model to IonAuth which adds optional MongoDB support.  

I tried to keep the fingerprint of the *IonAuth MongoDB Model* at its minimum in the original code. The new model has two external dependencies:  
* [CodeIgniter MongoDB Active Record Library](https://github.com/alexbilbie/codeigniter-mongodb-library/tree/v2)
* [CodeIgniter MongoDB Session Library](https://github.com/sepehr/ci-mongodb-session)

The code is pretty well tested but I hadn't enough time to write unit tests, please test and let me know, thanks.  

P.S: Here's the modular version of IonAuth library with MongoDB support enabled: [CodeIgniter MongoDB IonAuth Module](https://github.com/sepehr/ci-mongodb-ionauth-module)
providing empty value to `unescape` if GET parameter was empty 

 	providing empty value to `unescape` if GET parameter was empty
Support AT+CMEE - enable the use of result colde "+CME ERROR: <err>"

CME shows error relating to the functionality of the AG.
LPS-24878
better handling for rasing proper exceptions in Slice URL Service
Current user has one of the selected roles

If the currrent user has one of the selected roles, allow them to update the policy
Added links for Contributors - Corrected list of Public Repos
Color fix

Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Fixes the following tickets: -
Todo: -

Colors of selected rows were replaced (my bad). This PR fix it.
More Dalaran Sewers Arena fixes

On this pull request:

 * Repeat the knockback effect of the central waterfall while it's active. This should prevent anyone from staying inside the waterfall (How did they get there? Charge or Demonic Circle sometimes help.).
 * Remove Demonic Circle related auras on arena start and when players are jumping into the arena (*this behaviour was confirmed by a player on retail*)

In my opinion DS arena is almost ready for getting opened after this, what do you think about it, guys?
Latest Build: Includes all other PR updates

Includes all modifications

Major changes listed in other PRs included along with more minor changes.

See commit log.
Replaced deprecated call to logger.silence with logger.quietly

Works with Rails 3.2 after this change.
Initial stab at Share button
missing trailing space after colon in resp headers

I've seen response headers come back without a trailing space after the colon.
If no space after colon then $kv[1] is empty and php throws an "undefined offset" Notice.

Looked at some RFCs to see what "Content-Length:" headers SHOULD be. Will defer to someone more knowledgable.

I've also seen bugs around the web w/re to similar bugs so figure this is safe to suggest as an alternative way to parse response headers.
THis is the code from the demo
A few linux fixes to get the blackjack demo working on linux
POD error

perldoc threw an error for the encoding tag

```Around line 614:
           Unknown E content in E<failing test case>```
Fixed wrong references to ${package} where it should have been ${groupId} in functional-tests module pom.

Fixed wrong references to ${package} where it should have been ${groupId} in functional-tests module pom. To reproduce this issue generate an archetype where the package and groupId are diferent, then in the generated project run: mvn clean install. All modules should build and install fine except for the functional tests module due to the group id references are worng since it's pointing to the package placeholder.
Image

A bit of massaging to help all the upconvert/writer/image.py stuff play nicely, spread between bugfixes, style preferences, and a feature addition. Details in the associated commit messages.
updated smalltalk.py

added 50+ new commands, cleaned up existing smalltalk.
working on adding more.
Offline compression with jinja2ext.py

Offline compression wasn't working with CompressorNode in jinja2ext.py. While fixing it I noticed that there was quite a lot of code duplication between CompressorExtension in jinja2ext.py and CompressorNode in compress.py, so I tried to pull out the common parts and create a CompressorMixin that they both inherit from.
real height/width

i had a case in which i have 3 columns, every column has margin/padding, but height/width function don't use this, so outerHeight/outerWidth is better!
new functions for PHPUnit_Extensions_Selenium2TestCase_Element_Select class
Fix for #62, use class.name strings instead of actual classes to determine ignored notifications

Changed made

- Added syntax highlighting to the readme (why not?) and updated it to reflect my changes.
- Changed `exception_notifier.rb` to use exception class names instead of exceptions themselves to determine whether to send a notification or not. This seems like a more sensible option as it shouldn't have to care if the exception is defined or not.
Removed unnecessary part of the CI definition.

I believe that the last sentence is not a part of the definition but it is just as an introduction to the article.

Thus, it should no longer be a part of the definition.
Corrected minor spelling mistake

Changed 'textursLoadedCount' to 'texturesLoadedCount' inside updateTextures() method
C3 Pool checkbox
update pt_BR to 6.2.0.0

update pt_BR to 6.2.0.0 and fixed wye translate in pt_BR
The new CMG developer skills test

This is at least ready for some scrutiny, if not ready for primetime.


Michael Angeletti
Our usual dist.ini customizations.

Using get to track versions, etc.
Resolve custom redirection

* Add after_verification_path method consistent with #10.

* Assign proper redirection paths in registrations controller.

* Remove application_controller to avoid `require_dependency` issue preventing `thincloud-authentication`'s controllers from utilizing overridden custom redirection methods defined at the application level.

* Reviewed with @bousquet, @philpper and @sean-j-williamson.

* Closes #13 (which would have solved the same problem, but with Configuration settings vs. instance methods).
Handelabra changes: custom background, VoiceOver, iPad, etc

These changes include:

- Custom background color support (via delegate addition)
- VoiceOver support for correct behaviour of toggle switches
- Popover size fix for iPad display
- Ability to show value on child pane cells
Issue175 progres bar on medical imaging demo

@ Jerome:

It appears that with the transition in the loading system, the progress bar was closed, by registering a task, and the immediately firing it. This caused the number of remaining tasks to drop to 0, causing the progress bar to close. By removing this, the progress bar only closes once the actual asynch task completes.
Selective Tile Render for Course Pages

See the Following:

[Main] (http://www.barnsley-ltu.co.uk/moodletest/MoodleFront/test/parseTest.php?id=1)
[Test Page] (http://www.barnsley-ltu.co.uk/moodletest/MoodleFront/test/parseTest.php?id=2)

Please that there is no code difference between the two pages. The dependency is in the HEADER notice: 
?id=`x`

Modifications have been made to the panel and SQL database to represent these changes.
Test, który się nie powodzi
Introduce simplecov to test coverage.
[TIMOB-7555] iOS: fix touch event flow for Titanium.UI.Button

[TIMOB-7555](https://jira.appcelerator.org/browse/TIMOB-7555)
Test case in JIRA.
timob-5203 added unit test to test all other properties for android.calendar

timob-5203 added unit test to test all other properties for android.calendar, get/set/create will be done by TIMOB-5204
LPS-24494 - Avoid forcing data post on entries clicking
Upgrade to svnkit 1.7.5

Upgraded from old svnkit to latest version.
Quelques corrections

Bonjour,
Je lis actuellement cet excellent livre et je vous remercie pour votre non moins excellente traduction qui me facilite bien la vie. J'ai ainsi pu arriver rapidement au chapitre 8.
J'ai néanmoins remarqué quelques erreurs ou oublis que j'ai corrigé dans mon fork. Si cela vous convient, vous pouvez les incorporer à votre travail.

Merci,
Jocelyn
Some tiny fixes. Added cause of restart.
Add aptitude to failsafe.

Adds the possibility to use Ubuntu in your Vagrantfile.
LPS-21697

stagingcommunity/sites

http://issues.liferay.com/browse/LPS-21697
LPS-24293
updated spec file with common package containing ddl files

provisioner and agent depend on common.  common contains ddl files
Add test for DFL-2502.
Update jellybean overlay/frameworks/base/core/res/res/values/config.xml
collective.stickyitem and collective.portlet.ngcollection repos added
User Story US16130: 	Allow user to add permissions in the config xml to enable different features (eg. to enable geolocation or to get the PIN etc.)

List of changes:
-Finished permission handling. We now write the list of permissions to the blackberry-tablet.xml.
-Updated existing comfig parsing unit test to include permissions verification.
-Created new unit test to ensure we add the access_internet permission if unspecified.
-Created "contains" utility function for Arrays
Pull request: Dialog crash bugfix

This pull request fixes a problem when size and position parameters are omitted when passed down from JavaScript. It also fixes bps not shutting down after the dialog closes.

blackberry/BB10-Webworks-Packager#132
blackberry/BB10-Webworks-Packager#133
Adds support for checkbox arrays

Checkboxes can now be given an identifier terminating
in `[]` to denote that the value attribute should be used
as an array key.  For example, given the following form

    <input type="checkbox" id="foo[]" value="bar"/>
    <input type="checkbox" id="foo[]" value="baz" checked="checked"/>

the model binding would generate an array `Model.foo = ['baz']`

Setting the value to another array updates the checkbox states, so

    Model.set('foo', ['bar'])

would produce

    <input type="checkbox" id="foo[]" value="bar" checked="checked"/>
    <input type="checkbox" id="foo[]" value="baz" />

in the view.
model magazynu i stanu towaru

+encja cennika
+inne drobiazgi
[spec, triv] Issue 7830 - Document public imports better

http://d.puremagic.com/issues/show_bug.cgi?id=7830
Format Fixes (Spring 2012)

This includes format corrections based on the marker's comments on my last work report.

- There need to be some text in between headings
- `References`, `Executive Summary`, `Table of Contents`, and `List of *` should have upper cases
too slow on i5's running at 2.4GHz, none of my boxes can do it either. Note that this issue is SD vs HD, SD is fine, HD is the issue, 1080i mpeg2 specifically. 

As far as I can tell and from talking with bob, we have no method to fall back to something that works and even HD is an issue for other platforms too. Basically you cannot use YADIF for HD content.
I think you still need this.  Links are still files, so they'll show up in the files list; they just won't be recursed into if they're pointed at directories.
Maybe use `mktmpdir` instead? http://ruby-doc.org/stdlib-1.8.7/libdoc/tmpdir/rdoc/Dir.html
@harrah This is the only convention in sbt that refers to `~/.ivy2`. But `~/.sbt` has become quite an ubiquitous location now, hence the change. Let me know if you disagree.
Sure, but i think we can do this even more simple - begin-rescue-end with require's inside? What do you think?
@tekknolagi Not really, depends on which side you're from. It would be funny and shocking.
Dude... You left your credentials sit here for a month before making them inactive. Epic fail.
Need to wrap this in a capability check somehow. Not sure if we have a mechanism at this point to determine the system capabilities, but there are many out there that we can draw on.
"I need more hours thinking about rearrangement topologies. "

I've spent hundreds on this and I still need more...
Use the super(Writer...) version and pass the delegate instead. Change all methods that call the delegate to use super.xxxx() instead
Oops, left the cast in there >..>
close the socket after ip detected.
Ah, in my defense ;)

-> isohybrid --help
Usage: isohybrid [OPTIONS] <boot.iso>

Options:
   -h <X>          Number of default geometry heads
   -s <X>          Number of default geometry sectors
   -e --entry      Specify partition entry number (1-4)
   -o --offset     Specify partition offset (default 0)
   -t --type       Specify partition type (default 0x17)
   -i --id         Specify MBR ID (default random)

   --forcehd0      Assume we are loaded as disk ID 0
   --ctrlhd0       Assume disk ID 0 if the Ctrl key is pressed
   --partok        Allow booting from within a partition

   -? --help       Display this help
   -v --verbose    Display verbose output
   -V --version    Display version information

Report bugs to <pj.pandit@yahoo.co.in>

There is a -o option (version on openSUSE 12.1)

At least I am not completely insane ;)
wait why do you need to do this?
Just a note, if rules are separated in one section, keep them separated in both files.  This is only a temporary measure to adopt changes more quickly.
`` ~ `` is handled well in the registries - well, that's not true.

if you manually do: dylan-compiler and then set personal-root `` ~/foo `` it gets expanded

environment variables are expanded by the shell

there is no normalize-path, thus `` .. `` is just left as it is. which is annoying in my opinion
ARE YOU KIDDING ME!!!!!!!!!!!!!!  Gosh, I guess I was so use to typing that.
Strg+Shift+O
What a way to screw up code style.
Yes but at this point that promotion hasn't been defined yet.
Just refactored this bit of code so it read, nicer.
Is this going upstream to Spoon? Definitely a good catch, and does make more sense as an exception than a replacement.

I'd resorted to a fix in `frontend/modules/profiles/actions/register.php` to add quotes around the email when it's used as the `display_name` to force the `To:` header to be valid, but then I saw your earlier commit, which resolved the issue.

With this change though, won't the register action always throw this exception due `$values['display_name'] = $txtEmail->getValue();` in https://github.com/forkcms/forkcms/blob/master/frontend/modules/profiles/actions/register.php#L123 (I may have missed something though, as I've not tested this yet).

It seems that the register action in the profile shouldn't really set `display_name` to anything at all if it doesn't have an actual name to use. Now, I've got it to set `display_name` to `null` in `register.php`, creating the profile works and the email sends (though with a `display-name` of the site name, which I'm looking to fix now).
Possible to add some config based switch for this in order to move to master? Something like (bool) SpellAuraHolders.SafetyOp = 0
Sorry about asc.jar.  Since it was on a Mozilla ftp server, I didn't even check the license.
"ellipsed"? :D
we already have migrations 314 and 315
#fff is shorter
You don't technically need this.  It's needed for uglifier, but you could just use closure or yui instead.  I'm not opposed to the addition, just letting you know.
How often do you think we need to separate the list of parameters? I believe that in most of the cases there will be only one recipe and using one more level in parameters will be redundant. Am I wrong?
Why is :vm_options an array of hashes instead of just a single hash?
AWESOMEEEEEEEEEEEEEEEEEEEEEEEEE JOB!!!!!
You should have a include/MotorFilesystem.h file with classes, functions declarations. Preprocessor instructions (#include, #ifdef, etc).
Then the actual code inside src/MotorFilesystem.cpp with a single #include to MotorFilesystem.h.

Give me some time to clone the Git repository, I should be able to do it myself.
Разрешить пользователям изменятьтолько контент, который они сами произвели. (Отметить для сайтов с контентом...)
Вы действительно хотите удалить пользователя из
It's the same change. It just happened to be broken to begin with.
replace [] with () so it's a generator expression, and it doesn't have to keep all the data in memory.
It would be better to do this in the view, but as this needs to go into the Mustache template too, which to my knowledge doesn't have the ability to truncate strings?



On 19 Jan 2012, at 19:55, James Stewart wrote:

> Were you able to replicate the problem with a test? Would be good to get test coverage on this given that we were able to deploy before finding out it was a problem,
> 
> That said, might it be better to do this in the view?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/alphagov/frontend/commit/7a1b6e4d8f9b8c6492348baba8e14eafa3d1e6e3#commitcomment-883440
Have you committed that? I would like to take a look at it.
No more crashes in pathfinding. Thank you ;-)
This should be in a feature branch!
also commit pyglet xlib.py file, with a fix for it to work in linux centos 5 distro.
this will cause `homepage`, `support_email`, and `support_url` to show up for the add-on submission flow. is this what we want? either way, there should be a test that checks this.
And by acting this like you are proving him right..  I have to prove Peter wrong every day, it's fun ;)
There's already a field named itemStack.
Hi , Is there any roadmap about social_stream ?
Is x even used?
fine to me.
No lo subo porque tienes errores
Hum pourquoi qu'il fallait enlever les <Player>?
should we then switch to listBucket for the test? ListBucketHandler/ListBucketResponseImpl probably need a subclass to add the extra metadata

yet another good chance to use ExpectTest, noting there's one for S3ClientExpectTest started
If you're going to keep the point size setting, at least use a proper integer spin control for it, not a text field.
withKeyName -> keyName for method names
Now you can dynamically generate your footers by just specifying their widths in an array
dupe ^166
Most of this is exactly the changes that Bobby posted in pull request #99, with the exception of where I have added comments.
looks good, thanks!
The commit is only in the addon, so you would have to compile this, unless the 2.0.2 version is already in the addons repository, when it auto updated when version did you get??
this commit is also wrong, this way it will print single characters instaed of numbers - proper fix is to cast the gender parameter on int
This is weird .. I made this change locally, but I didn't commit this file in my local before I pushed to github. Somehow ended up getting updated.
зор прикол =))) 

alive.h <- ?
LOL, was just typing you email if you implemented those flags :)
Fixed here as well. Thanks.
Whoops, you somehow managed to include that “.DS_Store” file in the repo.

You can have Git automatically ignore certain file/dir names in 2 ways:
 * put the name in .git/info/exclude (private to your copy of the repo)
 * put the name in .gitignore (which becomes part of the repo)
unfortunately, this not last commit... less then 1/4 work on WG maked. good work, but need much more...
Lulzactive governor is not working correctly. It needs to be fixed first.
May be we want to check the convention of @composer . I noticed all are using small letters . Please have  a look .
I have to agree with Shaun on this. I really have always had the tab close button setting set to false. When my dirty state indicators mysteriously disappeared I thought that I had ran into a bug. However it seems this was intentional.

I enabled the close buttons for awhile, but It's just too hard to distinguish at a quick glance the difference between the close button, and the dirty indicator.

I have opened an issue to draw attention to this change.
Thanks for fixing this Andrew!
t.co links are now usable again.
Also it will throw an error in strict mode.
You forgot to properly resolve a conflict here and it has been part of the history since. @jebaldwin you might want to reserve time to read the git diff -w superamerica/master in future iterations to sanity check things. @danbolt when you start next iteration please reset your branch to superamerica/master so we don't carry the old commits around any more. Feel free to ask me for help if needed.
@mrazzari The commit message and related issue explain why this method was considered. The text-indent methods are completely broken in IE6/7 when used on inline blocks. Windows-Eyes's market share is not as high as you suggest - http://webaim.org/projects/screenreadersurvey3/#used - and Jason's opinion on the matter is informed. Also, your condescending tone and unfair characterisations are not appreciated. There is no reason why this project should not push changes to the master branch and respond to any polite and helpful feedback from others before cutting a stable release.

@scottkellum

> Why did you ditch this and go back to the old method?

-> "using em's"
In theory, performance would be improved and memory fragmentation reduced if the array was reallocated in increments greater than 1, similar to the way the mFunc and mVar arrays are reallocated. The main point is to leave it unallocated until it's actually used, since most scripts won't use it.
Ok. YOU WIN THIS TIME!
What did you do with this?
Why not use node? (In general, it is better to use the objects than their ids)
@lgrammel  is there a cleaner way to do this?
On peut enlever cette ligne, en principe ce fichier est inglus dans OpenGL.hpp
fuuuuuck, my inattention))
and we are not your server providers... thanks.
From what I can see you're all claiming that the problem is in the commit without even checking to see if the creatures in question have the correct values for spawn position or in the InhabitType field? Currently there are lot's of creatures that have wrong InhabitType values set or spawn point that are incorrect, without this fix they appear to be fine but usually fall underground once killed. Unfortunately instead of fixing the problem with the NPCs you are asking the dev's to remove the fix because if forces the creatures to obey the wrong settings that are set on them?
You're right, should be in a separate commit
Usefull but very slow
I’ve just asked Eclipse to generate all the constructors for `CategoriesList` that its parent, `ArrayList`, has.
http://www.mediafire.com/?bjiz2bi16mdbcyk 

http://www.fileserve.com/file/ZaA5wA8/SERVER  donor 2012

there is my souce my db 0 errors review it
BBC 95% with alpha = 7, beta = 2
Wow, this is really great. I'm sorry I haven't had much time to work on yard-state_machine over the past few months (been concentrating on work and panops/opapp) but I'm am really glad to see you took this concept and ran (!) with it. :D :D :D
oke de map is weer zichtbaar. dat spelers doodgaan nog niet goed gaat gebeurt alleen bij de laatste die doodgaat in een ronde. hij zegt namelijk van: oeps, ik klop niet helemaal meer op basecanvas, doe mij ff corrigeren op volgende tick. maar die komt dan niet meer. maar da's wel gemakkelijk te fixen dus. ik moet nu echt naar bed. ik hoop trouwens wel dat ie nu superstrakke performance geeft op je laptop ;-) OJA! nog 1 ding. dat kruis komt achter de snake. ik heb ff kort onderzoek gedaan maar het lijkt me wel dat deze gecalled wordt na enige tekenfunctie dus ik snap niet echt hoe dat kan. minibugje maar toch 1.
Uh, really? Wrong ticket number I think?
@freelancing-god Any chance you could release a new version of ts-delayed-delta containing this patch? Rails 3.2 is out since a while now. Thanks!
Your doing it wrong! One sec, I'll fix from the editor -_-
sorry, its working
im trying to copy it from recovery. it likely ums mode on cm9 bit unstable
ok, i think i'll remove prebuilt fileman & add fileman source to repo
how about it?
Now, a full build can be done :-)
Okay.. I understand now.. thanks! :)
Fair enough, probably misremembered. Fine to leave this as-is.
This method should just look up the distribution name from the already-built egg-info static metadata. Pip builds this when an editable distribution is first installed, so it should always be there; shouldn't ever be a need to actually run `python setup.py --name`. The `InstallRequirement` class contains code that shows how to do this.
I think it's important for the gem to be predictable. Typing `t set active s` won't actually work consistently, since profiles are stored in a hash, which has non-deterministic order (at least, this is true in Ruby 1.8 and it's not a good idea to depend on hash order). You could sort the items in the hash, but even this wouldn't guarantee that `t set active s` would consistently select `sam` (e.g. what if you later authorized an account with the user name `s`).

Additionally, I think there are benefits to having the logic in `t` match the logic in `thor`, since `t` depends on `thor`. Once a user learns the rules of `thor`, it may be confusing if `t` follows a different set of rules for its internal logic.

In conclusion, if a user types an ambiguous command, I think they would expect to see an error. If users depend on this working, it could be dangerous. For example, if a user expects `t set active s` to switch to `sam` but it actually switches to another user and then they proceed to delete tweets or direct messages, these are irreversible operations.
You may want to simplify this. I understand you probably did it for educational reasons, but it's a wrapper for a wrapper for Y.Lang.isNumber.
This creates a shallow copy of an object:

```javascript
var options = Y.merge(options);
```
Weird, I'll check it out.
what about sun.misc.Unsafe?
I have no idea what "the bundle metadata and the generated poms" should look like. Please enlighten me.

As for the MultiNodeSpec, I just made sure that the bundle jars, which are the only jars we publish, actually contain classes and are usable in the best way I could think of.
why keep these commented lines?
lookup will instantiate a new dispatcher for PinnedDispatcher etc, so this won't work.
Put this logic where routers are created.
is it intended to include the `Set` in the string?
I'd need to jarjar them in any case.

If no extra dependencies are needed can this stay here or do you think extra module is right in any case?

I'll get rid of them. From Guava I am using LoadingCache but that can be done with locks and HashMap (same as remoting is doing) and for ASM stuff I can always manually parse the constant pool table from bytecode.
They are a bit of a pain, but don't we need something like this for a couple of things?
Not sure either. Sometimes, I find myself writing np.loadtxt way too many times for loading bvalues.
I think these should be removed
typo - please correct...
Check the ProcessKeyRelease. That will ignore keyup's that are actually key repeats.
Лучше наверно добавить return mkdir, а в коде выше if($this->checkUploadPath($path))
Ah yes, forgot about this.
opa.. esse comentário tá no lugar errado. Na verdade isso é pra o trecho de código abaixo, lá explico o que eu quis dizer. :)
Comment is also not accurate here.
As of 2.1.2, there should be two tarballs available, one is with dash (source), one is with underscore (debian source)
Hi,

I included the optional uploadProxy in the patch I'm working on and I will use the helper not only for setProxyFile so this "If" would conflict, can you close this request please?
I'm sorry, I didn't want to add this
This should be named `didSaveImage` or something similar.
This needs to set the correct number of steps. If there are two steps, and the first takes 20% of the time and the second takes 80% of the time, then this would be 20, 80, -1
This doesn't look like it's necessary if the Makefile doesn't use the variable ever.
I'm afraid it also contains a fair bit of Engrish, both in spelling ('avaible', 'build-in', etc.) and grammar.
Actually, fileinfo is not an extension, it's bundled since 5.3.0.
Or is it because of Scala friendliness?
Correct me if I'm wrong, but on line 146, I check it (if (! buffer.hasArray())).
If no array, then I throw an exception to revert to "standard code".

For your information, I try to copy to bytes, but the result were not brillant, and even worst (both in memory and cpu), so the reason I limit this whan hasArray() is true.

Did I missed something ?
This would probably look better as a """...""" string. Hmm, maybe you could define a dummy test function and use its docstring in the comparison, that would avoid the indentation problem and make any future modifications easier.
Is s and spkstr_cleaned ever freed?
Added as suggested. Also, throwing a warn event instead of console log.
I would be fine with that by #multiple_of is only supported on `Integer` right now. Modulo works with floats as well.
Yea I was unsure wether this should be documented or not. The methods are mentioned in the docs for render partial and I had a hard time finding examples of an similar utility value object. I'll go with nodoc for now.
Good catch! :) I'll wait for more feedback on the overall direction and fix it then.
About this I started to document copying what he already have here https://github.com/rails/rails/blob/master/actionpack/lib/action_dispatch/middleware/cookies.rb#L244
@frodsan can you take a look at all that?
Makes sense. I'll open an issue for this to make sure we don't forget.
actually, both @specialized type parameters and generated specialized members get the SPECIALIZED flag. updated the pull request.
do we need the asarray in the _xxx methods? the main method should only hand off arrays
the asarray is included in the _xxx methods not consistently, see _logpdf
The LIKE constant will already do a case insensitive `LIKE`. There's no need to lower it here. Also: by `LOWER`'ing it, you avoid any indexes already on the `name` column.
The spec would fail unless this change was made. The `originator` on "Some other credit" would be the `PromotionAction`, matching the new promotion scope and reducing the effectiveness of the spec since promotions + non-promotion adjustments would not be included in the order.
Class name needs `Test` suffix
should this be an error instead of warn? what are the semantic differences?
Nice tests! :)
I still think this should be tested numerically.
This was added below (see Returns).
On Fri, Aug 17, 2012 at 6:09 PM, Matthew Rocklin
<notifications@github.com>wrote:

> In sympy/combinatorics/partitions.py:
>
> > @@ -0,0 +1,681 @@
> > +from sympy.core import Basic, C, Dict, sympify
> > +from sympy.matrices import zeros
> > +from sympy.functions import floor
> > +from sympy.utilities.misc import default_sort_key
> > +from sympy.utilities.iterables import has_dups, flatten
> > +from sympy.ntheory.residue_ntheory import int_tested
> > +
> > +import random
> > +from collections import defaultdict
> > +
> > +class Partition(C.FiniteSet):
>
> I had some comments in the previous PR. I was curious if this should
> inherit from Union instead. This may or may not be a good idea; just a
> thought.
>
>
>
OK, it inherits from Union. There are some notes in the commit log about
making the change.
On 13.04.2012 05:37, Christopher Smith wrote:
>>
>> -    dtype        = DMP
>> -    is_Poly      = True
>> +    This base class should be used for uniform access to generalized polynomial
>> +    rings. Subclasses only supply information about the element storage etc.
>> +
>> +    Do not instantiate.
>
> The next thing I see is an `__init__` method. If one should not instantiate, why is it there? Perhaps the comment could be elaborated a little. (And forgive my lack of OO-understanding.)
>

Well let me describe my OOP ontology, and you tell me what the pythonic 
way is (or what you think should be in the docstring).

In my book, there are abstract ("virtual") and concrete ("ordinary") 
classes. Basically a concrete class is one you can instantiate und use, 
whereas an abstract class is of no use, even if you manage to 
instantiate it. In python, an abstract class is characterized mainly by 
some key methods raising NotImplementedError. (Of course this is not 
really necessary, one could just leave it unimplmented. But I feel this 
is good to have in order to document the interface.) I C++, such methods 
are called pure virtual, and if there is any pure virtual method in your 
class it cannot be instantiated.

This does not mean that an abstract class is *only* interface. (Although 
it could conceivably be. In C++ this is typical [because of strong 
typing], but with python's duck typing there is not much point to it.) 
Instead, my abstract classes typically implement user-facing costumary 
methods based on a few key non-implemented methods. There is an __init__ 
method that does some amount of initialization -- this really has 
nothing to do with whether or not an object of this type will be useful ^^.

In this case there is BasePolynomialRing which implements most of the 
methods that are conventionally required for a `domain`. The two 
concrete subclasses GlobalPolynomialRing and LocalPolynomialRing deal 
mainly with coercion/conversion and unit checking (namely, the stuff 
where they differ...)
`issed` added here since PHPDoc clearly says:

> the default value to return when the attribute does not exist

It caused an error for inexistent properties of the object before this addition.
I think putting OpenCMISSExtras in angle brackets it just going to confuse people, what's that supposed to mean?
Is there a task for this already?
My goal was to test Union on the reduce side.
Union alone makes it run on the Map side.
This should be flipped to be easier to understand:

    if ($e instanceof HttpExceptionInterface && $e->getStatusCode() < 500)
"depending **on** the adapter latency"?
To expand on that, this code works as long as `sizeof(unsigned int) < sizeof(int64_t)` - but think of the Cray port!
The reason is to a simple type defiition and check.
The override system should check the types. and nil allows override with any value type.

One solution is to define a string value as '' and ask:  copy(Params.path) unless Prams.path.empty?
Other solution is to have the type added by the user:
Params.parameter my_param, nil, 'desc', 'String'

Also, I do not like parameters defined with no value, I think it might lead to errors.
This table makes no sense.
is this method required?
kill this space and indent the paragraph three spaces for the best results
can be or is? 

can we make this information actionable for users?
I don't think those pyflakes warnings are "violations" of any sort. I mean, I could get rid of those vars, but then you just have confusing free-floating calls to create revisions
Add `default=''` to make sure that in the DB region is either set or empty string and not `Null`
Correct me if I'm wrong, but I think that this is just removing the files we don't want to purge from the _list_ of files to purge. The actual deletion of the files is wrapped in a converge_by in the purge_unmanaged_files method, so I think the above is why_run safe (but I could be wrong).
There are lots of comments explaining what's going on here, but I would still reference the CHEF- bug and give history as to why we've giving so much backward compatibility for config options.
I have used a built-in sphinx option --stopwait which attempts to wait until the running sphinx search instance actually finishes the shutdown.
It means just the basic notification (shows up in the user's notification list)
In other words, revert this hunk entirely.
I'm guessing this and main_actions events in particular, and quite probably other acp events, should gain an acp_ prefix.
nocking isn't a bad idea. We might be able to use environment variables instead, to setup the test environment:

http://expressjs.com/guide.html#configuration

If the NODE_ENV is test, then don't use the cache middleware at all.
Changed - 7c7843151c35628d8ab181d63db91f0975c087eb.
whitespace addition nit
This might not work as you expect it to, in lua you need to do `params.details.target and params.details.target or nil`
What if this fails? Full disk, permission errors, etc?
`Should_deserialize_list_of_complex_object` also fails for the same reason, with the same value. Failure to parse date string into ticks
static ctor is to create a Noop Completed task  something like string.Empty so new instance does not need to be created. it is one of the optimization tricks when using TPL.  I'm also using the TaskCompletionSource as &lt;int&gt; because it has been optimized in .net 4.5 since there is no Task.FromResult<int>(0) in .net 4.0

```csharp
static SimpleOwinAspNetHandler()
{
    var tcs = new TaskCompletionSource<int>();
    tcs.TrySetResult(0);
    CompletedTask = tcs.Task;
}
```
Maybe this could be made subcell aware to solve #2417.
I thought it was only fixed in the case of min/max, but not in the general case (sum, prod, etc.), but maybe I'm mistaken.
Nevermind, it looks like `theano.scalar.all_types` will always contain only strings.
The MAX_EXECUTION_TIME column appears twice?
Because you'll have an import cycle otherwise.
``if hostnames``. Use the same name you assume to have elements below.
Yvonne was the only one with an activation_code actually, but that must makes it more amusing :-)
This and the subsequent ones can actually use `self.assertQuerysetEqual` (https://docs.djangoproject.com/en/1.4/topics/testing/#django.test.TestCase.assertQuerysetEqual)
I implemented a fix in Django for the problem with empty or `':memory'` DB names in sqlite3 in [revision 17702](https://code.djangoproject.com/changeset/17702#file0) partially based on your work here. I decided to use the ``DATABASES`` key name of de DB description instead of `float("NaN")`.
What is the benefit of parsing the statement here as opposed to just returning?
I figured this was questionable. 

and I thought I had fixed the white space changes. sorry about this. I drop this patch and make sure the non-whitespace changes are fixe.
Not dead set on one or the other, it was just my initial inclination so I wanted to at least discuss a bit.  You have clearly thought about it more than me so I'll go with your version.  Thanks!
I understand what you are asking for now :) But I think it would be overkill for now when there are so few settings. 

Let me know here or on the original issue if you have any thoughts about how EclEmma should be updated. I'm particularly interested in ideas about the best way to switch between viewing coverage with/without source directives. My current thought is to do this through the existing View menu which already allows switching between instruction/line/branch coverage etc. 

On 27 Nov 2012, at 18:55, johnduffell by <notifications@github.com> wrote:

> In org.jacoco.core/src/org/jacoco/core/analysis/filters/CommentExclusionsCoverageFilter.java:
> 
> > +import java.io.IOException;
> > +import java.io.Reader;
> > +import java.util.LinkedList;
> > +import java.util.Queue;
> > +
> > +import org.jacoco.core.analysis.filters.CommentExclusionsCoverageFilter.IDirectivesParser.Directive;
> > +import org.jacoco.core.analysis.filters.ICoverageFilterStatus.ICoverageFilter;
> > +import org.jacoco.core.data.ISourceFileLocator;
> > +import org.jacoco.core.internal.flow.MethodProbesVisitor;
> > +import org.objectweb.asm.ClassVisitor;
> > +import org.objectweb.asm.Label;
> > +import org.objectweb.asm.MethodVisitor;
> > +import org.objectweb.asm.Opcodes;
> > +
> > +/**
> > + * Turn coverage on/off using ///JACOCO:OFF and ///JACOCO:ON directives.
> I agree eclipse shouldn't read an ant file. Also, we would like a global configuration for all of our projects if possible.
> 
> As an example of what we need, with checkstyle, we have one checkstyle.xml file checked into version control with all our rules and settings, and both the plugin and ant task are easily set up to use the same config. Something like that would suit me fine!
> 
> However, for just a couple of settings, it's not the end of the world, just a "would it be possible" type nice to have. You've done so much work already, this is really a minor point! =)
> 
> —
> Reply to this email directly or view it on GitHub.
>
I think you mean `wrapCast(UInt32,...)` here. Is there a test for `sign(Int)`?
It'll only matter when the unquoted value contains `)`, and I feel that the obvious parsing issues in those cases justify a change in behavior.
The code here and below looks so similar.  Avoid as many duplicate lines as possible.  The only difference looks like the addition of mArray, but I'm not sure.
Uhm I'm not sure why git recognized these as new lines..
Nit: Don't need the blank line here.
The comment form is noticeably uglier than the rest of the page.  But I guess that could be handled in a different pull request, or by the switch to Bootstrap, if you wanted to go that way.
It does, I originally was thinking if a batch got interrupted this way we'd still know when it ended, but that doesnt' seem useful so I'll make sure to change this
Can use the FileUtils `rm` method:

```ruby
rm bin/"rpm2cpio"
```
`name=` and `version=` are generated the corresponding `attr :foo, true` calls.
`capser`, really? :)
This may not be needed. It was in @smarterclayton's original commit  2730fc5b79592e2e39348b602b1bcb01408bad65, but will not affect this bug. If the tests pass, this is fine. If not, we can remove this line with no impact.
тут оборотом является всё " сделанных для каждого из этих файлов во времени", поэтому запятая после "файлов" не нужна. Возможно, стоит исправить на " сделанных для каждого из этих файлов с течением времени"
So this forces the DEBUG line to be printed no matter what the format? What happens if the default format were set in a logging config file? Would it just print DEBUG DEBUG:... and is that desirable ?
hmm.. bin ich noch nicht so glücklich damit... ausserdem frage ich mich ob "current" ein guter bezeichner ist....

Im frontend gibts ja immer eine clang aber im backend ja eigentlich nur in der struktur+content verwaltung..
Oops, I think we do need it to build the docs.
Putting that line back in.
RuntimeDaggerPlugin --> RuntimePlugin
but what introduced that garbage? There was no garbage before
`code > 0`
* superしか呼ばないメソッドをオーバーライドする必要はありません。除去しましょう。
I originally tried just using fluid.set, but it threw an error - hence this annoying function, and the additional logic in applySettings.. I guess I'm missing some sort of configuration parameter telling to create the path if non-existent.. I looked in Fluid.js and the wiki for documentation, but wasn't able to figure out how to do it.. I guess it would go in the config along with the parser?
This part is a check of versions.
It is intended to check the correct version before deployment.
The default version is SNAPSHOT. If you want to deploy, then the release plugin will ask you to type a new version, e.g. 4.11.
The release plugin requires the snapshot version before updating the version to release version. Simply, the release version does not have 'SNAPSHOT'. Then the project changed the version, and all the life cycles continues:
+compiling main project
+ compiling tests
+ up to deployment
Thus the maven central will get release version. But the development is expected to develop the project with snapshot.
This section is only a check, that the snapshot version has a correct form
<major>.<minor>-SNAPSHOT

Note: the incremental version (like 4.11.x-SNAPSHOT) was never used in unit. So if you mistakenly type the incremental version, the project fails. Same if SNAPSHOT is missing.

The best for you would be to check it out with makefile.
The makefile helps you to start it without any knowledge how the maven works. Just read the logs and the outcome.
I will provide it tomorow.
Unless we have dependent utilities, I would suggest making "version" "sourceVersion"
Deleting all this has no effect on what we're able to track?
Why were these tests were changed? I don't see any change to the underlying data, so all previous tests should pass. Same question for all tests.

It's OK to add new tests. :)
common name for messages, dialog text, etc.
Minor nit: Add a blank line between return statement and paragraph
I know this is a pre-existing variable name, but I keep getting confused between `$resizeCont` (class "resizing-container") and the class "resizable-content" that's used in other places.  Given what "resizing-container" really does, perhaps a name like `$resizeShield` would be better for this var?
I think we can safely assume the MAX_NUMBER_FILES > 1, so this can just be "dropping these files" instead of "dropping this file(s)" :)
English nitpick:

* looks "a function" is confusing.
* how about: "this method" ?