labofsahil/github-event-dataset-2012 huggingface dataset:
Add tooltips for object actions

Add tooltips for object actions
added new colours 

@mecathcart added new colours in css, please review the codes. We like the new colours, turquoise and deepskyblue.
polish translation

I've just added polish translation
Just can't win with this one...


Sent from Yahoo! Mail on Android
I propose "Nevetheless, we could not avoid conflicting writes by schema design alone."

I believe the comment wanted "conflicts" to replace "it".

I think by adding "by schema design alone" we limit the statement to what is the case. We did avoid conflicting writes completely, and we did it by the serialising processes and session timestamping explained below.
yes, if there's a method to fallback to some other deinterlacer for HD, that would be fine. I poked and could not see an easy way to do this without rising side effects so punted and just disabled on osx.
No, it's just a rebased version. You reported a problem with this patch that neither me nor Ablu have been able to reproduce, so maybe you have some additional information about that issue.
added another class called Money which includes all three fields. Need to look into whether Django actually supports models defined this way...
Instead of something like:

    - ip:
        v4:

What about collapsing that and having a thing under network ```-type```

Something possibly like:


    eth0:
        network:
          - managed
          - enabled: True
          - type: ipv4

Also, the ```settings``` that looks like something set with ethtool. Perhaps there should be an ethtool module to set them. On redhat, can you actually set that stuff up in ```ifcfg-ethX``` files?
I would definitely think so, since there is already a `schema_search_path` key in the `config/database.yml`, that key could indeed be re-used in the rake task instead of flattening it to `public`. I will give it a try as soon as I get a chance to do it and report back (and provide a patch if I get to somewhere functional).
I will be submitting a new pull request with fixes for the tests that works on d5 and I am testing on final.
:+1: :star:
```ruby
options.empty? || options.any?
```

How does that make sense? They're mutually exclusive.. :)
Do you pull from my repo? Otherwise I just force an update on this commit.
0, 3, 1, 4, 2, 5はもう少し何とかならないかな。。
Your fix is obviously right, I was just explaining why (I think) it was written that way. Tom is really good overall about using absolute imports and not overriding builtin objects. I'll merge it later when I've got access to a pc of Tom doesn't beat me.

Sent from my iPhone

On Mar 3, 2012, at 19:29, Dan Colish<reply@reply.github.com> wrote:

> Here's the values implementation in cpython https://gist.github.com/1970445
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/dcolish/salt/commit/ba491efb9926f0dba0c64b608a4b9076e5f4052d#commitcomment-1039659
uh weird, thanks for fixing
On 1 February 2012 04:09, wkeese <
reply@reply.github.com
> wrote:

> Cool.   BTW my rationale for the suggesting using a single list for
> data-dojo-type and data-dojo-mixin was that it would be convenient for both
> your new code (doing the require() call) and also for _getCtor(), which
> needs a key to lookup cached values.   Currently it seems like _getCtor()
> is doing something a little confusing with nested caching, first level
> data-dojo-type and second level data-dojo-mixins, and I thought a single
> lookup key like "dijit/Dialog,dijit/Mixin1" would be more straightforward.
>
>
Yes, I realised the same thing.  That is why I didn't do it right away,
because I needed to make sure I could properly do _getCtor().  It is
confusing.  My morning mind I think just flipped on and realised how to do
it.

Of course the counter-argument is that _getCtor() will need to split the
> list into the first part and the remainder when there isn't a cached value.
>

Yes, but that should be trivial and will likely get rid of the recursion.
 One thing I have learned is that in a lot of ways recursion, while elagent
from a code perspective doesn't perform all that well in JS.
Ah I'll use another example. Thanks!

On Feb 16, 2012, at 6:50 PM, Kuzeko<reply@reply.github.com> wrote:

> I've tried to use the Autoloader with the Swift library and it doesn't work.
> 
> I needed to do 
> `require_once LIBRARY_PATH.'swift/swift_required.php';`
> 
> I suggest to clarify the example or choose some different library.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/laravel/docs/commit/42a86fdced2e4829a6f5ffd436464297bdc2aa53#commitcomment-976131
I see. It should be possible to implement #2930 now, don't you think? Replacing all "?" with the params should build the full query. I would suggest to add a link in the UI that simply alert() the full query (because copying to clipboard needs flash).
Oh, sorry the exact code is:

```
        ParseTreeTransforms.AnalyseDeclarationsTransform(None)(py_func)
```
Ill try to implent that aswell =)
This check was added in revision f19d801b16d4d8b75ef2c1ea9d99aa64a7874cf7
why not use $RBENV_VERSION? is that not the default variable anymore?
Oui, en effet, je ne l'avais pas vu. Réflexe.
ok, correct to say: stealth is removed in every Spell::prepare, because they do not have SPELL_ATTR_EX_NOT_BREAK_STEALTH
They were part of 32b32c2ac6b25f6ca867d72f23f2bb5b66d1cbee. Hmm, changing the comment will change the public history, but I'll do it anyway. After all, it's review ;)
why the `_stats` vs `stats_`? In mapnik a trailing `_` is used to indicate class members. Otherwise no need for `_` to indicate private things. Private things in C++ can actually be private members so no need to have the variable indicate that.
OK, now I understand your policy of developing qibuild. I'll modify the action soon. Thanks.
Cool, I was doing the same for a long while, but got tired of having to log in and restart it every time I had restarted my NAS. I've had it running for a couple of weeks without any issues, so it feels fairly stable. There were some issues in the beginning (some which I didn't quite figure out but after adding more checks they seem to have gone away).

