labofsahil/github-event-dataset-2012 huggingface dataset:
Caching is not working properly when the multi language support is enabled.

Hi,

Using your library to add multilanguage support I found a problem when caching is enabled. It causes always displaying the first page stored in cache, because CI uses the information on "URI Class" to make the hash of the file and it hasn't any reference to the language requested.

I just overwrite the "CI Ouput Class" to add the language code, stored in the "LANGUAGE" constant, and it works.

Greetings,
Ismael
Gingerbread

This is my updates.Use what you want...
Static media is organically connected to the app

With the new staticfiles app it was pretty easy to get this working.

Basically I just moved everything to the static dir, and changed the paths to be local rather than remote.

Not sure if you want to pull this in, but I saw it in the TODO and was doing it for myself already anyway.
Updated to work with uwsgi

Updated the init.d scripts to work with uwsgi
XML still being parsed when use_cache_flow is false

The fix that was introduced in 1.4.8 only worked when $this->use_cache_flow was true. Now $this->parse_the_response is checked in both places where $this->parse_callback may be called.
Javadocs and import cleanup
Allows the framework to support triggering callbacks with multiple arguments
all error folder
[Filter] fixed choice filter when the selected choice is '0'

This PR fixes a filter bug: when a choice key is 0 in a ``ChoiceFilter``, the filter didn't work (listed all items in the datagrid). Now it is fixed.
Description of Igor

I've written an Igor page, and linked to it from a parenthetical comment in the Shelley page.  I didn't want to mess with the sidebar.

I couldn't figure out how to view the resulting Igor page with images, but I tried to mirror the Shelley example sequence, so I suspect it all works out.
Updated file

:trollface:
Gh-214

Refreshes Sage's random state after forking.

Python's random() appears to be safe after forking since random.random() has used OS resources since Python 2.4, so such a fix is unnecessary for non-Sage mode.

Test code: Execute the following multiple times

```python
random()
```
Separate configuration of auto-brightness for button/keyboard backlights...

....

Presently, light-sensor based automatic brightness
(config_autoBrightnessLevels) controls both button and keyboard backlights
in addition to the screen backlight.  Unfortunately auto-brightness for
button/keyboard backlights doesn't work well on devices (e.g., epicmtd)
with neither (i) dimmable button/keyboard backlights (so when they're on,
they're very bright even if the screen backlight is relatively dark) nor
(ii) identifiable surface features on capacitive buttons (so they can't be
found if the backlights are permanently turned off).

For these devices, the non-auto-brightness behavior of button/keyboard
backlights, whereby they turn on during screen-on events and user activity
and timeout-to-off shortly thereafter, is preferable.  Thus, this commit
adds a config option, config_autoBrightnessButtonKeyboard (defaults to
true), that when disabled allows for automatic brightness control of the
screen backlight, but retains the timeout-based behavior for button and
keyboard backlights.

Also addressed is a bug that disables the button and keyboard backlights on
screen unlock, and reenables the backlights on touch events so that
capacitive keys may be located.

Additional fixes for button/keyboard backlight auto-brightness.

- Use sensor-determined brightness values when the button and/or keyboard
  lights are on, even if button/keyboard auto-brightness is disabled, since
  users may want to specify a custom brightness setting that disables the
  button/keyboard backlight entirely in low-ambient light situations.

- When button/keyboard auto-brightness is disabled, don't update button and
  keyboard brightnesses immediately if ambient light (the sensor) changes,
  as this may reactivate these lights after they've turned off in absense of
  user activity.  This is a particular problem when watching video or other
  circumstances when the lights are not expected to reactivate.

Change-Id: Idb26aa907ce4b04fb1f52a884e4e5bcb40fe695f

Conflicts:

	core/res/res/values/config.xml
	core/res/res/values/public.xml
Express: Fixed crash bug when remote server is down.

Express was sending a 204 no data when the remote server was down, this fixes that to instead be a 404.

Also, fed.wiki.org seems to be down.
Added export/import for role context levels (new in Moodle 2.1)

Hello,

We are using your "rolesmigration" report/plugin in our multi-server installation at Technical University of Lodz, Poland. This is a university-wide project called WIKAMP (Wirtualny Kampus, aka "Virtual Campus").

During our last year's upgrade to Moodle 2.1 I've added an option to export/import roles contexts (a newly introduced function of Moodle 2.1). Today I've commited my version to github.

Please have a look and maybe incorporate these changes into mainline. Keep in mind that this extra function would not work correctly in Moodle 2.0, as there is no concept of role context yet (added in 2.1).

Best regards,
Wiktor Wandachowicz
--
Teacher & researcher
Technical University of Lodz, Poland
Institute of Computer Science
Update Ukrainian/main/DownloadProvider.apk/res/values-uk/strings.xml

исправил апостоф
Fix a trivial typo in status codes

The SEED_WAIT code was the same as the SEED one in the TransmissionStatus enum (or whatever that js structure is :) ) and it doesn't comply with the transmisson codes.
Change the option[value=""] selector to "option:contains()" to solve strange symbols bug.

*Changed the "option[value=" + selectField.val() +"]" selector to "option:contains(" + selectField.val() + ")".

This fixes strange behavior when the option value field has ".", "-" or other strange symbols.
Changes for version 3.0

Many many changes - native JSON, enable Sites as FB apps, Spring 12 auth providers.
corrections relecture chapitre 4

