labofsahil/github-event-dataset-2012 huggingface dataset:
Handle sprite imports correctly in SASS filter

Previously importing sprites would raise an error. The error is raised because `additional_dependencies` returns the glob used to impor the sprites. Invoking the pipeline would try to invoke a "icons/*.png" task which of course does not exist. This PR fixes the issue by using Compass's sprite import to expand sprite urls to absolute paths.
Added ability to add fields to the request sent to TransloadIt.

Added a 'fields' parameter to the Client 'send' method, so that fields may be added to the request sent to TransloadIt. I've maintained back-words compatibility by type-checking the new 'fields' parameter; when a function as the second parameter it assumes it to be 'success_cb' (as it was before this update).
fix for new express module

removed the createServer proto
pull-request-485
VerifyError fixed on Android 1.6

Wasn't able to run the dialog using 

AlertDialog.Builder builder = //
builder.create(); <--- Failed

One of the constructors failed in the stock 1.6. 

I removed the annotations, but not sure if they broke anything.
added events on appendComment on form_data.parent

In  #224 I forgot to fire a `fos_comment_add_comment event` when a new comment is added from the parent form.
transforming it into an "independent" gem + more support for rails 3

moved tasks tasks around, created a railtie to include both rake tasks and the gem itself on rails 3 and a bit of cleanup
High Availability notes
Added example apps and uses BuildManager.GetGlobalAsaxType()

This fixes the issue where HttpContext.Current.CurrentHandler and HttpContext.Current.ApplicationInstance are both null when the application is starting.
Added callback for onComplete

May be handy, nothing major just something I needed
Small typo in Gemfile

The gemfile should reference httparty (the typo was httpary).
Explain why this is useful

Hi Nico!

I added an explanation to the readme to explain why your gem is so helpful.
Event Fix to StringElement and SubCaption fix to CheckBox Element
Fix 2i unit tests when not using leveled as the backend.
[CHEF-2296] chef-solr debian package fail to detect existing rabbitmq user

Addresses http://tickets.opscode.com/browse/CHEF-2296 and should be backwards compatible.

Do we have a test rig?
cgg isue correction
Improved handling error and clean code

Responses are all in CJ format.
Clean code
Fixes to make the link selector from ck work
New activity feed widgets are going in the next release!

We have new activity feed widgets (made in angular, pulling in data from the remote couch), they will reduce the number of objects in memory, and eliminate teh instability in  the app when the user syncs a few times.
Add replaceCSSImageURLs

I needed a helper function to relative background image URLs in CSS files with the correct absolute path, so I made this helper.

It takes one argument, the root of the site, and uses it to form the new absolute URLs to the correct image content.

I've tested it with some pretty complicated stylesheets, and it worked flawlessly.
Add overflow: hidden to method accordion-group
Fixed logic bug in appendPartWithFileURL:name:error:

When checking whether the synchronous request to read file data from disk succeeded, the error condition used is "!error", but since error is of type NSError**, this only checks that the parameter is not NULL. Instead, we should ensure that either the error we were passed is NULL, or that it is not NULL and *error was not set to a non-nil value by the request method. This commit fixes the error.
Updated snippet encoder
Proposed Logger rework to fix issue 728

Hi,

this evolved into a larger patch then intended, but hopefully satisfies all parties. I tried to refactor the Padrino Logger so that all the fancy stuff @DAddYE built is still working and - as an added bonus - can be added to other loggers as well. This works by splitting the logger implementation into 2 modules for proper extensions (like the #bench method for benchmarks and the #devel log level) and one for colorization (as some loggers, like syslog, cannot handle colors). Extensions are always added to any logger that is passed to #logger=, Colorization is only added on the users request. Example:

    Padrino.logger = Lumberjack::Logger.new
    Padrino.logger.colorize!
    Padrino.logger.debug("Horay, a colorized Lumberjack logger!")

Also, I took the liberty of adding all those modules to a `Padrino::Logging` submodule, while keeping `Padrino::Logger` as is.

This pull request also adds tests using Lumberjack and the ruby stdlib Logger as examples.

Merry Christmas, by the way.

Best regards,
Florian
fix tokenizer and validates_format_for with activemodel < 3.1
replaced collective.testcaselayer with plone.app.testing

Please review it and fix it in case it is need it.
Constant data

All our data about schools, dorms, and other buildings? This puts them all in an `initial_data` fixture so that it's automatically loaded whenever we syncdb.
6681 - Regression with unions in CTFE

Add VoidExp to explicitly specify uninitialized variables.
When a union member is written to, set all other members of that union to VoidExp.

90% of this commit was written by yebblies. I've only added the union part.

Note: Please wait until this clears the autotester, it was not developed on my normal machine.
Updated docs to reflect provider name change
Advertising

We modify the advertising documentation due to changes in the API.
Bug 816922 - [activesync] Improve error handling and logging

r? @asutherland: This just improves error handling for the ActiveSync code, so things should be safer. See also the downstream commits in jsas and jswbxml:

https://github.com/mozsquib/jsas/commit/260e9e7b949c96b232178d136220afb8db70f240
https://github.com/mozsquib/jswbxml/commit/e69000c41a5fe2be0b1913c6b1e4d8840aa1b138

https://bugzilla.mozilla.org/show_bug.cgi?id=816922
Update Super Script with installer swap to see if that will fix 9.9.9 profile issue?
New *.ini files

I added some *.ini files, so I don't have to rewrite them all the time.

Benchmark.ini = WFS parameters
pre_ini.ini = pre_ini parameters
yaro.ini = Yaroslavsky 2002 optical parameters with WFS conductivity/perfusion.
change msm7x27

change msm7x27
Timob 7199 Option dialog implementation
Organisers section

A pull-out organisers tab on the top right of the page, contains some information as requested in your earlier email :-)
Issue 44 dstitle