My scripts for Debian and Gentoo are working ok, but not great. I don't know how to link the Python Daemon pid to the script, so I can stop the script by using /etc/init.d/rsstorrent stop.
For maintainability, I think it'd be better to add this condition to the one on L221. Returning early with "true" in this function means that any code added after it at a later date could accidentally be skipped by this early-return.
Those were in there to test whether it was picking up the right classpath, but they aren't otherwise necessary. Killed.
Do all the tests pass?  I'd be great if you could construct a test that reproduces the error so it doesn't happen again.
I'll try to change the hashing algorithm soon
http://pastebin.com/hb8H406L with this.
Fix also Shadow word death back damage
Why you don't say if the RewSpellCast is negative the spell will be hidden?
This would be simpler (core sql updates/ no extra column is needed).
In that case `Ptr{Int32}` does not correctly describe the signature of the function.
filesystem::FileReadString() ?!?!
Müssen wir mal beim Markus aufm Mac probieren :D
zum proggen is das (in der .bashrc) übrigens auch ne schlecht:
PATH="$PATH:/home/rbs/projects/Empire/bin"
alias emp='Empire'

da braucht man ne immer verschieben/kopiern
Thanks! I was hoping there’d be a better way…
You have to use `[1, 2, 3].last` instead of `[1, 2, 3][-1]`.
I might end up putting it in a repo of its own. Not sure.
Could make private and then use reflection. But hopefully the annotation is enough of a sign that this shouldn't be used
If you prefer shorter code:

    color = enabled ? 'error' : 'success'
track_object doesn't even exist in this scope.

It should be timeline_object here
в даларане лагает совсем по другому, к тому же "пакеты от текстур" это бред. пакеты идут от юнитов/ГО только, а их под далараном - ну десятка 4 максимум, слезы.
i not be move Sindragosa LOS (another principle already in testing) and for some other need find more correct place. need recheck :)
thought detail will still appear somewhere :/
@games RealmZone = 11 (spain)
18:06 <@fligtar> cvan: kumar: yeah, pp is required
The problem being that #vote is only a true boolean in some databases (notably, not SQLite)... was trying to make it work across as many databases as I could at that point. But considering the rest of the query broke SQLite, your point is valid :)
okay, this would solve the issues where people have another language then English. Season folders are not optional imo.
Thank you very much. We already built packages and informed the folks in the forums about the findings of today.
Ohh.. Do you want OR zero OR min length < 5?
The comment should have been: Added ipec tunnel test stub for issue #586
More line breaks if it's a multi line function def.
see comments on a1d43b63097a852908d993c6cee6b138b9aca1b4
Please talk to djs55 about replicating this in his xenopsd, which will eventually replace vmops.ml and related modules.
I'd refactor this one a bit so that everything wasn't created at the top-level. I like the pattern of creating the layout in one function and then setting up listeners in another using selectors. As it stands everything gets created when the file's loaded rather than when `start-explorer` is called.

Also, adding a `-main` is a good idea so it can be run outside of the repl.
I've got a report from guy with Mac: `clang` treats implicit conversation from `long` to `int` as error, cause `sizeof(long)==8`.
64bit Linux also treats `long` as 64bit value. It is not big issue, since we'll never so much loaded features, but `clang` demands to be correct with types.
We need to do both - .html('') to clear the message set earlier and .removeClass('error') to remove the extra bottom margin added by class "error".
https://github.com/Undergarun/mangos/blob/master/src/shared/Common.h#L177
Why do we block queries longer than a year? the UI and the users would find this useful. Do we truncate the table?
jQuery plugins seem to be working fine but notify me if something is wrong or does not work, I will fix that ASAP then.
Thank you! Yeah, you're right about this - I was blocked in my thoughts... We can already discuss and specify the interfaces even without knowing how we would exactly realize an "engine/provider" concept. Thanks a lot, I will point to this when discussing the corresponding point from the proposal.

About the engine concept: I think I have something for Java, but I still struggle to find a good way for registering a provider from an additional library in C. In Java, we have static initializers where we could call "KryptCryptoProvider.addProvider(new ThisProvider());", but what would we do with a C library for krypt-core (C). There is no real equivalent to static initializers, or is there? Unfortunately, we don't have the convenience of a Init_xxx method like we have for Ruby extensions, this would be just a regular .so that we link against. What could we use as an entry point during initialization there?
You are correct, I was under the impression it repeated until false. I'm fixing the bug now.
"return if not file exists" seems a bit of a funny way to write it.  

Is "return unless file exists" slower to execute?
A python idiom query - can the built-in `functools.update_wrapper` help with this?
@PheonixKotori : The issue with #35 was this line, basically. Chats worked before because we were just using 0 as the guid for all clients, so this went out to everyone. Now that we're actually assigning guids, we have to go out to the client manager to grab them.
hast da ne kleine macke drinne im random movement generator. wennde mmaps aus hast und denn ein mob random movement hat, rennt der nonstop in eine richtung
KnpPager merges the existing tree walkers to add its own instead of overwriting the array. Does it make sense to do it ?
we said sync so we start with that
copy copy, thanks!

Sent from my iPhone

On Feb 3, 2012, at 8:12, susanafong
<reply@reply.github.com>
wrote:

> I build my project against Google APIs 2.1 ... I recommend you guys do the same too so we don't end up with more methods that aren't supported on phones that have OS < Gingerbread  :)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/legendaryfox/bamft/commit/7f5fee1242bbb8a599af7c4a1a7e8be7859b9ba1#commitcomment-928451
Wow thanks..Make sense. I guess i would understand it more when i get to the routing guide section. I just added a commenting functionality to the blog :d
I actually made a follow-up commit that does this. See:

https://github.com/tarcieri/celluloid/blob/master/lib/celluloid/pool.rb#L66
That is the paths variable defined in stylus not lumbar. (stylus-images is a stylus plugin) Unless it's used somewhere actively I don't think we should provide the param.
if you have any further sync issues, take it up with github. it's using your preferred transport now.
so onpreloaded and onparsed could return false to indicate that the automated process should stop
the returned object could have methods to do:
 obj.parse(options, images)  // options being an object identical to the one passed to html2canvas above, omitted options would be taken from the options object passed to html2canvas originally (the call that returned the obj)
 obj.render(options, parseQueue, images)  // same as above

currently with FlashCanvas users are unable to know when the rendering is complete, because at the time the Renderer function returns the fake canvas, the Flash file may not even be loaded yet

asynchronous rendering would be possible with those changes, just return false from onpreloaded and onparsed and trigger the next step a little later

the object returned by html2canvas (the state object) should probably be passed as 'this' inside the callback functions

did I miss anything?
This should be lv.As(...) not tn.As(...).  This will fail with a nil reference panic when there are no explicit enclosures.
Use xlt() function:
Break it up into two strings:
Link to Patient (optional)
and
click to select
Thanks for the quick review!  Will have a new pull request in a bit.

I had missed MoveList.  I put the check in its own function to allow the short-circuit when checking for legal moves.  With MoveList that becomes quite unnecessary though :).

On Feb 5, 2012, at 11:08 AM, mcostalba<reply@reply.github.com> wrote:

> Because we have the lone king I'd write simply:
> 
> return        pos.side_to_move() == c
>           && !pos.in_check()
>           && !MoveList<MV_LEGAL>(pos).size();
> 
> 
> I would even inline at the only call place so to be more clear what we aredoing and because we don't need a new function for a simple condition.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/glinscott/Stockfish/commit/0347339970b68b998c7202e22c01560850dcd13e#commitcomment-934336
Adressenproblem
Then it's broken because it's not working :)
I was kind of experimenting with that idea. Hope would rewrite it soon with
some tests.
On Jan 1, 2012 3:02 AM, "John Hume" <
reply@reply.github.com>
wrote:

> This is interesting, but I'm having a hard time following it. Definitely
> looking forward to seeing the tests. :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/vinothkr/clj-record/commit/cf8c627fb65782c6606ca014326418d143f6a106#commitcomment-829989
>
这一块不能这么改，报过错，to:getMark("@kuiwei")的结果是一个bool值
commit message: fix(docs)....

otherwise LGTM
You could use this code instead of those hack fixes up there

```
-- Grounding totem AOE proc Fix
DELETE FROM `spell_proc_event` WHERE `entry` = 8178; 
INSERT INTO `spell_proc_event` VALUES      
('8178', '8', '11', '0', '0', '0', '98304', '0', '0', '0', '0');

```
Hoooly Crap. Yes.
Store the API url in a constant in case it changes.
ну да наверное)
а куда ты хочешь перенести настройки из этой вкладки? на вкладку подписок? но она и так нагружена нормально.
In the past, we had problems with some platforms which insisted on loading all referenced classes at startup time instead of on demand. But I think this was with some phones running Android 1.5, so my concern may be moot.
Mainly for these two reasons:

- testing purposes. Sometimes it is very useful to test c:geo not being logged-in esp. because we still have login problems for some users (maybe only on specific devices, don't know). Doing a logout we get into a well-defined state 

- if a user upgrades this membership from basic to premium he has to re-login. Re-login is normaly a sequence of logout followed by a login. Our UI heretobefore was not very intuitive.
Golly, thanks. But it definitely needs to be .rsplit('/', 1)[1] if you're going this route.
Michael, can you clarify, please, why this more strict upper bound is needed?
Is there some bug introduced by compiling clientsession along base64-bytestring-0.1.1.0 ?
These are valid points and I'm fine with this approach, but a few counter points for your consideration:
1. Facebook moved from the heavier weight "fan" to the lighter weight "like" to encourage interaction.
2. Facebook uses "like" and "recommend" interchangeably, so people might not be making the finer distinction that you are.
3. We could use likes on the backend but just display recommend on the frontend, same as you are doing with diggs.
4. It feels like duplicated effort to me to work on features of a plugin that are basically identical to likes. We're already spread very thin, so reducing work and making the work we do have the broadest effect possible is attractive to me.

I don't think we'd need to use it anywhere else. But discussion would be the only other candidate I can think of. It could be useful to reward people who ask good questions and give good answers by liking those things.
! right. This means, my Infinispan patch needs to be re-synced. will take me a bit more to double-check all parts work together as intended.
I read everything, but scripting is not my field
唯一find().toArray(function(err, items)这个方法成功操作过.希望有中文文档.
@jduan that isn't necessary
Are you checking out branches instead of tags?
Haha, 0 errors, 1000 errors, 10000000 errors, whats the difference if core starts and no crash in some hours xD.
@lucascs works fine! Plz, discart modification...
It might be worthwhile extracting asWidget != into a method. Similarly, asWidget.getSvgElement() could be stored as a field.
I the German...
It is not summoned by the boss, it is summoned by the player.
Hmmm, I did it this way to avoid doing any work unless this method was called.  I then populate the array on demand... but because that changes (an otherwise unused variable it can't be const.
melhoras -> melhorias (not sure about that, the second looks much better to me, but the dictionary does give a clear-cut definition).
Yeah I will do it tonight. I really need to update the whole core as lots have changed in the framework. Been swamped lately adjusting to the new job. :koala:
Hi,

I though you may have a reason to keep the "extends Model".
Of course I prefer to only change these 2 lines.
Do you want me to submit another pull request ?


2012/1/26 Lawrence
<reply@reply.github.com>:
> Would it be better to simply make these two changes:
>
> 1) change JPAPaginator constructor to:
> public class JPAPaginator<K, T extends GenericModel> extends Paginator<K, T> implements Serializable
>
> 2) change ModelPaginator constructor to:
> public class ModelPaginator<T extends GenericModel> extends JPAPaginator<Long, T>
>
> It doesn't look like you really change much else, and the two lines of code above should accomodate your needs without breaking existing code.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/franarge/Play--Paginate/commit/a695877410ce696c0d6e9cb9be9fd1373e61877b#commitcomment-904456
I don't know if platforms are supposed to support this algorithms (and therefore `NoSuchAlgorithmException` should be catched) or not.
Yes, use ReadValue + auctionSize. It avoids having to use many ifs
Good job, David!
The ExtendedListBox should be as lightweight as possible, because we do not test it. Therefore the ListBoxPresenter interface needs to be modelled according to the capabilities of ListBox. Having a removeItem(int) method seems to be a better fit.
Cool. I think, having the possibility to have @Configuration in constructors in most cases would be a feature people using constretto with Spring are going to use. The ones I know that uses Constretto without spring, seldom use the annotations at all. So in order to really get this feature to shine I think we should make sure it works with Spring as well
Testing the commit comments... Does this work?
Thank you for the comment,  it was just a quick hack I wrote to play the game (as I didn't read the code well enough to understand how to fix it).
Didn't think having money laying in different position would matter that much. I'll fix it as soon as possible now that you told me how to :)
Оно пишется в log. А пользователю HTTPError видеть не стоит
I think this should be

($ sel).fadeOut(200).html(res.children()).fadeIn(200)

and $tmp shouldnt be needed. Also, I doubt the html gets emptied only after the content is faded out... Are you sure about this?
What exactly is the motivation behind doing this?
Ah, I think this is in regards to line 69 of lib/Junior/Server.php, yes? Setting the ENV to blank here is not what we want to do though, we should update line 69.
This is why I hate these web front ends, Iv'e also seen them 'rip out' some include statements.

Yep we can do it your way, just I took the view that it was a demo and not production code, so not worth the hassle.

I have updated the code, and set it so that the variable code will be optimized away to a final.

will issue another pull request
Just remember to either 1) recover passwallet (best option) 2) invalidate it..
Maybe the old preprocessing/__init__.pyc file was still there.
The methods can't be overridden if they are private. Did you intend to make them protected?
#hshow even listener should be after this line.
Yes. It's better to connect on require "error" event and remove it when require process done. By this the "error" handler will only call once if require "error" occur.
see commit ：https://github.com/eric-wang/dojox_application/commit/cc031147695d1fbb805ab2de3a57845afcf58228
I am going to agree with Charlie here. Fork from the @endevver one, unless you can figure out how to grab those commits.
Answer of the first comment: Yeah , it is very useful to make a zip file and sign it if you have dsixda kitchen.
Answer of second commit: I don't understand what do you mean?
Why can you just pass a second parameter to gpc_get_string() that provides the default value?  Will that work?
gotta try this :trollface:
How about we lift this as <tt>Utils.generateRandomBigInteger(maxBytes)</tt>?
We have a report of this revision and the next one breaking Linux compatibility:
http://www.ps3mediaserver.org/forum/viewtopic.php?f=14&t=8883&p=65177#p65177

It doesn't seem to have solved the bug it was supposed to solve - considering the debug.logs we've had about the issue, I suspect this only worked around the bug for some users instead of fixing it - so I will revert it since I was already thinking of doing that before that forum post.
output archive name does not match myArchive name created earlier, should be myArchive.jar
Désolé pour toutes ces remarkes, au moins, c'est exhaustif! :)

Sinon ya ptet moyen de ne pas afficher à chaque fois ce bloc de js dans `Response(`

Je pense qu'une fois que tu auras corrigé tout ce que j'ai noté, faudra refaire une petite inspection et de l'éventuel refacto :)