tout est dans le titre :)
Can you make it a Q_PROPERTY ?
and add comment with what the default.
Hmm, is there a better way? This is just a fallback, so it doesn't have to be perfect.
Hmm.. so must have been some two stage commit. I did a git revert.. Would have expected the code before to compile, but apperently not.
All tests fail on my Win32 ActiveState Perl v5.14.2, for example:

#   Failed test 'Case Insensitivity'
#   at t/test.t line 7.
#          got: 'Can't locate Win32.pm in @INC (@INC contains: t/testlib) at C:/
Perl/lib/Cwd.pm line 758, <DATA> line 1.
# BEGIN failed--compilation aborted, <DATA> line 1.
没有评论当然返回空。。
Looks good other than the werr thing
One issue coming up repeatedly is whitespace handling. Suppose a an argument has whitespace in it, is it guaranteed to arrive as one argument on the `scalac` side?
Ahhhh crap....
We're using 0.4+ features, and considering going to 0.6...

Suggestions on how to handle this?

On Tue, Feb 14, 2012 at 6:23 PM, Ezra Nuite
<reply@reply.github.com>
wrote:
> It's the node.js dependency. Aptitude doesn't host anything past 0.2.? . It can be fixed, but adds a non-aptitude dependency.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/etsy/statsd/commit/82f64c18102b2fcb2f76cad79586757717ccfaf8#commitcomment-967067
Why do you pass in an ActorSystem?
The last point you bring make complete sens, having a tool tip manager or
tool tip decorator seems to be the way to go.

ctkToolTipManager or ctkToolTipDecorator look good to me.

Since the user base is small, we could also easily make sure that everybody
will be aware that ctkToolTipWrapper will disappear and that
ctkToolTipManager (or other name we will choose) will be the way to go. We
could also keep ctkToolTipWrapper

For sake of clarifying a little bit :)  What motivated my previous
statements is "readability" + "clear API" + "backward compatibility". The
design that currently exist is confusing and is mixing multiple concepts.
:)

Jc

On Sat, Mar 10, 2012 at 12:40 AM, Julien Finet <
reply@reply.github.com
> wrote:

> I don't think there is enough "feature" to split the class in 2. I believe
> 1 class that contains everything that is tooltip related is good enough.
> Every object can always be split into smaller objects, but the line has to
> be drawn somewhere. What if you want to control the delay, color, position,
> behavior, animation,... of tooltips ? You would create 1 class for each
> property ? You would then need a tooltip manager to control all of those
> properties because you don't want the application developer to manually
> instanciate all those classes. Why not have this ctkToolTipTrapper class be
> directly the "manager" of tooltip properties (display of tooltips:
> visible/hidden, plaintext/richtext...) and keep all of the work in 1 class
> ? Having separate classes sounds a bit over-engineered to me, especially
> when the logic of those can be written in 1 line of code (hide tooltip,
> convert to richtext)
> The name ctkToolTipTrapper might sound a bit too specialized and it could
> be renamed into something more generic (ctkToolTip, ctkToolTipManager, ...).
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/commontk/CTK/commit/4314c15836b87f2946665bda7e442ecd4593cb16#commitcomment-1068542
>



-- 
+1 919 869 8849
Makes sense, however I'm not sure to understand yet how it would work with the packaging... Sounds like a fun post 4.1 project. I would be happy to help if needed.
Please do!  Note that this will need some work to merge against master -- but nothing too difficult, I think.
ouch. I have uncommited stuff for that. I thought to use array of strings instead of individual strings for each case. I thought it would be more flexible to have an array. I might be wrong. Though one has to guarantee consistency, also locale should be overridden to get SQL parts i.e. non-translated array entries from default locale.
This line confuses me, it looks like "type" can either be a scalar or an array. I thought to rename it to "types" and have it always be an array.
ya i'm all about readability.  i don't usually use the inject method for that reason.  it confuses me when i look at it.

the first time i tried it i didn't.  that builds an array of all the fib numbers up to 4 million.  which is more memory than i have on my system, and it blew up.  but really we only need the last two numbers of the fib sequence on each iteration, so i'm dropping the first number in the array each time for memory management reasons

but i'm going to ask my boss if there's a faster/better way to do this.  mine had been running for an hour when i left for work this morning...
William? And yeah we can; backwards compat! :D These ids are like block IDs, should never change unless we want very messy code handling transitions for the rest of the codebase's life.
In the constructor of MGWTPlaceHistoryHandler you can put defaultPlace = homePlace so it will be flexible, people can use it or not...
The `x2` is represented by `v2` there =).
Yes, this assert is clearly bogus. Could perhaps change to "assert requestValue is None or isinstance(requestValue, basestring)".
you don't need to coerce this to a list `pager.object_list` is fine to iterate through it in the template
Ah yes.  Thanks for catching that.
This is just an FYI.  I'm not sure if it's clear from the API or not but you can shorten the 3 lines to `_vimBuffer.Process("%%%")`.  Either way is fine by me
I agree.  MIT did it.  So should Heimdal.  But it's up to Love, of course.  I'd be fine with disabling the warnings for appl or parts of it.
What about windows here ?
Doesn't ibus-pinyin have that feature?  Anyway I think that I'll extend ibus-setup-m17n (the setup UI of ibus-m17n) to allow users to select blacklisted engines.  As a workaround, you could always edit /usr/share/ibus-m17n/setup/default.xml.
Yep. But still, we'll need to figure out some day the proper normals for these side faces.
This is isn't quite true. `read-ns` returns a sequence of top-level forms (as data) and not lines of text.