Issue 44 - Long title overlapping content issue corrected. Configurable and tooltips enabled.
Changes to PDFViewer

Hack on the experimental PDF viewer:
* Make some strings translatable.
* Enter number to jump to another page.
* Display an error message (thrown exception as string) when an error occurred during load. Such as when the PDF is password protected.

I feel wrapper.xul is a better place to have user interfaces for pagination etc. but I don't have knowledge around there to code quickly.
Revision

ajustements chapitre 2 - deuxième passe
Merging v1.2.3 back into unstable

Unfortunately, there were a bunch of white space changes making the diffs kind of hard to read.
service-api-changes
Fix for README about internal zk handle.

Signature and usage was wrong as the callback returns ZooKeeperClient, though getHandle() will return the new ZooKeeper instance.
JBIDE-13330/JBIDE-13331

https://issues.jboss.org/browse/JBIDE-13330 - JBoss Developer Framework Quickstarts section on JBoss Central should be JBoss Quickstarts
https://issues.jboss.org/browse/JBIDE-13331 - Names of quickstarts are wrong
Change paren match highlighting

I think these are optimal colors. I took them from Emacs (though they
were already being used for other things, so I didn't go outside of the
pallet of course. The old colors made matched parens impossible to find
in a large text.

These are just colors that I grabbed and threw in and they look pretty good to me. However, if you've got a better idea for the colors, by all means change them. All I ask is that they be bright enough against the background that you can pick them up.
INT-2407 Remove Channel use="required"

Builds on https://github.com/SpringSource/spring-integration/pull/326
JIRA: https://jira.springsource.org/browse/INT-2407

    Parsers automatically generate the channel when none is provided.
    
    - JMX
    - JDBC
    - SFTP
    - Redis
    - Feed
    - XMPP
    - Mail
    - FTP
    - HTTP
    Polishing
LPS-24900 - User Management - Demo 2

 portal/controlpanel/organizations/usecase/demo1
http://issues.liferay.com/browse/LPS-24900
Updated Readme with a minor fix (added a new Line)
Here's my implementation of FastFullJoin...

Feel free to use this or create your own if you're having fun doing it.

It assumes the lists are sorted already.
Ffrhein splash mobile

Mobiles design.

sollte auf smartphones jetzt um einiges besser aussehen, konnte nur iPhone4 nicht testen, da es schwer ist die hohe auflösung richtig zu testen
Update sbt plugins

- add target and var directories to the ignore list
- update plugins for sbt 0.12.0, 0.12.1
- add sbt plugin for ensime (to use emacs as editor)
Adds array support

i18n.getString('root.sections.section[0].questions.question[0].text')
Update Ukrainian/main/SuperMarket.apk/res/values-uk/strings.xml
For some reason I can not figure out, it still needs curPageSpanPre and curPageSpanPost to be set to li tags. Otherwise all links will be inside li tags except the 'active' page/link.

So for a list use something like this:

$pager_options = array(
	//other options skipped
	'curPageSpanPre'	=> '<li>',
	'curPageSpanPost'	=> '</li>',
	'linkContainer'		=> 'li',
	'curTag'		=> 'a',
	'curPageLinkClassName'	=> 'active',
};
that's just :rtype: generator right?
These calling convention changes for the Plugin classes would be great to back-port to other apps sooner rather than later. I understand if that's more than you'd like to take on, but if you could summarize them quickly for me, I can take them through to OCS and OMP.
I guess movie data is the culprit most of the time, but is it possible to generalize this to all tasks?  Otherwise, I think it looks good.
Hi bjorn,

The indentation here seems weird, IMO.
Ok I think I have successfully ported everything. I am having some issues with my webserver not wanting to go through index.php so it keeps looking for subfolders but I don't think that is a problem with the code as much as my setup.  The only fields that I did not see in the file that I needed to change were...

plugins table - needed to add a 'file' column, didn't know the type so i just set it to text
tickets table - needed to add a 'assignee_id' column, the rest seemed to be BIGINT(20) so thats what I gave it
milestones table - needed to add 'name' - varchar(255)
milestones table - needed to add 'is_completed' -  boolean
milestones table - needed to add 'is_cancelled' -  boolean
milestones table - needed to add 'is_locked' - boolean


I'll let you know if I run across any others. Please let me know if I have chosen incorrect data types for these ones above and also specifically which time stamp fields were moved to date time as I think they were previously being stored as integer and have no way of differentiating which fields need changing.
85f38a270e8fc06467e9578da8878899a637c215
the same locks are needed also in 'get_interface_document' so no thread will get partial wsdl when asking it while it's generated
之前你的写法

```ruby
validate vote_result.vote.has_voted_by?(VoteResult.user)
```

语法都不对呀。。
自定义校验是需要定义方法的

```ruby
# 校验是否重复投票
validate :validate_is_revote_by_user
def validate_is_revote_by_user
  errors.add(:base, '用户重复投票了') if self.vote.has_voted_by?(self.user)
end 
```
Ok, if you really don't like shaving the Yak :) pull akka sources from wip-camel, use the AkkaBuild, work in the sub project akka-camel and don't depend on snapshot binaries? (well, that might include some Yak shaving, I don't know your setup)
Really odd this change was made.  "weird" that is highly subjective.
This naming structure is specific to sinatra-respond_to. The sinatra convention is to just use the .bldr extension.
C'est laborieux mais je compte bien y arriver!
`IFF` -> `IF`?
As you can see, there's a lot of inspiration in similar projects.
Хорошо когда Коммит логически полный. Тут миграции и схема были залиты разными коммитами
You the idea was having a command-line only short way.
lots of time was saved by this post ;)
If we change it to use {{Trampoline}} it will work in constant space.  I don't think there is a way to make it work in constant space out of the box in Scala.
I read Oleg's response to mean that while full set of laws is not agreed, `m >> mzero   =  mzero` is commonly accepted.

We can resurrect the instance with a warning in the documentation and test; perhaps that warning can be echoed on `PlusEmpty` itself.
what's wrong with this code?
I don't have openbsd installed and can't check.
yes.. and you need to disable the buttons (union) during the run of algorithm and then re-enable them, when the algorithm finishes...
Ah, yes. I was also wondering ;D
This looks cumbersome, why not just pass a null pointer in case no options are given and do a simple engineOptions != 0 check in EditGameDialog?
Yeah, I guess you're right. But unfortunately I still can't reproduce the problem. I have a project with fragments and support library. ImageLoader works fine there. https://github.com/thest1/LazyList/issues/7.
Get rid of processor count everywhere, use concurrency instead as we discussed.
Potencjalni odbiorcy tej recepty: Ilość osób z Kindle Fire << ilość osób bez Kindle Fire :P
It certainly should.  I cherry-picked the changes in [836f913a64930b91fa6d8350b3f8880f76f525ab] and [6d3c65926055f542b64be7e56e2c6383c9fb097f]
We keep bracing in such nested situations for better clarity, so this brace removing is wrong
@edgegfx Thanks for that

I have added it to the new branch too
Oops, I meant to remove the dependency on Model. Fixed by 5666aded52
Might be a stupid question, but why ` (function(){ return this })()` and not just `this`?
these types are implied
Why would `'Brain'.freeze` return `'Brain body.'`? Where does that `body` come from?
62218 isn't Launch, is Frenesí
Holy... this is completely intense!
When using a string in a method like this, it's best to use a `StringBuilder` and `Append()` or `AppendFormat()`.
Not sure why you don't see the link with WebKit... WebKit is what provides the remote inspector, right? Something internally there isn't working, and I haven't been able to fix it :) 

I agree that your fixes are much better--in fact I had originally done exactly what you did right here. It's just that when I was testing I was using webkit trunk (instead of qt 4.8) and I couldn't get it to work until I loaded the script in a file from the hard disk. So for now this is not a problem, but if/when we are using a newer Webkit and run in to the same problem, we'll know where to look and a short-term fix :)

I have a blog but haven't blogged about this. Good idea though. Until I get around to it, just do this:

./bin/phantomjs --remote-debugger-port=8080 script.js

<open 127.0.0.1:8080 in a recent-ish chrome>
set breakpoints, call __run(), etc.
steht ja auch schon im absatz unten drunter
Fixed DebugKit in cakephp/debug_kit@0b21dae47edef4a17d41f74fa72d2ed9c734b7c4
if this is a legitimate fix (seems fragile to me), you should probably add this to the [new marketplace developer hub stylesheet](https://github.com/mozilla/zamboni/blob/master/media/css/devreg/refunds.less#L30)
I think it is to reduce the amount of data stored for each request but I does not have a lot of meaning when the route collection has changed (thinking of an import here).

May be we can store a signature and only display the routes when the signatures matche and display a notice otherwise.
`messages.error(request, 'Refund already processed.')` maybe (and hide the `<form>` on the page)?
-  $param['forwardaddress'] = trim($param['forwardaddress'], "'"); 
   69  +  $param['forwardaddress'] = trim($param['forwardaddress'], '\''); 


Did you test this?  Escaping does not work with the literal single quotes.  You are trying to escape a single quote within single quotes.  I print out of this should show:

\'

Where I think you want:
'
May i ask which state the variable name with the suffix "open" should reflect? Is that an opened object or a closed that should be opened when you click the toggle? You mix up "open" and "opened" so i cannot figure out what you are trying to achieve.

Why do we have to use 2 variables rather than the previous generic one anyway?
Just in case anyone cares: According to <a href="https://github.com/SpoutDev/Vanilla/commit/ac245513feca39220f5aaf8b1e27f8c95eff3276#L23R26">this commit</a>, it was RoyAwesome.
I will try and revert back and recommit with a new pull request with changes as suggested.
it'd be neat to show a preview before submission. but that's just asking for too much. this is an admin tool!
Wouldn't that make impossible to click on the link to open the wave?
so i was right when i decided to offer the choice ;-)
Oh well, I've just noticed where was this print statement (in the function definition), I only looked at the diff not the whole file. :)

Forget (forgive?) my noob comment :D
1. вот здесь также можно применить cancan
2. не очень понятно почему здесь идет build коммента. Я так понимаю это новый коммент, почему его не сбилдить прямо на форме (тут создается лишняя переменная) - возможно я не прав и это оправдано
(I tested it with arrows not melee) So I don't know what happened.
Hejsan svenson
do we have quick and dirty documentation of the intended workflow that surrounds {merge,extract}_po.sh and compile_mo.sh?  Maybe on the browserid github wiki?  that would help me understand how these pieces fit together.

Also, I'm confused about MPL2 and usage of GPL code (the gettext js impl).  Should we check with gerv about that?

neither of these issues should block merging.  given others have already reviewed, I'm cool with merging down and working out kinks after that.
This old and busted form generator can specify initial values but you have to set them wholesale for the entire form by doing:
$Form->values(array("id"=>"value","id"=>"value"...))
It would be useful to explain the motivation behind this in the commit note i.e. the "why" as well as the "what".
Just a test now
I don't hate it, but I do like the whole notion of "under the hood" hinting to only venture there if you know what you're doing.
The problem was that we used MAV\_VAR as suggested in common.xml to transfer our paramters... and it did not work since the parameter data type is checked against MAVLINK\_TYPE in QGC...
It'd cost more to allocate the array needed to do it, (setting something to zero costs just about nothing) and this prevents any possibility that a call made during the event processing that (somehow) would up accessing this class again would see the state.
Just tested in IE8 on WinXP, seems to work fine. (I think I previously noted, this _only_ fixes the left-hand edit buttons, not the right-hand import/export, embed, etc. buttons.)
can i ask for what this one?
Just pinged if you interested in such change. I spent a lot of time digging autotools before got this right :)
Consider pulling lines 223-233 into their own method. Perhaps `ensure_inverse_exists`?
While I'm super interested in the idea, it was just acting weird sometimes and the egg files laying around just stressed my nerves. I hope to use a similar feature once distutils2 is out though, it's just the setuptools part that's smelly, IMO.
OK, I have addressed the issues. will ping vojta
Ok, thank you! Very good. Please like this in the future. 