Sinon good job, tu me montres demain!
This is a duplicate class definition
Maybe you could display this as info message instead when someone tries to use it?
Looking now :)
I think this broke something...those voltages are nowhere near the same as here: 
https://github.com/zachariasmaladroit/android_kernel_samsung_aries/blob/android-samsung-2.6.35/arch/arm/mach-s5pv210/mach-aries.c
https://github.com/zachariasmaladroit/android_kernel_samsung_aries/blob/android-samsung-2.6.35/arch/arm/mach-s5pv210/mach-aries.c
ok! will merge to test!
it's possible to avoid code duplication if you create method
```
protected Iterator<ExperimentResult> createExpIter(Attribute attr) {
                List<ExperimentResult> allExps = Lists.newArrayList();
                if (getUpExperiments() > 0)
                    allExps.addAll(atlasStatisticsQueryService.getExperimentsSortedByPvalueTRank(row.getGene().getGeneId(), attr, -1, -1, StatisticsType.UP));
                if (getDownExperiments() > 0)
                    allExps.addAll(atlasStatisticsQueryService.getExperimentsSortedByPvalueTRank(row.getGene().getGeneId(), attr, -1, -1, StatisticsType.DOWN));
                if (getNonDEExperiments() > 0)
                    allExps.addAll(toNonDEResults(atlasStatisticsQueryService.getScoringExperimentsForBioEntityAndAttribute(row.getGene().getGeneId(), attr, NON_D_E)));
                return allExps.iterator();
            }
```
in Expression class
The maven-javadoc-plugin is defined in version 2.8 in jasig-parent pom but your remark was relevant, the maven-source-plugin version is only defined in a specific profile in the oss-parent pom.
I will add the version for both plugins to avoid problems.
I suggest that the above keys be derived through the API and the appropriate class itself, to improve future refactoring efforts. I believe the classes are all publicly accessible. So something along the lines of:

```
import org.jasig.cas.adaptors.trusted.authentication.principal.PrincipalBearingCredentials;
...
this.put(PrincipalBearingCredentials.class.getName(),...);
```

 etc.
I can't help but this looks weird if you don't know the side effects the `data` property access has. Should we have a `Dat.fetch_data()` function which is then used by the `data` property?
The function has only been around for ....not very long.  I imagine any of these EOC functions have yet to be implemented by scripters.  I really don't feel as though this will break any scripts, which is why I suggested the name change.
I will remove that, but that will be done as a separate ticket (today)
CAtch all is fine here, Try handles NonFatal :-)
Braces superfluous
whenever e.g. sendSystemMessage feels like it
Hmmm, will test manually now
This should go with the other coord conversion routines
I think there is some confusion here. As I see it this "subscriptions" page is actually a page showing all the things you're following, so that you can unfollow them. So it's really a "following" page. Since we don't have an extension point for extensions to register new types of object that can be followed, there's no need for them to register things to appear on the following page either.

If we're going to extend the whole concept of notifications in CKAN to go beyond just activities from activity streams, then that's a different question.
Something is busted here, this gets generated as a mailto link.
This is exactly the same code as above in SyncStatusCommand.run(). In fact, the two classes are 100% identical except for differing default values for the description parameter of the init method. Can we just get rid of this class? Pretty please? :)
The fact that all three are four characters long placates my OCD.
Something I was wondering last time I looked into this code. Why don't we use a list of items that need additional art here (obviously negated and with &&) instead of a list that doesn't need additional art? Because if we add a new type and it doesn't make sense to fetch any artwork for it (e.g. "tag" right now) we never look into CVideoThumbloader and add it to this list (as you can see "tag" is missing in this list because of that reason). Buf if I add a new media type (e.g. "sport") and I want artwork for it I automatically look into this implementation and see that I have to add it to this list.
This stuff is ugly, create a dummy postprocess class instead.
You don't have to specify cflags_cc if it's identical to cflags.
Can we delete this log file please?  Shouldn't be committed
Yes, you're right. The new lines "bug" is something related to my XML formatter. I've merged all changes and adapted accordingly. Thank you :-)
This one should probably have a simplified api too, but I won't add an more to this branch :)
Il faut décompresser dans `/tmp/`
document these size functions.
Well, there is also another problem to solve - how to load some topic page in FluxBB_Home_Controller::get_post(). I mean:

		// FIXME: second parameter for $page number
		return $this->get_topic($tid);
Raising an exception seems a bit drastic.  Perhaps an NSLog() would be more appropriate (to fall in line with that UINavigationController does).
I prefer "extends" instead of "overload", as attribute name
It would be nice to change this example to do something more interesting than +=; for example, maybe you can use ++=, a method that adds multiple items to a Set and would not be available on a normal accumulator.
It seems like what this example actually does is send the content of the same file 10 times consecutively.  I'm not sure this is a good example.  How can a user test this example?  There's no client part.  Could we just write an example like:

* A client sends a list of files (DelimiterBasedFrameDecoder)
* A server sends the requested files (e.g. `<filename> <length>\n<file> ..`)
I think I will leave this until issue [1892](http://projects.scipy.org/numpy/ticket/1892) is dealt with.
All memory leaks that I could find have been rectified. Please test again...
Done. It now returns a `zend_bool`...
Didn't seem useful if it was in a file :) Will change to print_metadata_stats.py
у нас как-то принято через /// однострочные doxygen-комментарии писать
oops! will fix, rebase, and push.
Can we use `merge!` instead? `self.attribute_aliases.merge!(new_name.to_s => old_name.to_s)`
Thanks for the feedback @josevalim. Are you thinking we should provide an option of which formats that should be escaped or a list of those which should not be escaped?

Do you have any preference for what the configuration option should be called?
We should not escape the block only the text.

Try to change it to something like this:

```ruby
content = ERB::Util.html_escape(text.truncate(length, options))
content << capture(&block) if block_given? && text.length > length
content
```
Could you kill this extra space and amend? :D :scissors:
The Select tree must not come from mkAttributedSelect in that case.  It looks like makeAccessible does the same thing, but will not have the same bug because it uses "isInPackageObject".  Ah, so many code paths to approximately the same place.

There are also possibly ugly implications to the fact that isInPackageObject will give false negatives if either the package or package object symbol is not initialized.  I don't know if that ever happens.
I agree this needs to be worked out. However, we currently have a request to use custom hashing on TrieMaps. I think we should at least enable the TrieMaps to take custom Hashing objects for now.
Given that we don't do this with the rest of the collections, it should not be hard to switch to a different strategy once we figure it out.
Didn't we prohibit assert(false, ...) ?
I think peek() would work
that sounds good. but let's be more inclusive. Let's say the observer needs to be inside the solar system so < 40 au.

Steven Christe, Ph.D.
Solar Physics Laboratory, Code 671
NASA Goddard Space Flight Center
Greenbelt, MD 20771-0001, USA
Telephone: 301-286-7999

On Mar 30, 2012, at 11:02 AM, Keith Hughitt wrote:

>> @@ -221,6 +224,16 @@ def _draw_grid(self, fig, axes, grid_spacing=20):
>> 
>>         return fig, axes
>> 
>> +    def _validate(self):
>> +        """Validates the meta-information associated with a Map.
>> +        
>> +        This function includes very basic validation checks which apply to
>> +        all of the kinds of files that SunPy can read. Datasource-specific
>> +        validation should be handled in the relevant file in the 
>> +        sunpy.map.sources package."""
>> +        if (self.dsun <= 0 or self.dsun >= 2.5e11):
> 
> hehe, 2.5e11 (~1.5 * AU) is just an arbitrary number I chose to represent a value that should never be exceeded for a solar-observing mission. A few weeks back I was getting strange errors with HMI continuum images in Helioviewer.org, and tracked it down to an invalid dsun value (something like 1e19).
> 
> The goal of the _validate method is basically to perform a sanity check on the metadata for a file and make sure nothing is way-off.
> 
> This could be split across BaseMap and it's subclasses. For example, you could define some reasonable defaults in basemap._validate and override them in the subclass to provide a more tailored validation. This means you could also increase limits (e.g. for a hypothetical future > 1.5AU solar mission)
> 
> Regarding the idea of renaming dsun, etc. I think that is a good idea. We should sit down and think of what would be good name choices for the parameters we haven't already mapped.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/sunpy/sunpy/pull/134/files#r626400
This hasn't been an issue in the "Kane" class, so I presumed it wouldn't be a problem here but that could've been an erroneous assumption.
Ohhh :-( Thank you and sorry again!  I'll fix it shortly.
if -> elif if neither da nor db are changing above
Should be `mb_strtolower($data, Yii::app()->charset)`
I think most views in `kotti.view.edit` are guarded by `edit` or `add`.  I believe the actions menu is guarded by `view` so that people who cannot edit the context can still copy it.
Don't think we need 2011-2011 :)
Though you could use `sizeof(bogus_model) - 1` here.
What are the pros and cons of detecting excess attributes?
What needs to get fixed here?
So, in other words, if `/en/HTML` with `Accept-Language: de` redirects to `/en-US/docs/HTML`, it stays there.

What might be more useful is to link directly to `/HTML` and let the redirect bounce to the appropriate translation or the `en-US` default, which is what the document view does
A patch was just merged into olympus that removes these kinds of corners, please rebase your pull request onto the latest develop commit.
Then how would sphinx be used for indexing anything other than  the phpbb installation that generated the configuration file?
Specify an array key here instead of using `[]`, then use isset() above.
I don't see a comment...
Docblock needs to have parameters and return value. Function name needs phpbb prefix if it is in global scope.
indentation messed up? ^^
Sorry again, it's my first day.
heh. yep. just need to use it.
It's checked when you send a new message. You can send an empty message.
This is not about multi-line comments , this is about marking #endif macro. What I want to say here is that one should follow the code style this project possesses. I have every #endif macro with /**/ marker, hence the renaming.
Copy & paste -> it's pretty bad :)
I did a check for `typeof precision === 'number'` since any other type would through a SyntaxError.
@zhouyaoji thought these dimensions belong in mojito, but I wanted the change set to be minimal wrt non-hybridapp users and the mojito dimension issue seems to be in flux.
parameter assignment
One more note on this one: you freeze once, you _have_ to copy everytime, so this is not apple to apple.
How do we test whether this walk process works correctly? I think it would be good for us to start migrating anonymous functions like this "outward" where we pass them via name. So perhaps the function(settings, config) {...} logic becomes:

this._YCB_CONFIG_WALKER: function(settings, config) {...};

Then we can write nice unit tests targeting that helper function by name.
the formatting is weird here (I had to read it 4 times to conclude there was no syntax error in it). It would be more readable to use the following:

```php
return $this->dumpIndent($indent) . implode(
    PHP_EOL . $this->dumpIndent($indent),
    explode(PHP_EOL, $text)
);
```
You need to add this to d2k and cnc, too. It will crash otherwise.
Defaulting to None means the attribute will be excluded from the tag, which seems reasonable to me.
Controller has been defined twice, possibly formatting while beautifying it. Please push the code that works!
@johnstoecker @nathanleiby  
Even after correction, this commit should not go in main Repo, Please consider closing this pull request and opening the pull request after next commit from Mukesh!
@mukesh966 
You may amend your changes in this commit, so that these changes should not be reflected in any commit!
Again, why is this here but commented out?
Why does a QueryExpression, rather than a QuerySpecification, imply that it's a tuple set operator? I looked at the code (briefly) and couldn't figure it out.
got it. what about `window.require`? I don't like `this` because it can change its context.
Missing input validation (doc1 or doc2 might be NULL).
I limited the checking to the only case I've seen, but I'm curious about other opinions. Good idea, or should we check always?
* 時刻の取得が1度きりになったのでOKです。ですが、tableView numberOfRowsInSectionメソッドは、「セクションの行数を返す」という目的のメソッドです。"メソッドには目的以外の処理をさせない"のが、読みやすいコードを書くコツです。もっと適切な場所を探しましょう。viewDidLoadで良いと思います。
Okay, lets back this out for now then. We can clean that up later.
No, thanks ;-)
Do we really need continuous value?  If you want some id, I guess serial can do.  Wouldn't it acceptable to have invalid id?
These for-loops have also been replaced with matrix operations.
Looks like the sha1 changed? I'm getting `e517e9fb66dc285321b38a398aae35956c83a0ea`
``sandbox.require = mod.require.bind(mod);``
We still need to figure out how to avoid taking a strong opinion on what the user needs.
nuke this crap!
On Sat May 05, 2012 at 01:15:57PM -0700, David E. Wheeler wrote:
> > @@ -129,43 +163,45 @@ sub go {
> >      my $sqitch = $class->new($opts);
> >  
> >      # 5. Instantiate the command object.
> > -    my $command = App::Sqitch::Command->load({
> > -        sqitch  => $sqitch,
> > -        command => $cmd,
> > -        config  => $config,
> > -        args    => $cmd_args,
> > -    });
> > +    my $command = App::Sqitch::Command->load(
> 
> Is there a way to tune perltidy to allow the original? That is, if
> the only argument to a function or method is a hash ref (or array
> ref, for that matter), have it open on the same line as the parens.

Yes and no. The braces can be left in their original position with:

    --stack-opening-tokens 
    --stack-closing-tokens 

But this keeps the content indented two levels:

    # 5. Instantiate the command object.
    my $command = App::Sqitch::Command->load( {
            sqitch  => $sqitch,
            command => $cmd,
            config  => $config,
            args    => $cmd_args,
    } );

So it is one way or the other as far as I can see, but I'm not an
expert (although I have read the manual from top). What I found after a
while was that the particular formatting became a small issue in
comparison with not having to worry about it.

Mark.

p.s. By the way, your mail client, and I noticed whatever you write
commit messages with doesn't wrap long lines. At least on a Debian
system "git log" is kinda hard to read, and I can't seem to find a
less(1) option to stop chopping lines.

-- 
Mark Lawrence
Could we give the classloader a different name? The lower L looks a lot like the number 1.

Also, please wrap the line after the open paren... it's a bit long
Let's make these private
Long line, s/consistant/consistent
Yeah this is all written out basically because it was copied from a spreadsheet to play with the variables in other ways.  From playing with tweaking though only the BACKOFF_FACTOR seems to be where we'd want to tweak, so I could reduce the equation now.
Why do we need fallbacks for old names? Shouldn't `window.cordova` be enough?
Also, please return an object `{x: ..., y: ...}` instead of an array.
We should mention the use case where an extension may want to access both the core Brackets strings module and their own strings module. Either separately or together.
encodeURI won't encode '?' and ';' that can be used in filenames. We may have to consider encodeURIComponent or a combination of escape and encodeURI. I can't remember what we did for our default brackets folder path.
can we detect a right-click?
Should this info be added to the XPNotSupported message below?
Mm, Maybe I'm misusing the term, but is not a Tower of Field what is
built in the DifferentialExtension?

On Tue, Mar 27, 2012 at 1:11 AM, Aaron Meurer
<reply@reply.github.com>
wrote:
>> @@ -544,6 +576,9 @@ def _important_attrs(self):
>>              self.backsubs, self.E_K, self.E_args, self.L_K, self.L_args)
>>
>>      # TODO: Implement __repr__
>> +    def __repr__(self):
>> +        return "Tower(f={%s}, D={%s}, fa={%s}, fd={%s})" % (
>
> Why did you use `Tower`?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/asmeurer/sympy/pull/42/files#r607169



-- 
Jorge Eduardo Cardona
jorgeecardona@gmail.com
jorgeecardona.blogspot.com
github.com/jorgeecardona
------------------------------------------------
Linux registered user  #391186
Registered machine    #291871
------------------------------------------------
One last nitpick: could you rename `merge_base_oid` to `_merge_base` or the likes, just so to let people know you're supposed to use the external wrapper?
I think we are using 4 spaces everywhere.
I do not understand this comment?
because i had to do that to get it to pass at one point. like i said, i expect to fine tune the waits later.
Let's use something like http://tablesorter.com/docs/ to give us a usable table (e.g., sorting) without wasting cycles on UI here.  Something we can use with external JS vs. pulling it into our tree.
Should this be `Layer 0` like it is in the first trackEvent?
you can actually remove the `li` from this, i.e just  `.butter-editor-header-li`
for..in loops should wrap their body in a hasOwnProperty check.
It's slightly different in this case with combining them.  Only one header block can be ! important, it seems.  This is a hack to get them both to show, since it's popcorn + butter.  It's good enough for now, I think.
*shrug*

It's that way in the rest of the file.
There's also a double space here.
Do you have to use 1970... here? If you just need an arbitrary Date object, why not use Date()?
Uh? I don't get this... they should be numbers anyway...
There may be problem with such input string:

    < x ! x - x - x xxxx x - x - x >  

It's not comment.
Was going to ask why you're using a string key here, but I see it's a BinData object.  It looks like you could also do packet.sequence_number.
Špatný styl. Používá se "Probíhá něco něčeho" a v tomhle konkrétním případě (před slovesným podstatným jménem) se "probíhá" povinně vynechává.
Also added 1.1 to specs now. (ref. 73260d2)
can remove the extra leading '#'
@iNecas I added default_scope to system group model. Not so big change, behaviour is the same.

https://github.com/Katello/katello/pull/176/files#L37R79
Minor thing - could you move the bind to be on document to avoid the lookup?  i.e. `$(document).bind()`
Sure.  Added some details with commit be720c99b16906382dded2ddb8d71afd17645a49 .
I 'd argue this group lock check should be on a validate or a before_save.  How is the interaction expected here,
```ruby
system.add_pulp_consumer_group record 
system.save!
```
<-  raise exception if locked, else expect a save! to be called.

To me a more reasonable approach is save! raising the error and this be a typical record invalid call.
Addressed in dec4482.
Not sure we should call it an ActionDescriptor, too close to MVC. Not sure what a better name would be though.
I thought that was only for long
I wonder if it's not better to use a :before css rule. What do you think of that?
What about:

      var view = this.view;
      var height = view.scrollHeight - view.offsetHeight;
      if (view.scrollTop === height) {
        clearInterval(this.viewScrollingTimer);
        return;
      }
      view.scrollTop += Math.ceil((height - view.scrollTop) / 2;
It also doesn't even seem to work based on my own experimentation, but maybe I'm doing it wrong?
Ok, I did this, and it turns out that the choice of the year of the algorithm is unimportant for the accuracy of the corrections - in all cases the results agree within 1e-5 arcseconds of the reference.
Changes made based upon feedback.
A problem with allocating it statically is that it is declared in both init.cpp and test_bitcoin.cpp - and these globals must be accessible from rpcwallet.cpp as well, which includes init.h for this purpose. I would prefer the wallet RPC calls to be encapsulated in an RPC class which takes a wallet map in its constructor - and then we could remove any dependencies on these externs entirely and restrict the variable scope to init.cpp or the unit test executables. But for now, the easiest way to get this to work (least amount of modifications to source) is the way I did it, I think.
No need for casts --I've added the `PRIszx` / `PRIszu` / `PRIszd` macros in util.h for for these.
it does not. we might also have to check on ModelAlias.field then.
Qt 的在前面
Fixed Inst's previous behavior is now part of InstModel. The ability to build facilities prior to/at the start of the simulation is now part of simulation construction (it's basically connecting nodes to leaves on the simulation graph).
Just a little note: `motion.author = user` doesn't need to be explicitly called like this because we are calling `user.authored_motions.new` which will automatically set motion.author to user.

Don't worry about fixing this though. I'm about to remove the line myself and merge the branch.
You may want to store a reference the model/view/handler here so that you can update the ui if one changes later on.
I should note this is an assumption.

In Symfony, the doctrine DB service and swiftmailer are defined in app/config/config.yml. 
The parameters they should load with are defined in app/config/parameters.yml.
My guess is the container calls everything into life when called, and looks into config.yml for services to do so.
Are suggests still loaded in here? If so, they probably shouldn't be.
The reason that I went with this is that JavaScript doesn't support
optional keyword arguments.  So for functions like execute that take
multiple optional callbacks, there really isn't a way to do something
like:

def execute(code, execute_reply=None, cell=None, output=None, ...)

Some of the other methods doesn't need multiple callbacks, but I
wanted to make a uniform interface, so I went with a callback struct.

On Wed, May 9, 2012 at 9:02 AM, Bussonnier Matthias
<reply@reply.github.com>
wrote:
>>          return msg.header.msg_id;
>>      };
>>
>>
>> -    Kernel.prototype.complete = function (line, cursor_pos) {
>> +    Kernel.prototype.complete = function (line, cursor_pos, callbacks) {
>> +        // When calling this method pass a callbacks structure of the form:
>> +        //
>> +        // callbacks = {
>> +        //  'complete_reply': complete_reply_callback
>> +        // }
>> +        //
>
> I was wondering, as there is only one argument for the `callbacks`, could we not directly build it ourself and use `complete_reply_callback` as an argument of the function ? We can even support both by looking at wether the third argument given to the function has a `complete_reply` key or not...
>
> Same  for `object_info_request` I think
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/1711/files#r795774



-- 
Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com
Even though this is JavaScript and not Python, I try to follow PEP 8 coding conventions, so 1 space after the comma in function argument please.  Here and elsewhere.
If you call enumerate with `start=1`, we shouldn't need `i+1`.
javadoc comment explaining why 59 minutes is a correct value
actually, I'm not sure this is the best way.  the attachment api is also a part of cinder itself, and the same api class:

https://github.com/openstack/cinder/blob/master/cinder/api/openstack/volume/volumes.py

I think the code related to attachments should be in the same class, as this is the same degree of modularity as the corresponding openstack project