We should rename this function to `read-file` since it now takes a `Reader` containing clojure source code.
I just backed out this change.  It was causing tests to fail.

What motivated the need for this revision?
atm I'm working on the sql part, so it will take a while. This patch should solve the problem, but I didn't test it:
https://github.com/PKX/TrinityCore/commit/e2296c2bc127fb4cdf71280f1475c5cf674eb6df
Mitch,

Sorry for my late answer.  If you are still interested I will review it and send you the pull request.
best, francisco


On Jan 10, 2012, at 6:10 AM, Mitch Garnaat wrote:

> This looks useful.  I've had some requests to add support for weighted records.  This looks like a good start.  Could you send a pull request?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/aficionado/boto/commit/c942428b337c27cc9b3be9369eb9eb49540247f8#commitcomment-853165
oops, should this be extend Sidekiq::Worker instead of include Sidekiq::Worker?
pex is superperms compatible...
You're right. fixing...
Please merge again, there were some noticeable regressions fixed.
What do you fellows think of these tests? I borrowed a lot from the experiment control demo, and am just checking that there sync pulses were added correctly and that onset times can be collected. I also subclassed StandardError to create a TimeoutError, which is useful for testing and describing what's going on, and added a timeout option to launchScan. All of those are opinionated decisions and I'm not sure you'll agree, so feel free to tell me what you think! Also, since I'm explicitly referencing the tuple, the test won't pass without my earlier commit, even on the case that currently 'works'.

Regarding the clock, Jeremy and I were talking off-github:

> also, I suspect that you might want self.clock.getTime() instead of core.getTime() ? 

And I explained I chose core.getTime() because...

> I used core.getTime() because it looked like the format similar to the ones 
> used by pygame (I think milliseconds since epoch, the time reported by 
> time.Time() [and aliased as core.getTime() on OSX) and not an elapsed 
> time from 0, but I admit I didn't check the format of self.clock.getTime(). 
> If they're the same format, you're probably right in that we'd want to use
>  the global clock passed to the emulator. 

I threw print globalClock.getTime(); print core.getTime() into the demo right before the loop started after the global clock was initialized, and found the formats; even though both getTime() methods have the same name they return different things:

globalClock.getTime()  =>  0.0158641338348
core.getTime()  =>  1329938687.99 

Since core is the right format for button presses (and when using core things look like they work right) I think we should stay with the original commit.

Hope this helps!

Erik
Isn't that fixed already (in 1.4.1 / master)?

davilla <reply@reply.github.com> schreef:

>libcec-1.4.0 needs it's configure.ac fixed to compile under darwin.
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/Pulse-Eight/xbmc/commit/355959aa9762a56729b5e2f79179284126892789#commitcomment-903116
This looks good.  Maybe log it as info instead of debug.
Looking good :) Send us a pull request when you are happy.
I don't think the effort to make job.record thread safe is blocking your patchset, @dgrnbrg. @jzupka, we can work on it on a later phase, feel free to open an issue to track the work, otherwise we'll all forget about it :)
This is a tricky thing to calculate indeed :) Although maybe I miss a point, but where do you update total taking into account potential change in future files?
Which part of this change locks the installed zypper packages? When I attempt to use this as a base, any changes result in the following text:

 Solution X: Following actions will be done:
  do not forbid installation of XXXX

I have created issue https://github.com/jedi4ever/veewee/issues/236
ааа! моск сломал на сон грядущий :-) завтра^W сегодня днем перечитаю 

в django templates в принципе нельзя делать `<%%>`
This worked.
I'd just like to point out that the Unix (X11) build scripts have not been updated
to reflect the new apu code. I'll try to fix them by looking at the Windows / gtk
build scripts.
PWM_LEVEL_ADJUST  should be 241.
At the moment the full 100% are not adjustable. (the oxygen-kernel has the same bug). (papatsonis found the bug)
PEP8 actually says to leave a newline before the closing triple quotes. but meh.
Thanks very much for this.

As it stands, it does not work as expected in Lua 5.1, i.e. require "curses" no longer sets up a module "curses". Please could you make this happen (only for Lua 5.1)? I suggest that you test the version of Lua, and call the "module" function for Lua 5.1, and use your replacement code for 5.2.
Thanks man, sorry about that.  My bad for being too busy to handle all of the merge requests in a sensible order
Close function prototype has changed.
Hi @juliantaylor,
your suggestion is very interesting, I have to confess I didn't know that feature of distutuls.
Thank you very much

Anyway we currently need to retrieve the library path so your patch can't be applied as is.
I guess there is room to improve our setup.py and I will try to do it ASAP.
If you have any other hint or a patch please let me know.
Don't forget to put these lines back as well, otherwise static files (in the media directory) will not be served. We don't have static files yet, but nevertheless the best is to add this code to urls.py before we forget.
CMake is as essential brew, I'll direct all angry homebrew users to you.

Also I agree with @bpiwowar if we no longer want patches in gists then we should update the documentation.

@bpiwowar if it's not a big patch put it after `__END__`.
It needs to be first, though, and I think the problem is that ActionDispatch::Cookies may not always be in the right place
I'm a bit confused about this line. Is it always right to define a new object here? Shouldn't we be attaching everything to `koExports` if it is defined? In other words, shouldn't this line be:

    var ko = typeof koExports !== 'undefined' ? koExports : {};