I also got a merge conflict that I resolved manually. Please try to merge cleanly on your side in the future. The conflict was:

From https://github.com/nulogy/spreadsheet
 * branch            for_zeno   -> FETCH_HEAD
Auto-merging test/excel/writer/worksheet.rb
CONFLICT (content): Merge conflict in test/excel/writer/worksheet.rb
Auto-merging lib/spreadsheet/excel/writer/worksheet.rb
Automatic merge failed; fix conflicts and then commit the result.

Thank you for your time! Gem spreadsheet-0.6.8.gem with your changes is out.

Let us know if you find any more bugs.

Best
Zeno
We might be able to fiddle with this case. I think the problem was that the reifySymRef(sym.sourceModule) did not always give a module symbol, right? So taking moduleClass is wrong. The conservative thing to do would be to create a new freevar only in the case where reifySymRef(sym.sourceModule) gave you something that was not a module class.
Right-- this translator was a very quick hack. Thanks for the clean-up!
These need to be separate pull requests.
Can you simplify the code to this please.

```ruby
 c = if Money.assume_from_symbol && %w($ € £).include?(i[0,1])
       case i[0,1]
       when '$' then 'USD'
       when '€' then 'EUR'
       when '£' then 'GBP'
       else
         nil
       end
     else
       m = str.scan /([A-Z]{2,3})/
       m[0] ? m[0][0] : nil
     end
```