Please correct me if I'm misunderstanding your use of this variable.
As the JavaDoc states, DateFormats are inherently unsafe for multithreaded use. The detector has found a call to an instance of DateFormat that has been obtained via a static field. This looks suspicous.
For more information on this see Sun Bug #6231579 and Sun Bug #6178997.
This implementation actually assumes that the data is stored on AppEngine datastore. Please make the comments more specific to appengine. A link to datastore is even better.
MarkDown sytax PDF file.
Tsja, je kunt ook zelf testdata invoeren natuurlijk gewoon, om nou bij iedere testdata wijziging een nieuwe dump te maken is best kut. Tenzij Ab bereidt is op zijn server een hook te maken die wordt aangeroepen als er naar git gepushed wordt die automatisch een dump maakt ergens op de server daar.
Maybe verify file permissions as well?
The name should be `RsaTest`
Yes... it was a code that was already done. Basically I addressed the BasicEntityPersister by changing its logic.
It was just included in the commit when I pushed everything.
pefrect work thanks!!1
Here's Swig's compile cache, as referenced from the comment in `ender.js`
hey guys I'm seeing an issue that may be related (or this very one). could use a hand.


ActiveRecord::StatementInvalid (SQLite3::ConstraintException: keywords_users.created_at may not be NULL: INSERT INTO "keywords_users" ("user_id", "keyword_id") VALUES (1, 1)):


but I check the time stamps they're all good
RUN TO DEGABA key info created_at? 2012-02-05 17:41:45 UTC @user created_at? 2012-02-05 17:41:42 UTC

This occurred this morning while upgrading a rails 3.1.3 app to rails 3.2.0
this is wrong... please adjust 
if (normalize('/') == path[path.length - 1]) path += 'index.html';

to:

if (normalize('/') == normalize(path[path.length - 1])) path += 'index.html';

broked all the frameworks who used this:

railwayjs
express
etc..
за названия методов ататат
getMonsters мб?
すみません、今直してコミットするところでしたが、直して頂いてありがとうございました。
On unix, try the venerable "true" or even "/usr/bin/true"
I thought the same thing when I first saw it, but it works really well.
Start from year the script was born. So Copyright year is just 2012 (not 2005-2012).
How about using new shiny `send` function?

```python
pool = Pool(size) if size else None
jobs = [send(r, pool) for r in requests]
gevent.joinall(jobs)
```
Well, the fact that Python's default mutable kwargs aren't cleared between calls is actually a feature that get_module_info is trying to take advantage of for performance reasons. Having plone.testing remember the func_defaults of get_module_info and then restore them later won't work when get_module_info has been patched, because it remembers/restores the defaults of the wrapper, not of the original function.

I guess maybe what we need is some convention for storing the original function under a reliable name in the module so that plone.testing can find it.
@parndt Shouldn't that read `:parent => parent_page`?
Really Ted?  Seriously?
damn, there goes compatibility with my MySQL-specific Active Record add-on library.
That's an even better way of doing it :)
add in c68f42c97ded86f4f2e887154d5e29727b7c916f
as in 

heim_db_open(const char *dbtype, const char *dbname, .... heim_dict_t options, heim_error_t *error);

and yes, the option might be that the backend must require support for transactions or options to the backend that isn't passed though
for example, tblname could be passed that way
Hi,
The changing of the directory structure will break if users have custom forms,
forms from contrib/forms, and if users build their own forms via the
offered form building tools. Recommend using a much more lightweight
mechanism to assign specialties. Something like a mysql table that has
a sql column for the specialty (populated by list_id from Specialy list) and a sql
column for the form id will provide a much more lightweight mechanism do get what
you want. It will also allow assigning multiple specialties to one form (for example the
vitals form is pretty universal). This will also decrease the footprint of this code rather 
drastically.
-brady
I meant to call this commit "Added CRUD to sections"
@elemoine: If you look at a merge commit in github, you see all its parents. Now if you want to revert a merge commit, you have to check which parent is the one you want to go back. They are numbered from 1..n, in the order listed in github. So in my case it was parent #2, and I just said 

    git revert 12f191a690aa2e0166962553889ad1c4027d8254 -m2

with ```-m2``` pointing to parent #2.
Ouh la oui, c'est surprenant qu'on ne l'aie pas fait avant !
I had to replace this line with
    require File.expand_path('../base', __FILE__)
For it to work for me.
Can you please remove all of the binary files (.class, .apk, etc.) that can be generated from source? As a general rule we do not check in binaries since you run into the problem that the checked in binary files and source files can differ. You can add the bin and other related directories to your .gitignore file to prevent this.
Add "String... moreValues" parameter to allow caller to set multi-valued query parameter in a single call.  Maybe add an Iterable<String> version also
allow for filtering of posts in a topic by userID.
This change breaks two things. Firstly, the decimal separator is hardcoded now. It could be a comma instead of a dot too. Secondly, a price like "10.50" will be trimmed to "10.5", while it should keep its two decimal digits. Either trim all decimals, if possible, or leave the number as is.

If I'm not missing something, I'd like to see the regex restored. Thanks.
I also removed some unnecessary initialization of variables. If they aren't set by the .ini file or the command line then it should be an error.
I also adjusted some spacing on the output of the build information, which shows up as a big change.
Thanks. I've committed that version of the regex in d92066.
Same goes for assert_equal as well, correct, since it also prints out information?
!(http://g.static.memegenerator.net/cache/instances/400x/12/13259/13577396.jpg)
Hi ryan, 

I had some issues with this and ended up switching back to the main twython branch. 

I'll see if I can get it sorted out, but this implementation in too naive to bother pulling.
This should be gems2.example.com to be proper.
you might want to add .m too, for objective-c
sure it is!
PR please :)  You can edit the file directly online with github.
insert into gameobject_loot_template
instead changing delete table above
Okay, but there is no comma we have to add...^^
That said, this has been changed.
Really thanks for this commit.

@damnedsoul as @DDuarte commented: "Any error/relevant info, please report." please use tc issues.
This method should recursively output the whole expected tree structure (it is used in the message when the test fails).
I'm fixing the underlying issue with multiple gems in one repo, namely, the bundler gem tasks. I should be able to fix it tonight. Will require you to run on bundler edge if you want to release the gem, but other than that, should be good.
I think I got all of your new styling implemented correctly in the project. Check out the page on heroku and let me know.
color and color_light are calculated twice it's the reason why I dit it.
I had no time to generate a helper that would return the two values. But I reckon this will be nicer !
`ans` is not initialized and the standard doesn't say what state it is in if *getaddrinfo(3)* fails. I'd initialize it to `NULL` before calling *getaddrinfo(3)* and check for `ans != NULL` before calling *freeaddrinfo(3)*.
Let me know if you have any issues... I didn't have a chance to test it fully, or actually add the color values yet... but I don't believe it should be operational. Cheers
=nil;
This is stuff is all fine. This commit is the important one: https://github.com/6wunderkinder/CoreTextToy/commit/caf0e21b0894d26ffe59de4c88bd68f90f534523

Check the AppDelegate I added to see yourself.
false argument is optional.
We only have 4200 max on the swift
Check my commit message.
ha nice...
I'm populating a Backbone.js model with form values, that handles the validation and then calls jQuery.ajax under the covers. In my view I'm calling $("#form_field").actualVal() to pass in the values. I did have an issue in browsers that already support placeholders; $.fn.actualVal is currently being defined in the wrong scope. I moved it outside of the closure around the event handlers and that solved my problems. I'm not exactly sure what you mean by 'valHooks'. Can you explain that a little further?
>> Doom

I know the vision_oc
but 2011 xperia don't have "acpu_freq_tbl" in their kallsyms ...
Is this a mistake? It makes it impossible to start agilefant without modifying this file unless you have a system wide Tomcat on Linux.
Ok. Updated README file
I don't think so. StackMobQueryCallback.success would have to take a List<StackMobModel> rather than List<Author> and you'll have to downcast it. I'll work, you'll just lose that connection there.
I don't understand what you mean. I do realize I should have separated this into two commits, but what do you mean about git add *-p*?
How do I use it, and how would it help in this case?
@maoueh the only point is that you have to define the custom repository for the FB SDK in your base package as the SDK does not have a composer.json yet
@scottjehl 

`git checkout -b perf && git pull git://github.com/rwldrn/iOS-Orientationchange-Fix perf`

and open "perf.html" in a browser - wonder if that can be throttled
I saw the post on Gerrit just now! 
Thanks 

0002647302 <reply@reply.github.com> wrote:

>Delete
>/dev/msm_vidc_reg 0660 system audio
>/dev/msm_vidc_dec 0660 system audio
>/dev/msm_vidc_enc 0660 system audio
>
>Add
>/dev/msm_vidc* 0660 system graphics
>
>Video is fixed
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/IceColdSandwich/android_device_htc_ace/commit/81dcb5f5f74d0e4c13241cf2a9ff9257da9280af#commitcomment-1010059
XML Tag is wrong, it is actually
<HiddenBuilding>1</HiddenBuilding>
Problem would be that we need to override "widget_attributes". We have a bunch of different formatting themes in our application and we need this attribute for javascript based error handling on every form.

But its overhead you are right, this should be an optional field extension.
We have an organization and I will try to move this over there tomorrow and
of course you can use the code all I want is to get a stable build

On Sun, Jan 22, 2012 at 12:16 AM, SunRain <
reply@reply.github.com
> wrote:

> Hi, I'm porting spyder to gingerbread now(in porting bootmenu progress),
> and will push my code soon, could U make a organisetion and may I use some
> of your code?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/droidth3ory/android_device_motorola_spyder/commit/103aab66ea58af0f21e1d4f1405f394bd730f684#commitcomment-889461
>
'# factorValues -- renamed (used to be called "EFV"); can go
And that's OK for now, presumably? OK, I will implement.
Please put var declarations to the beginning of the function
Yeah, I just realized that I had a hard time shortening those names while still maintaining context.
A GreaseMonkey-script that adds a tooltip to the branches pulled from Assembla? ;-)
case msg @(_: FailureResult | _: MessageResult) => context.parent forward res
here I guess you have forgot `context.setReceiveTimeout(Duration.Undefined)`

but I think it is confusing to do that on receive of ReceiveTimeout, combined with throwing exc,