Notice I also put an else into your case statement.
Perfect simple example.
Yep, I can confirm this change fixes issue #69
I get an error compiling about non-existant HLintText. Should it be in the same project or is it in another one?
I'm curious why you would take away this functionality?
BGM就是桌幼稚的意思啊，乍一看还以为是BackGroundMusic……
So this would solve the issue of compilation under linux?
nope. I use codestyling localisation + poedit
Проверка на корректный адрес сайта
Hehe. May be "Consider upgrading"? :)
JD, ¿se podrá renombrar de `Generacion-de-la-Documentacion.md` a `Generacion-de-la-Documentacion.markdown`? El sistema que renderiza la documentacion los requiere así, sino podemos modificar ese sistema y pasar todos los archivos preexistentes a `.md`. ¿Que opinás?
Puedes Hablar En Español Conocemos Los Dos Idiomas
That would be Issue #11 :D
Cool. Never seen a LinkedHashMap before.

Did you find any performance issues ? What have been the reason the change the code here (your implementation is ok) ?

Shouldn't we use the LinkedHashMap for CacheCache,too ?
This javadoc is wrong.  ukWac documents aren't parsed, they just have labels.
Okay, this change broke the test case, @ahocevar I assume the test case can just be changed to use _blankImageUrl so:

''''
        t.ok(OpenLayers.String.contains(eventPane.style.backgroundImage,
                    tile._blankImageUrl),
             "backgroundImage of eventPane is set.");
''''

?
maybe "our customer - Erlang Solutions."?! ... Not sure.
Thanks for making sense of that :P
なるほど、ランタイムエラー = NonLocalReturnExceptionと解釈すれば良かったのですね。訳注として追記したいと思います。
vom implement her, gar nichts. nurnoch fixing HoR hab ich neulichst auch fertig gemacht, als nächstes werd ich den encounter reset bei malygos drachenphase fixen, denn machen wa icc trashmobs in c++ blabla, also todo list is noch recht lang

du gibst andere scripts als deine aus finde ich immer super 
https://github.com/silverteddy/SilverCore/commit/1f62d02b5dfd36386ab0b332345f8e5a8f48fadc#-P0
soviel dazu .... bist ein idiot wie die meisten machen privat repo und haben nur kopierte scripts da fragt man sich ob man überhaupt was selber kann oder nur auf andere angewiesen ist .... 

sry aber da platzt mir der kragen....
I know *wipes brow*

On 2/13/2012 10:47 AM, ihough wrote:
> Looks good. Pretty nice that we have a crude oil EF in FuelType!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/brighterplanet/numbers/commit/1dd8dc3c7feb32c7b20a3328d4bc5d56a25a2d71#commitcomment-961301

-- 
Andy Rossmeissl
co-founder and product design director
Brighter Planet
aim: brighterplanetar
office: (802) 458-0441 x302
freenode: #brighterplanet
twitter: @rossmeissl
http://brighterplanet.com
i dont think git-1.7.6.5.tar.gz ,git-1.7.6.tar.gz will have the same filesize and hash ?
I would normally use TTF as the prefix standing for *T*emp*T*able*F*lags