clear the timeout when receiving another ordinary message instead
and then that should perhaps be mentioned in the above doc, and not only refer to the conversion
I'd remove the default param and made a no-args apply method on MetricsGossip to return the empty instance, because right now if you do MetricsGossip() you get a new instance every time.
All test should be WordSpecs with Mustmatchers
missing space before brace
use foldLeft instead of for + var
having different types for different failures (which are then actually distinguished by the actor) is definitely good style, because then we can fully use supervisorStrategy to decide what to do
That would be linear in the size of the ring. The TreeMap implementaion in theory should handle subrange queries in the log(n) order... Maybe benchmark?
This explanation tells the reader nothing about what this is useful for.
So how would I get hold of the value during the transition?
nodes collectFirst { case (`myself`, i) => yes(i) } getOrElse no
I totally agree, but a problem with that would be that the `javaAsk` is immediately stopped in the below `finally`, and the whole point is ruined.
Is it possible to demand that the targetActor is stopping itself after processing, then the stop of the `javaAsk` can be done inside `JavaAsk` by using `watch` of the target and this finally stop is not needed. Perhaps demanding too much of the target actor?
still using "e" variable name here.  will break.
Yes sorry. I will correct this shortly.
The column widths for the row tabels are not set correctly on page load without a width equal to the header width.
Add tests for this, not well covered now.  I think it is working but not retested recently.
A single crcGenerator per resolver is not thread-safe, so create a new one everytime.
in finally block?
Agree, might be simpler
Good point, I should had some helper for integration tests and a config.
On a normal directory fetch, you call GetDirectory() with the path (directory) passed in to Update (with any filter argument removed) then set that (directory) as the path on the items straight after they're copied across.  So, any path set on the CFileItemList within the directory classes is being overridden (and unused).  This could introduce regressions if some other directory class is intentionally changing the path of the fileitemlist.
does this work on Windows too ? Btw, it does not seem to escape the arguments (for instance the path is likely to require some escaping)
s/can registered/can be registered/ ?
That should be fine, though I'll have to try it to be sure.
Now, the match is the entire matched string, not a Match object.  Instead we get an array of just the groups for this chunk, plus the card_params in case any methods in this subclass want to use them.
Yeah, they would have been reasonable before as well. Is just that I spotted it now :)
better (faciliate php dynamic typing, a core principal of php):
```php
$arrPks = (array) $arrPks;
if(!$arrPks) {
return null;
}
```
Please add .php
> In benchmark.c:
>
>>  @@ -105,9 +107,18 @@ int main(int argc,char *argv[])
>>     DUM_MATRIX = DUM_SOLVER+6;
>>     NO_OF_SPINORFIELDS = DUM_MATRIX+2;
>>
>>  -
>>  +#  ifdef OMP
>>  +  int mpi_thread_provided;
>>  +  MPI_Init_thread(&argc,&argv, MPI_THREAD_SERIALIZED,&mpi_thread_provided);
>
> did you try the performance influence of this change? When I tried it
> with the SPI stuff, it seemed to me that |MPI_Init_thread| lead to
> slower code...

I didn't test that explicitly, no. This could be on the sub-percent 
level I guess. We can also launch with 'funneled' if necessary because 
in most MPI implementations the serialized and funneled implementations 
are exactly the same except that the funneled one is slightly faster. 
IE: there is no danger in calling from threads other than the master 
thread even though you launch with 'funneled'.

Launching with MPI_Init_thread is the correct way to do things though 
and unless we can guarantee that doing a simple MPI_Init doesn't put us 
at risk of obscure errors, I would stick with MPI_Init_thread. Having 
said that, in many MPI implementations (this is what I read), 
MPI_Init_thread([serialized,funneled]) is exactly the same as MPI_Init. 
Only with the 'multiple' argument is thread-safety and locking really 
turned on.
`mv -rf web/meta .` to force overwrite of potentially already existing `meta/` folder?
Oh, haha, that should be commented out right here, so it won't be called. In case you wanted to use this with a UIView, you'd use associated objects and insert [self initializeInternal]; to set it up. For TwUI, we have the source, so it's already been taken care of.
I'm not sure I understand the "less checking" part – any code that would check for an empty array would work with `nil` as well (e.g., `count` will return zero and fast enumeration will simply skip the loop). Like I said, though, either `nil` or an empty array is fine, as long it's documented which one will be returned.
This should be a `static` function with fully documented inputs and outputs. This is impossible to read.
你还留着$isPC干毛....
ok, will look into it.. althoght using IntPtr is common in C#.
I'll look into it.

On 7/18/12 1:05 PM, "Robert Martone"
<reply+p-1191380-92464bb823c8cf6d80b195fd5d3d310e00ff0e96-1663570@reply.git
hub.com> wrote:

>> @@ -30,7 +30,70 @@ Carousel =  List.extend( function (List) {
>>      NEXT_EVENT = 'next',
>>      OUT_OF_BOUNDS_EVENT = 'out-of-bounds',
>>      PLAYING_FLAG = 'lu-playing',
>> -    PAUSED_FLAG = 'lu-paused';
>
>Are paused and playing flags used anymore? They should conform to how we
>notate states - see List
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/linkedin/Lu/pull/180/files#r1191380
It would be sad if that was a real highlight, and not simply something implied by the version number. I know there's nothing really big in this release, but what about ``where=``, ``vectorize`` improvements (#282), sample function (#151) or datetime improvements for example?
Didn't we decide that MapIterBind should have a return code? Why aren't we checking it?
The test for None is

```python
if argspec is not None:
```

(This holds for all the None tests below, such as ``default`` etc.)
поправил.
Can you add a `# :nodoc:` to this class?  I don't think this class should be considered "public".
This will change the global state of `Reply` for all tests that run after it. It would be better to create a new class and just use `self.table_name=` to set the table to `replies`.
It would be more readable to have it

    if vowels.include?(word.downcase[0].chr)
      "an #{word}"
    else
      "a #{word}"
    end
@dmathieu this was implemented in this way by performance reason.

See https://gist.github.com/d0d3f89e34d3fa654941
We can't do a mass conversion to 1.9 hash syntax in activerecord because it will affect many pull requests. Instead, we encourage people to write the changes or new code in 1.9 syntax. I don't think it will be more difficult, we don't have any agressive conversion plan, but this will make the transition smoother. Thanks.
I'm not sure that the user half of an email (`username` here) is always a suitable slug. See: capital letters, punctuation, and plus signs. Can we pass this through a slugifier?
On Fri, Jul 20, 2012 at 4:43 PM, Vlad Ureche
<reply@reply.github.com>
wrote:
>> @@ -242,7 +242,7 @@ trait ModelFactoryImplicitSupport {
>>
>>            available = Some(search.tree != EmptyTree)
>>          } catch {
>> -          case _ =>
>> +          case _ => // !!! [Martin]: One should never silently catch all Throwables. Please sharpen the exception type.
>
> It should be case _: TypeError =>. Should I do another pull request with this?

Yes, please do! Thanks - Martin

>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/scala/scala/pull/955/files#r1205800



-- 
Martin Odersky
Prof., EPFL and Chairman, Typesafe
PSED, 1015 Lausanne, Switzerland
Tel. EPFL: +41 21 693 6863
Tel. Typesafe: +41 21 691 4967
This needs to be scaladoc'ed. It's the only member the user needs care about, and there's no info about it!
That simplifies things.  I might make mention of the fact that any format
acceptable by parse_time is also acceptable by the VSO client. Thanks!

On Fri, Oct 5, 2012 at 5:52 PM, ayshih <notifications@github.com> wrote:

> In doc/source/guide/vso.rst:
>
> > +then download the relevant data.  SunPy uses the VSO through the 'vso'
> > +module, which was developed through support from the European Space
> > +Agency Summer of Code in Space (ESA-SOCIS) 2011.
> > +
> > +1. Setting up the VSO interaction
> > +--------------------------
> > +
> > +SunPy's VSO module is in sunpy.net.  It can be imported into your
> > +IPython session as follows:
> > +
> > +    >>> from sunpy.net import vso
> > +
> > +We will also be using the datetime object from the datetime module, so
> > +we'll import that too
> > +
> > +    >>> from datetime import datetime
>
> You should be able to skip all uses of datetime in the tutorial because
> both vso.attrs.Time() and query_legacy() will accept the tuple directly
> (since they call sunpy.time.parse_time() )
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sunpy/sunpy/pull/323/files#r1777922>.
>
>
> All the Series implement get_points. Hence added the alias for uniformity.

I see... Frankly I have no idea why I did it like that... Anyway, it
is coherent with the current style of the code so we may as well leave
it as it is.
And suddenly SymPy can do another one of my favourite things in FP languages.  Great job!  I'm looking forward to using this functionality.
We do have gcdex (and igcdex), but I don't know if there is a multiargument version.
I do not want this line; see line 125 below
This line can be removed entirely.
1. It does magic, it cannot be explained to mere mortals.  
2. I'll add some explanation and push back.
I don't know that it's worth mentioning old revisions. The docs just track whatever is currently in git.
IMO, you should replace function by method
If you're running it on your own servers you don't need the keep alive request. If you're running on Heroku, you can only make it work by requesting the public facing URL, which is always port 80.
By the way, I suggest you set `fs_fd` to -1 in `uv__loop_init()`. It's technically possible if perhaps inadvisable for `port_create()` to return 0 if stdin is closed.
can these 'set link = ' assignments be refactored into a macro and an argument passed?
BTW we already have the data in `$('#datetime-tip').data('date-end')` maybe you can use this one?
Moved the comment to docblock since it applies to the entire function.
The test functions are now identical.
Aici ar trebui sa fie Add Spell cred, nu Add Player
There are about 4 other places where we hand-make a "modules" config.  They need to be updated too.
I think  you also have to specify `@method` because yuidoc doesn't parse it out automatically.
what was the thinking behind not using _getSubElements()?
Is there an issue with that code, if so I'd prefer to fix that then use the getElements('input') - just makes things more flexible as this js class is extended by several different element plugins
jsbeautify used on it, there was an error because of tab vs 4 spaces.. fixed now
Full files coming in Diff from lack of correct files endings in original file, I believe.
Please link to suitable documentation (using sphinx rst cross linking).
ok let's do it when we have 5.4
should be elapsed
\Sky\Model\Request
You should use a rng with a fixed seed: look for utt.fetch_seed in this file for examples on how to do it.
I don't think so. There is the string conversion, but we might also need to change the type of C variables so that they can store the value.
I think `repr()` is what you want here, it is supposed to be the function that gives a less-readable, more-detailed string.
How does this prevent existing indexes that are getting dropped as the new index is being added from causing the new index to be deleted?
Hmm. It's still in the enum. We should probably remove that. My desire to not use it was fighting my desire to not skip a defined enum value. I'll remove it.
I add the sql exception checking. Thanks.
"to pull", this is still a part of the sentence above.
Dont use relative imports
`BEM.util.extend(...)`
emailIsNotSendWhenConfirmationIsFalse
I assume the optional registry arg is simply for backwards compatibility?

If it's not passed the registry that was just created will poof?
More like :runner: :runner: runner:  :walking:  :walking: :poop: for me these days.
this is not a BC break. PHP does not support named arguments so we can renamed them :)
This needs to be changed
This `try`/`except` can be condensed into simply:

``` python
known_hosts = env.get('system_known_hosts')
```

as `env` is a dict subclass :)
I realize this is bad. I just threw it in here to get the point across.
Missing the tabs/spacing here.
#Oops, need to group the EOL stuff
res = re.search(r"/*.**/((\r\n)|[\r\n])", fileData, re.M | re.S)
Requirements were not provided in original ticket, however I expect usage of this mojo from command-line, which means that it should be configurable from command-line, but seems that this is not possible now.

Moreover what about extensibility of this configuration? i.e. what if tomorrow would be required to enforce different coverage for different packages / classes, but not only total ?
Maybe just put the return on the next line? That only adds 1 character (semi-colon) and keeps the return value consistent.
This block of code (from checking numRows == 0) may have to be moved in a upper level, to be consistent with other modules that require args other than (x, y) to initialize the state, e.g., logistic regression (actually any iterative algorithms) requires a previous state.
Based on Caleb's comments, I will use random() instead. It seems the backend already provide a random seed. Then, I do not need set the seed.

I will change and verify.
Just put #define isatty _isatty in platform/windows?
Like the rename of the variables, for reason I couldn't come up with anything better than new and old when writing this. We could also consider having a ``INTERNAL_SCALE`` constant instead of repeating it.
I think it might be worth while to just call out to django-mobility if possible.
No particular reason, just I created the formula with the URL from GCC install manual. I agree making kotnet the default seems more reasonable. Will adjust in next commit.
I have to learn how to define dependencies with a certain feature. 
I am also about to make homebrew's python defaulting to framework.
you can use np.array_split for this:

In [51]: np.array_split(np.arange(10), 3)
Out[51]: [array([0, 1, 2, 3]), array([4, 5, 6]), array([7, 8, 9])]
I might be wrong here but are we potentially sending `event` messages to an inbound message queue?
About getting the application_name, would it be possible to have it when the consumer is firing the callback function? So the dispatcher get the information as a parameter. i.e. 

def dispatch_outbound_message(self, msg, routing_key=None)

If we only want to provide this kind of consumer for Dispatchers, we can imagine a Todo like 
1) subclass a DynamicConsumer to have an extra callback parameter
2) subclass Worker for Dispatcher  to use the new kind of DynamicConsumer