there are so many exceptions in this file you wouldn't be able to guess that
Does it have write access to the logfile?
```s/it //```
derp wrong commit message
This Cleanup have errors, please fix it.
Big Thanks!
Right, the JsfViewFactory of course! So we have to deliver the event ourselves then. The question is what the best way is. I wonder why PostRestoreStateEvent is handled this way in Mojarra's RestoreViewPhase and whether calling facesContext.getApplication().publishEvent(..) would work. I presume there is some reason why it's not done that way that we should probably understand. What does MyFaces do?

If we can't use publishEvent, I would prefer to see a private method in JsfViewFactory that does handles PostRestoreStateEvent explicitly also indicating the reason we had to do it this way. As opposed to having a general method in JsfUtils with a ComponentEventFactory. My point is that this looks right now like a very custom case. No other ComponentSystemEvent requires this type of handling.
It’s not a getter. And it is not just “get” account, it’s like “build” it from its name.
Right, but what drove me here is the if else if else... structure - implies exclusivity so I looked for the validation checks and there are none.

In __xml__ we can supply both release-strategy (a ref to a bean) and release-strategy-expression (which is ignored because expression is after beanRef in if/else).

All I'm saying is shouldn't we flag that this is illegal?
Managed to fix it.

For future reference, I had to:

    git reset --hard HEAD~1
    git push --force origin master # dangerous command
    git cherry-pick cb3350d
    git push
I don't understand why it is stated that masking avoids a copy of data. If masking is "using boolean mask", then this is not the case:

```
In [3]: a = np.random.randn(1e8)

In [4]: b = a[a > 0]

In [5]: b.flags.owndata
Out[5]: True
```

And by looking at top, I can  tell you that the memory is allocated for both a and b in my python process.
Currently, the value isn't overwritten!
prepending -- fixed the start-stop-daemon argument issue
This just does what line 31/32 should have done.
Hi Jon, jquery-timepicker now has a 'maxTime' option that will let you customize the dropdown list end time. You can make the list stop at midnight like this:

```javascript
$('#someInput').timepicker({
	'minTime': '2:00pm',
	'maxTime': '12:00am'
});
```
sure, check my last commit

regards

Atte.
Miguel Michelson Martinez
www.artenlinea.com


2012/2/13 Onur zgr ZKAN <
reply@reply.github.com
>

> Could you please write a simple example how to use logo, bg_color and
> border_color at readme.md please?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/michelson/paypal-recurring/commit/6b88b6d53e7a17e6f441da81a749d2274be05804#commitcomment-960521
>
Here I am !
1240,214,1,566,1,0,0,3,566,,,,6422,,,,,,,,,,,,,0xFF01FE,2,1,13180,360,1,
can we change this to '$(document).ready()` will only run once the document is loaded and the DOM (Document Object Model) is ready for JavaScript code to execute, whereas `$(window).load(function () {})` will run once the complete page is fully loaded, including all frames, objects and images.
Nope -- happens in both Safari and Firefox though:
List with NestedModel with a Text field that has a "required" validator.
Add new list item, put focus in that Text field. Press Enter without typing anything in it.

Without this patch the browser submits the form for me using GET to the same URL (the form fields go into the qs)
or maybe you can edit the script to include scripts such that it can automatically install clk or magldr version of the kernel ?
Yeah I haven't actually been running the changes I've made for a while.
Just programming what I think is right. I'll get around to bug fixing once
the heavy lifting of native conversion is done.
On Jan 29, 2012 6:07 PM, "Jonathan" <
reply@reply.github.com>
wrote:

> After you committed this, I am getting like error: Error: No resource
> found that matches the given name (at 'style' with value
> '?attr/actionModeStyle') in abs__action_mode_bar.xml,
> abs__screen_action_bar.xml and abs__screen_action_bar_overlay.xml
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/JakeWharton/ActionBarSherlock/commit/2b1cf650a135734e9ba391a4f2d6f7137258017f#commitcomment-912953
>
I take this back -- the repositories are different !?!
That might have been me actually >.> Wouldnt merge automagically so had to manually remove. Might have snagged that by accident too
it works now.. looks like I didn't commited to ics branch when I was changing it :)
move version to /parent/pom.xml and add it to dependencyManagement
NOTE: On further inspection, no reverse engineering was done in the implementation of the DSPManager.
@maoueh There is no other solution. Custom repositories are used by composer to retrieve the list of available package to create the dependency solver. So you cannot add them from the deps as you don't know the deps at this time (it is the work of the solver to determine them)
de fato, não teria como fazer login diferente em background, me equivoquei achando q era login apenas de admin.
hmm, ok. i sent pull request, you can view it. most things should be fixed.
Super, real good.
Thanks for the tool by the way. I'm getting so spoiled by it! :)
Is there a reason we can't move the `ensure` logic to `within_tmpdir` ?

So when `Dir.chdir` has finished it will clean up the `@trash` without having to nest twice?
Fixed - hopefully this is what you were after.
Nevertheless, if n_rows is zero, the return arrays will not be allocated, which yields a segfault later. Better to initialize all objects upon their creation and avoid "shell-objects", which are in invalid state and get filled up later.

Creating such "shell-objects" is a C-like style. Better to stick with C++ paradigms like "object instantiation is initialization.".
Where is this registered as the permission handler?
Once a reactor is installed, those imports will just reuse the installed reactor.  I created issue #171 to do further cleanup.
why wrap this inside NonFatal, this should be enough

    case i: InstantiationException =>