I would suggest a deeper refactoring extracting the choice of the Consumer from the Worker: Interface rather than inheritance. Then we have a 'consumer_class' in the config file of the worker (like what is done with the Dispatcher in a Router). 

Any other idea?
widths/colspecs name: Let's see what other people think. I'd lean towards getting things like this right, rather than copying every detail from R.

real widths: I think it's worth having, just because it would be so simple to use, especially if spaces are automatically stripped. Your example below goes from `[(0, 3), (5, 6), (8, 19), (21, 32), (34, 42)]` to `[4, 3, 13, 13, 10]` - much quicker to type and you don't have to think about fencepost problems. Of course, the more complex specification should be available if you do need the extra flexibility.

So the API as I would design it: `read_fwf(filepath_or_buffer, widths=None, colspecs=None, ...)`. Either widths (a sequence of integers) or colspecs (a sequence of 2-tuples) must be specified. Specifying both or neither raises an error. If `widths` is specified, it's simply converted into equivalent colspecs for the internal functions.
The component parameter is more like the module id of the component. It's a little confusing using the same variable name as in the CssRenderer.load(). I think ``moduleId`` it's better. If you decide to change this, also make the necessary changes in CssRenderer.load().
Simplified per this suggestion in 53862ff
It still wants JDK7 to for testing everything.
The constants should start with an upper-case character. Perhaps you can add them to a namespace instead of renaming them?
这一行修改也接受。其它所有修改麻烦撤销吧……多谢！
Good point! We should never get an invalid path. Probably should throw an exception. Later on, we need to add in some global exception handling. But for now, I'm fine with crashing the app if you pass us an invalid path. :)
@kcooney 
This will never happen because the listener will not call add/removeListener -only for internal use according to JUnit Javadoc.
Adding or removing listeners is done by JUnit core.
As I told you in the main commant, I want to avoid using COWAL because it's blocking -uses synchronization.

If you can see this sideeffect as an issue, then I can give much more simple non-blocking locks:
`
run() {
 tryLock()
...
 unlock()
}

removeListener() {
lock()
...
unlock()
}
`
I like it more above!
if we have a `var options = req.body.options` above and use that these will be easier to read the rest looks a-ok to me though
Is this the same as the `default` filter? If so should probs be an alias
It is the point of open-source, the community contributes, and the community must make sure their contributions are correct. See all ever pull requests on Spout where there is bad formatting.
Do we want to make this an unmodifyable list, to prevent plugins altering it.

Making a copy is good, since at least plugins can't break the core server, but maybe they could be protected against each other
I think the datalist item checked state should be a boolean instead of a string:
```
this.dataList[i].isChecked = true;
```
The name "copyPath" sounds like a specialized string function for file paths -- not the files/directories themselves. What do you think about exporting "copyFile" and "copyDirectory" functions instead? I think this is more descriptive.