no need to use zipWithIndex if you're going to use a sum variable anyway, just use another var for the index.
I guess I should look at the code which uses it, but still: why is it necessary to give a Deploy if all we want is a cluster-aware router? The same is not necessary when using RemoteRouterConfig.
Yep. I agree. I fixed it to your favor. Along with all the other stuff. Take a look.
Not always.  I can fix this by bringing forward the changes that you don't like RE: trash.  I don't think it is slower, but we should have a clearer analysis of it all.
This method has no body, and there's a _db_to_form_schema() above, same comment as for form_to_db_schema(). Don't understand why example_iorganizationform tests aren't failing because of this
I'm not happy having a new controller contain a deprecated item suggest removal
The idea was to differentiate build_progress_report from build_final_report.

It's a relic of the development process. It's actually never even called outside of ProgressReport itself. The update command used to live outside of the report, which called this. Now that update is in the report, I entertained making this a private method to reflect that the user probably wants to call update instead. My OCD kicked in and I liked the symmetry to build_final_report and left the _ off  :)
this selects cells, not rows...
Only one of these seems to exist, and it's ../images, isn't it?
What if we have to sequences with the same name? Should have two maps I thinks.
that codec, width and height is the same doesn't mean extradata is the same after a channel change. we should still trigger parsing of potential new extradata. if it is the same, it will not trigger a re-open of codec, so it should be fine.
Will break %20 names on local file systems
change internal_name to external_name, or just name for consistency.
Actually, I'm wrong. This can support `nil` values as well.
Commit reverted
No, jsonp is json + param jsonp_callback, not it's own format (we wouldn't know what to call the padding function, if we didn't have the arg)
Also don't delete the `rescue Beanstalk::NotConnected` block above, for the same reason. Instead, replace `failed_connection(e)` with `raise`.
The terminology is a bit ambiguous here.  I would prefer to call them "typed messages".
Could indent the case block. I have mixed feelings about that, but the 'toothbrush' is more used than the 'comb' in Numpy code, so I went with the first for the style guide.
Also, please use a space after `#` for comments. Not a big deal, just consistency.
The original comment is correct, since "encoding" means returning a JSON string for the value.
Yes please. We had a pull requesting from @gazay merged yesterday only changing these style things on Active Support
Backwards compatibility I guess - there's existing code that calls `Model.select [:foo, :bar]` which means fields will be `[[:foo, :bar]]`.
"comentarios"
The retraction, for completeness, is that the above is only true if sym is not inherited. My earlier change to !isPackageClass is for exactly this reason; the inherited symbol has to get a shot at matchesInfo.
Must we duplicate this wrapper? How about making 'def wrapMissing' final instead of private.
Traffic cone? Pizza slice? Cheese wedge?
Hmm, what about using a `<macrodef />`?
The error made by choosing the cutoff is of order `1/(2 z^2)`, and floating point runs out of numbers around `z ~ 26`, so getting good accuracy by adjusting the cutoff is unfortunately not possible here.
Already fixed in master, but this should be ``signal.ricker``.
Why did you remove the check for `session[:currency]`?
this should have the deep option passed along:

```
>>> trigsimp(cos(cos(1)**2+sin(1)**2), recursive=True, deep=True)
cos(cos(10)**2 + sin(10)**2)
```
Such stateful code seems rather dangerous inside a library (sympy is not only for interactive use). And it is not really an object-oriented approach.
What is a difference between:
```
Piecewise((0, z < 0), (1 - exp(-z), True))
```
and
```
Piecewise((1 - exp(-z), z >= 0), (0, True))
```
?
Add a space after the first comma.
`exhaust`, `memoize`, `chain` all can be verbs. It would be nice to find a verb meaning "to add a condition" for this one.
This change would be like the change made for divmod.
If it is possible to get here with an attempt to raise a matrix to a non-integer power, I'd recommend checking for this explicitly and producing a more meaningful error message.
Also a typo. Should be "Unicode-Ausgeben". :)
I'm not sure what you're trying to cover here, but the only case this is real is when the base is ``I`` and the exponent is imaginary.
sorry I did not realize you were already there...
@resurtm I guess this should be removed ;)
Users aren't going to care about implementing a fallback to our failed api calls. They'd want us to do it for them or not do anything at all. If they don't like it they can edit this stuff from source.
Couldn't this just be subclassed from ContentSchema (instead of copying the schema nodes from there)?
You're right.  It looks like the filename-as-title functionality is broken.  I've just tried it in the browser, and the title is 'colander.null', as per your comment.  This is a separate issue then that needs to be fixed.
semicolons?! in Python?!
The problem is then that this will bind the providers to Silex ..
Тоже не нужен
any reason to comment this out rather than just remove it?
Please just tell me…


On 15 Nov 2012, at 16:43, Oleg Pudeyev wrote:

> In phpBB/api.php:
> 
> > +
> > +if (substr($path[1], -1) === 's')
> > +{
> > +	$path[1] = substr($path[1], 0, -1);
> > +}
> > +
> > +if (is_readable($phpbb_root_path . '/includes/api/' . $path[1] . '.php'))
> > +{
> > +	include($phpbb_root_path . '/includes/api/' . $path[1] . '.php');
> > +}
> > +
> > +$class_name = 'phpbb_api_' . $path[1];
> > +if (!class_exists($class_name))
> > +{
> > +	$api = new phpbb_api;
> > +	$api->throw404($user->lang('API_CONTROLLER_NOT_FOUND'), $format);
> There is no throwing of 404 in http. Please read the http rfc and find the correct terminology to use for referring to returning a response.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
I am not yet sure what the right solution is.
.. topic_in_queue')), $topic_ids);  has an extra closing )
Putting it as a string in the response object is exactly what I propose. Why is it ugly?
I prefer to write it as:
"camelCase"
(because it's sounds more auto descriptive)
Anyway "camel case" is also acceptable. I had never heard camel caps, though.
Yeah, that's where my habit came from. I had a particularly strict professor who taught my C programming classes who would dock marks off any assignment where there was a line over 80 characters long.
`if ` <-- space
@p This trick is "hmm, weird, suboptimal". Let's say row 1 updates fine (i.e. no unique key conflict) and row 2 does not. Is there a guarantee that row 3 updates fine assuming there is no key conflict?
No need to redefine the query here.
You should add a comment here as to why you don't need to handle an `err` parameter, or better yet, throw the error if there is one (it looks like none of the middleware pass back an error, but otherwise @Kami will come along in a few months and create some super-generic-noop-factory that injects code into the parallel that does error, then this callback will be fired before the other middleware finish and everything will end up in a mess).
I agree we should check for an update on startup AND poll about once a day.
state might be nil.
Any issues here with backward compatibility? 
Let's release this and see if anyone complains that their integer and float type nils are no longer showing as 0!
> Windows can have X11
0_о
For now its OK to have X11 REQUIRED, but it should be improved in future in context of Windows porting.
что-то я не вижу где заполняется таблица кэша
default.solr.server is now gone from pom.xml
There is now https://github.com/DSpace/DSpace/blob/master/build.properties#L23

Currently, this causes xoai import to fail:

OAI 2.0 manager action started
org.dspace.xoai.solr.exceptions.DSpaceSolrIndexerException: java.net.MalformedURLException: no protocol: /oai
	at org.dspace.xoai.app.XOAI.index(XOAI.java:156)
	at org.dspace.xoai.app.XOAI.main(XOAI.java:419)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:616)
	at org.dspace.app.launcher.ScriptLauncher.main(ScriptLauncher.java:183)
Caused by: org.dspace.xoai.solr.exceptions.DSpaceSolrException: java.net.MalformedURLException: no protocol: /oai
	at org.dspace.xoai.solr.DSpaceSolrSearch.query(DSpaceSolrSearch.java:39)
	at org.dspace.xoai.app.XOAI.index(XOAI.java:131)
	... 6 more
Caused by: org.apache.solr.client.solrj.SolrServerException: java.net.MalformedURLException: no protocol: /oai
	at org.dspace.xoai.solr.DSpaceSolrServer.getServer(DSpaceSolrServer.java:42)
	at org.dspace.xoai.solr.DSpaceSolrSearch.query(DSpaceSolrSearch.java:32)
	... 7 more
Caused by: java.net.MalformedURLException: no protocol: /oai
	at java.net.URL.<init>(URL.java:583)
	at java.net.URL.<init>(URL.java:480)
	at java.net.URL.<init>(URL.java:429)
	at org.apache.solr.client.solrj.impl.CommonsHttpSolrServer.<init>(CommonsHttpSolrServer.java:146)
	at org.dspace.xoai.solr.DSpaceSolrServer.getServer(DSpaceSolrServer.java:36)
	... 8 more
Ah, okay. So if you add scope:this to your Ext.Ajax request, then you can call controller functions from inside success or failure -- an example is in src/app/Session.js the getUserPrivileges function
Yeah, this solution is very bad. Could be specific to my system time -- which may be a minute ahead. I am unable to POST encounters if my system is a little ahead. However, I didn't want to change my time to fix it, as others will most likely have that problem as well.
naming consistency -- ```billItemAdjustmentId``` vs ```BillingItemAdjustment```. choose one and be consistent through all classes.
Preferred approach:

```
if (cs is not None and 
        not isinstance(cs, (iris.coord_systems.GeogCS, 
                            iris.coord_systems.RotatedGeogCS))):
    raise ValueError(...)
```
From: http://stackoverflow.com/questions/5879043/php-script-detect-whether-running-under-linux-or-windows

```php
<?php

if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
    echo 'This is a server using Windows!';
} else {
    echo 'This is a server not using Windows!';
}

```
Finish the sentence?
Table names are case-insensitive?
Too much JS.
Why didn't you just references the strings in opts?
No need for backslashes here, please remove them.
Unneeded empty line. Sigh, I think I should just say this in general :flushed:
please add a use statement for this class please
with the change i proposed on L64, you can just do:

```
return ret.join(glue[0]);
```
This file generates a list of files that need to be packed in the installer, and fills in the template accordingly. Misses some documentation...
This definitely won't work on linux/osx as they don't use anything like "C:\" etc, for path roots. use System.IO.Path.IsPathRooted
I think we should explicitly unicode-ify the name in this case.  (Or, if it's always unicode, we should str-ify it in __str__.)
I'm not sure about `ENV.fortran` though. At one point I had it set `ENV[FC]` but I see that isn't here anymore. Would be good to get @adamv's opinion Fortran for Xcode >= 4.3, MacOS >= 10.7 environments.
While this actually works (it correctly links against the /usrlib/libz.1.dylib), zlib isn't in HOMEBREW_PREFIX.

Since it seems to be able to find zlib fine on its own, this line can just be removed.
Why are we renaming the tool?
thanks for the explanation :)
Im curious.

now I get the ivar

On Sep 18, 2012, at 2:22 AM, Harry Jordan <notifications@github.com> wrote:

> In Classes/OMMiniXcode.m:
> 
> > +	if([projectDirectory length]) {
> > +		[[NSWorkspace sharedWorkspace] openFile:projectDirectory];
> > +		lastProjectDirectory = projectDirectory;
> > +	}
> > +}
> > +
> > +
> > +- (void)openProjectInTerminal:(id)sender
> > +{
> > +	NSString *projectDirectory = [self projectDirectory] ? : lastProjectDirectory;
> > +	
> > +	if([projectDirectory length]) {
> > +		[[NSWorkspace sharedWorkspace] openFile:projectDirectory withApplication:@"Terminal"];
> > +		lastProjectDirectory = projectDirectory;
> > +	}
> > +}
> I'm saving the ivar of the last project directory so that you can trigger the actions even if you're currently looking at another non-project window i.e. the Organiser or Documentation window.. hence the need for an ivar.. though you're right it should really be private..  selectFile: inFileViewerRootedAtPath: is also a good suggestion.. must have missed it.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
> PPS: Please rebase to master

How do I do that?

> try/except block and return false

I would rather not do that: The return value is lost while jumping from delete to delete through classes (see https://github.com/owncloud/apps/issues/240#issuecomment-10676418). Thus, the DAV client never learns that the delete action has failed. If an exception is raised and not handled, however, it is passed back to the client (e.g. for me, Thunderbird's plugin Sogo connector tells me that I probably do not have the permission to delete the entry). Otherwise, if returning false in case of an error, this return value should be check, e.g. by Sabre/DAV/Server.php, and it should inform the client. This sounds like much work since dependencies and interactions have to be taken care of.

Keeping it simple for now, until the code is refactored, deleteFromDAVData() can be condensed to two lines:
``$oldcard = self::findWhereDAVDataIs($aid, $uri);
return self::delete($oldcard['id']);``
Do we need to convert the tuple to a list?
^subreddit
Das ginge recht problemlos, aber ist das sinnvoll? Denn man wird ja in der Regel die Werte casten, bevor man sie in die config schreibt.
Wenn man es einbaut, müsste man aber denke ich den Default-Wert der Einheitlichkeit wegen an die dritte Stelle verschieben.
ja, das problem: durch den autoload wird halt firephp - falls jemand vor dem include der debug config irgendwo ne ausgabe macht - einfach mit den default settings geladen, d.h. enabled.. -> ausgabe pauschal, admin check greift nicht weil noch nicht "in place".. weiß akut keine bessere lösung. @r4 hab ich halt die includes vom check abhängig gemacht. das hat aber auch wieder n nachteil: alle vorher auftretenden firphp calls werfen dann fatals.. d.h. so ists fast besser auch wenn unschönerweise was im firephp core geändert.
Excellent, thanks :D
The level argument should be documented.
I think these messages are useful to have in the scaffold, but I'm not sure about the `lp_` prefix.
Ok, let's just remove that so it's simpler.
For Windows support, either path.join()
this doesn't seem necessary, where is it being called several times?
Did you forget something in the comment?
Out of curiosity, when would we use this flag?  How is it even passed? (e.g. if I pass args to brackets-shell, are they turned into URL params here somehow?)
Since the code above is hardcoded to _not_ pass undefined, we can remove this length check (or alternatively, move the arg and hardcode a reference to MAX_USED_COLORS down here).
No, I didn't see any issues. Just want to make sure I understand your comment.
do you mean lowercase experimental?
This line crashes if you select text that's partially commented out (including the first line).  It's assuming that every line has a comment.  Generally, the way other editors behave for line comment is that if the lines are at all mixed (there's > 0 commented lines _and_ > 0 non-commented lines*), then you always add comment markers, not remove.  This would mean the code up at top that sets ```addComments``` would need to use a loop for the line-comment case.

* for the Sublime-style behavior here, you naturally would ignore all-whitespace lines when doing this check.  The old impl does this too.
Instead of `!(DocClass && DocClass === CSSDocument)`, couldn't you just do `(_classForDocument(doc) !== CSSDocument)`? That seems simpler and avoids the need for the docClass variable.
the most common word in applications like ms office, browser and so on is "modifica"
Missing parens on the function call, which means the condition always evaluates to false.  But I think it's unneeded anyway (which is why the code still works): getSelection() will return you the cursor pos when there's no selection.
I noticed this too. I think JSLint expects it to be a fully formed html structure since it sees the .html suffix. Since this is really an html fragment it doesn’t have <html> or <body> tags.

I don’t know if the JSLint options you can put in comments work in HTML comments.


From: Jason San Jose [mailto:notifications@github.com]
Sent: Tuesday, August 14, 2012 10:12 AM
To: adobe/brackets
Cc: Ty Voliter
Subject: Re: [brackets] Combine jdiel's "language switching" pull request with tvoliter's templatization of index.html using mustache (#1358)


In src/htmlContent/main-view.html:

> @@ -0,0 +1,176 @@

> +<!--

I'm getting a jslint error for this html template fragment. I'm not sure what the solution is though.

—
Reply to this email directly or view it on GitHub<https://github.com/adobe/brackets/pull/1358/files#r1374550>.
why bind is needed? it will be called as `ext.disable();` anyway
So, if `scales$limits` is NULL, and breaks is set, then this will expand the range? I think that's not the correct behavior.

If you run this code, then the x range will include -20:

```R
d <- data.frame(x=1:10)
ggplot(d) + geom_point(aes(x=x, y=x)) + scale_x_continuous(breaks=c(-20,5), labels=c("foo", "bar"))
```
I haven't looked into what `ServiceAccessor.MakeApiCallPost` looks like, but I'd consider moving the `JsonConvert.SerializeObject` into that method, or at least create an overload allowing it.
I would just require that default is full URL.
It is more natural to do on `dispatch`, because condition should work for both request types.
When I initially coded that I thought it was too simply to be true. I'm slowly getting over that :)
`_isSaved` is really `_isPublished`, right? Might be better to rename that variable to discern between `project.isSaved`, `_isSaved`, and `_isDirty`.