labofsahil/github-event-dataset-2012 huggingface dataset:
Hey Alec, the OJS reviewer interest field uses the interestsInput.tpl template, which I didn't touch for this bug.   However, reviewer interests does appear to be broken right now in OJS master.
this.namespace.join('\\');
What do you mean? People can simply don't pass an 'agent' option.
I haven't looked into slashem specifically, but if the package uses pkg-config to determine the linking options, you might get away with supplying a dummy pkg-config "tool" as configure argument instead of creating dummy libraries just to make the linker happy:

```
# use 'true' as dummy pkg-config
NOUX_CONFIGURE_ARGS += PKG_CONFIG=true
```

This way, configure will be told (by true) not to add any arguments to the linker command line. Dunno if this works for slashem but maybe worth trying.
@ry after trying to reach you at irc for two days, finally commenting it here :D

Can you please explain what happens here? And why do we need such limitation here?

More discussion here: https://github.com/joyent/node/pull/2508
大哥你这是神马破名字阿.....必须得改....
Rather use message class hierarchy (Message, QueryMessage etc.) instead of generic message containers like ArrayBuffer or Tuple.
How did you get the > mine ended up looking like this when I updated:

![](http://content.screencast.com/users/brainyone/folders/Jing/media/759120d4-cbed-4d8f-8775-e8d8886af40b/00001645.png)
hi tim, why do you want to enable it for all user which clone the repo?
config file location detection is done in a totally different place.
1. each isolated installation would have its own .springrc in its folder
2. --isolated-dir seems broken cause the config file location code (ConfigLocator.cpp) doesn't seem to respect that option!
Par rapport à ce que tu disais par mail. Paul a eut une erreur lorsqu'il a voulu mettre à jour une proposition sans tag id.
Ah sorry. I'll remove this change.
Is it useful for any particular reason to be able to customize the name of the document column?
Looks pretty good. Glad you are pulling "handle_message" apart into get_message and send_message as well as pulling is_old_message and is_from_self into separate function.

Also it was awesome to do a git pull and watch the code get automatically reloaded in my console by sync... jane doesn't miss a beat.
You might not need so many catches since we catch exceptions at the root. Just let them bubble up using .Then
I am not able to run all test successfully for your v0.3.4 branch. Do you have and idea why it's failing ?
<pre>
Bacon::Error: unexpected invocation: Boson::Index.find_library('sweet')
unsatisfied expectations:
- expected exactly once, invoked twice: Boson::Index.find_library(any_parameters)
- expected exactly once, not yet invoked: Boson::BinRunner.abort()
satisfied expectations:
- expected at least once, invoked once: Boson::Manager.load()
- expected exactly once, invoked once: #<Boson::RepoIndex:0x101720e00>.update(any_parameters)
- allowed any number of times, not yet invoked: Hirb.enable(any_parameters)
	/Users/me/.rvm/gems/ruby-1.8.7-p352/gems/mocha-on-bacon-0.2.0/lib/mocha-on-bacon.rb:60:in `it': autoload_command - with non-core command not finding library, does update index
	/Users/me/.rvm/gems/ruby-1.8.7-p352/gems/mocha-on-bacon-0.2.0/lib/mocha-on-bacon.rb:54:in `it'
	./test/bin_runner_test.rb:170
	/Users/me/.rvm/gems/ruby-1.8.7-p352/gems/bacon-bits-0.1.0/lib/bacon/bits.rb:20:in `describe'
	./test/bin_runner_test.rb:136
	./test/bin_runner_test.rb:5

300 tests, 576 assertions, 1 failures, 0 errors
rake aborted!
Command failed with status (1): [bacon -q -Ilib -I. test/*_test.rb...]
</pre>
the spacing is inconsistent here. You should stick to 2 space indention to avoid running into http://projects.puppetlabs.com/issues/9555 again
As discussed in the forums, seems to be better to place this within the messages script;  The Calendar screen is not used by everybody (note there is a globals option to hide the calendar and use the patient search screen in place of the calendar), so wouldn't assume that the calendar is the "central" screen. From a workflow perspective, as a physician, in EMRs the main screen generally tends to be the "Message screen". And I am guessing this is also the case for RN's, receptionist, billers, etc.
  My thoughts are to change the title of the Message screen to "Message Center" and place a Reminders section at the top (your widget) and the Messages section under this (your widget works nicely for this since it is expandable/collapsable). I am willing to bet that this "Message Center" script will become the focal point of OpenEMR when not looking at patients (in that case, the patient summary screen is the focal point); it will also make sense to migrate Authorizations there and then at some point create a UI that actually combines them all (still keeping the backend separate data structures though).
  This is just my opinion; very curious what others think
removed, will send with next commit
Thanks, I will delete and re-do, all my changes are saved in my local machine elsewhere, still figuring GIT out, I think I will use the gut bash this time round. Wish me luck (I need it) 

Was the suggested fix better ? 
Sent via my BlackBerry from Vodacom - let your email find you!
I think we still use "programming language", not "coding language" lol.
Собственно я и выражал сомнения в корректности этого исправления в своем месадже выше.
Creds at least .. -__-
But yeah, this is nice. Totally pretty. I've thought before of wanting to do `Uint8[1,2,3]` etc. but I guess the pun on `ref` kept me from it. It's too nice not to have though.
Your commit messages don't really make it easy to figure out what you are doing with all these changes. Would spending 30 seconds to describe your changes hurt your performance?

Replacing `std::vector` with `std::map` is not a good idea when we have possibly tens of thousands of files. You switch to less efficient container types for what gain?
сломали миксологию, теперь время не увеличивает настоя ледяного
shouldn't be necessary.  Doesn't declare do this?
Sorry for the delay. The only part of this I'm immediately timid about is the tagging. I'll look at it more tomorrow.
see #3234
inherit from object, please. Among other things it makes pickling more efficient.
Nice, I love this. So much easier.
Regarding formatting, what do you say on doing long_desc with %{} ?

```ruby
long_desc %{
  Run code given in parameter on one of application servers. 
  If a file name is given, run contents of that file.
}
```
```
This was used in several SAI. you`ll have to fix the SAI too.
In SBCL 1.0.54, (format nil "~r-~a" 1 '#:byte) -> "one-BYTE", even with *print-case* set to :upcase.  This patch uses READ-FROM-STRING to, hopefully, Do the Right Thing.
It make sure element ID is unique
This should be:

    bin.install Dir["bin/*"]
Should we support textarea automatically? Maybe if TEXT is the data type in MySQL, we return textarea as the default control.
Need a try, but remeber than min([]) raise an error ;)
good job. I like how this works, thanks for working on this.
> Hightlight
redundant parenthesis
Might as well make this a docstring?
Ah I see. I don't mind if we remove the rotate everywhere. It's out of fashion :)
np. rock on!
Ok, there seems to be two things you're referring to here.

First, push/pull is indeed not working. I tested it now by setting up two chests, one with a push filter for cobblestone, the other with a pull filter for cobblestone. Loaded up the first chest with lots of things, including cobblestone. Then I set a looping powered rail between the two, and a cart-with-chest on it, and started it, it now passes by both chests.

Nothing happened, as you said, so I will investigate this.

As for the suck filter being slow, this I can't reproduce.

Basically the algorithm is as follows:

1. For every floating item in the world
2. For every chest with a SUCK filter
3. Check if the item can be put, partially (like filling up a stack in the chest) or completely into that chest
4. If we get here, and there is still more left in the item stack (ie. nothing or not all has been dropped into chests), go through all chests without a SUCK filter
5. Check the item against those chests

Now, if an item survives all of this, ie. is still in the world, it means that:

1. Either there is no chest in range of the item
2. All chests in range are either filtered to not suck that item, or are full

So those items I put into a short list, with a counter of 1. The next time I check all the items in the world, if the item is on that short list, I increase the counter. This is done to increase performance. Chances are that if an item is out of range or all chests nearby are full on one tick, it's the same state on the next, so avoid all the heavy processing of the item for a short while.

Once the item has been in that short list for a while, the counter has reached a certain threshold, and the item is removed from the list, and on the next tick again processed fully.

This means that you should see the following behavior:

1. Add a SUCK chest, leave it empty
2. Add another chest (not a dropchest), fill it with items
3. Break the chest full of items
4. All the items inside will pop out into the world, and almost immediately disappear into the SUCK chest

However, if the SUCK chest is full, you'll start seeing this behavior:

1. One or more of the items disappear from the world, but not all
2. Over the course of a few seconds, the same thing as in 1. should occur
3. Eventually (we're talking seconds here), all the items that fit into a nearby chest should be removed from the world

If you see otherwise, can you describe the situation?

Vennlig hilsen

Lasse Vgsther Karlsen
lasse@vkarlsen.no

On 27. jan. 2012, at 17:09, sorklin wrote:

> It simply doesn't work.  Also we noticed that the suck was delayed, it would take a good amount of time to actually suck the items.  I didn't test it rigorously, just an observation.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/lassevk/DropChest/commit/200bbe49ae3ac4476ffd12d4108791e8d382ed08#commitcomment-908593
That would be awesome! What I'm doing there right now is getting this right to a point where Text is actually usable in Firefox. This one fixed one test already. Next steps:
- Fix right-to-left selection in FF (super annoying)
- newlines are still a problem
- clean up getSelectionRange and setSelectionRange
- extract code, put it into a jQuery plugin :)
Or something like that.
Wil said he'd like it to add descriptions of what groups are used for.
I know you didn't touch this, but it seems weird.

Does imagefile get used? Why read up_file into StringIO.StringIO and then pass that (a file-like object) to Image.open?
Sure, I grant you permission for both updates.
This line actually needs to be removed completely
@Vasago LordJZ just did it.
I'm not sure. This is old code, it comes from Mark's validator.
This is missing a return.
yes, they are there. just currently doesn't have any files in them since I haven't launched the game after I deleted and restored the savegame
on code:
            Unit* pUnit = m_creature->GetMap()->GetUnit((*i)->getUnitGuid());

            if (m_creature->GetCombatDistance(pUnit) < dist)

if pUnit is NULL, we are got a crash!
Yes end will always fire, and I plan on documenting tomorrow
that'll need an option either for the number of "base domain parts" or (better) specifying the base domain.   ("www.example.co.uk", "foo.local", etc).
What is this file for ?
@sebers this should make the application work again. I couldn't push it to the eit/gh-pages branch for testing...could you do that?
Just for the record, I don't really like this.

The reason I don't like it too much is because it tells writers of code in wiki pages that it's ok to put code in there instead of coding high-level APIs in Java and making them available through a script service.

I really hope we're never going to use this API ourselves since IMO it would mean we're not using best practices and we're putting too much unmaintainable code in wiki pages...
`-O0` is mea culpa, since otherwise compiling the test suite takes a long time =).
It needs to be mentioned that you have to pay for this.
You're right, it seems I lost some code in an old refactoring :p

However, from my point of view the patch you're submitting isn't OK for 2 performance issues:
- when that.compressionMethod === 0, reader.readUint8Array should be used instead of calling reader.readBlob and then transforming Blob to Uint8Array
- I'd prefer to implement a bufferedCopy function (similarly to [the one used when deflating](https://github.com/gildas-lormeau/zip.js/blob/master/WebContent/zip.js#L725)) for scalability
Waarom dit gedaan?
We can remove that function..its not being used right now.
Don't do this. Lines should not be more than 80 characters wide: http://git.gnome.org/browse/gtk+/plain/docs/CODING-STYLE
Check this out: http://pastebin.com/f6PHFbtR

Seems like ID gets lost somewhere during copy process in "for (@orig_keys) {". I'm investigating further.
Fixed for the last two that are longer, all other enchantments already have short names.
Tengo mas trabajos como este(plagio), ademas el campo `created_at` es incorrecto, así como la identación
It would be ideal if we could use consistent formatting for our code, adhering to the Google Java style guidelines here:

https://sites.google.com/a/android.com/opensource/submit-patches/code-style-guide

We find that using consistent style for the code is critical for long-term maintenance.

It should be easy to highlight all of the code in Eclipse and hit "reformat" to get the indentation to be the same, at least.
Just ignore my neediness dude. I do need socks though. Really really bad.

On Fri, Feb 17, 2012 at 2:40 PM, Charles Thompson <mail@kinesis.me> wrote:

> I have a blood sugar problem that is preventing me from programming and
> working to my potential. Whenever I eat a lot of refined carbs (which is
> terrible) I feel sharp and can focus.  I need a steady diet and a gym pass
> for awhile.  This is truly the medical problem, as it can also cause severe
> mood swings.
>
> I'm going to see what I can do about it on my own though.
>
>
> On Fri, Feb 17, 2012 at 11:29 AM, Jack <
> reply@reply.github.com
> > wrote:
>
>> recursive fibonacci with memoization -- nice, dude, nice.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/kinesis/project_euler/commit/73f030f40d987d0a09505d8147fbacd17561a1f8#commitcomment-979511
>>
>
>
>
> --
>
> Sincerely,
> Charles Thompson
> *UNIX & Linux Administrator*
>
> Tel* :  *(650) 906-9156
> Web :  www.kinesis.me
> Mail:  mail@kinesis.me
>
>


-- 

Sincerely,
Charles Thompson
*UNIX & Linux Administrator*

Tel* :  *(650) 906-9156
Web :  www.kinesis.me
Mail:  mail@kinesis.me
Add #import &lt;Foundation/Foundation.h&gt; to hyphenator.h or .pch file and comment 'typedef char unichar;' line.
For use with Obj-C in your app, you may write:

//word - NSString or CFString object

CFIndex wordLen = CFStringGetLength(word);
CFMutableStringRef lowerCaseWord = CFStringCreateMutableCopy(kCFAllocatorDefault, 0, word);
CFStringLowercase(lowerCaseWord, CFLocaleGetSystem());
    
UniChar* buff = malloc((wordLen+1) * sizeof(UniChar));
CFStringGetCharacters(lowerCaseWord, CFRangeMake(0, wordLen), buff);
buff[wordLen] = 0;
CFRelease(lowerCaseWord);
    
struct word_hyphenation_t* wh = hyphenate_word(buff, patterns, pattern_marker);

You get warning because type unichar is already defined for NSString char type and unichar == unsigned short, not simple ASCII char
Later I will commit xcode project for iOS
Thanks dude. You saved SFML :p
The main instance of Application should be based on the first uri segment, wouldn't it be more handy ?
Drawable marker = null;
					if (imageProperty instanceof TiBlob) {
						marker = makeMarker((TiBlob) imageProperty);
					} else {
						String imagePath = TiConvert.toString(imageProperty);
						marker = makeMarker(imagePath);
					}
					if (marker == null) {
						marker = makeMarker(TiConvert.toString(imagePinProperty));
					}
The end result is very similar indeed, except that it "works" (so far, so good…)
I am having a hard time locating a reference document. I've been doing this for a while now and has become a habit with me to use : . On most systems, /bin/sh is supposed to be a POSIX compliant shell. Even if it is linked to /bin/bash, bash tries to behave in a POSIX compliant way when invoked with a sh name. A sample script (say ./shtest) containing

``sh
:
sleep 60
``

get launched as `sh ./shtest` . A `ps auxwww` in a different terminal confirms that behavior no matter which shell (bash or zsh etc.) invoked the script. But I am hard pressed to find a definitive document now.
Note the _oops_, on the extra comma instead of semi-colon. Fixed in 78aacf4f1b24939da7955c4065927143e8ae3254
Heya
compile error on Linux ( debian 64 bit )

Linking CXX static library libauthserver_pch_dephelp.a
[ 10%] Built target authserver_pch_dephelp
[ 10%] Generating sharedPCH.h.gch/shared_Release.gch
Linking CXX static library libworldserver_pch_dephelp.a
[ 10%] Built target worldserver_pch_dephelp
[ 10%] Generating collisionPCH.h.gch/collision_Release.gch
[ 10%] Built target pch_Generate_shared
[ 10%] Generating authPCH.h.gch/authserver_Release.gch
In file included from /home/trinitycore/trinitycore/src/server/game/Grids/Notifi                                                                                                                                                             ers/GridNotifiers.h:33,
                 from /home/trinitycore/trinitycore/src/server/scripts/Precompil                                                                                                                                                             edHeaders/ScriptPCH.h:12,
                 from /home/trinitycore/try25k/src/server/scripts/scripts_pch_de                                                                                                                                                             phelp.cxx:1:
/home/trinitycore/trinitycore/src/server/game/Spells/Spell.h:678: error: extra q                                                                                                                                                             ualification âTrinity::WorldObjectSpellTargetCheck::â on member âWorldObjectSpel                                                                                                                                                             lTargetCheckâ
Linking CXX static library libgame_pch_dephelp.a
[ 10%] Built target game_pch_dephelp
[ 10%] Generating worldPCH.h.gch/worldserver_Release.gch
[ 10%] Built target pch_Generate_collision
make[2]: *** [src/server/scripts/CMakeFiles/scripts_pch_dephelp.dir/scripts_pch_                                                                                                                                                             dephelp.cxx.o] error 1
make[1]: *** [src/server/scripts/CMakeFiles/scripts_pch_dephelp.dir/all] error                                                                                                                                                           2
[ 12%] Built target shared
[ 12%] Built target pch_Generate_authserver
[ 12%] Built target pch_Generate_worldserver
make: *** [all] error 2
Bug:
Now water in Wintergrasp deals damage
X: 4692.749512 Y: 2696.259277 Z: 349.000824
Liquid level: 351.790863, ground: 346.510681, type: 81, flags 1, status: 8.
You mean by having a shuffle that doesn't ungroup at the end?

If you had a ```group(shuffle(...))```, that wouldn't be optimized away, since the expansion is done late in the exporting process.
delete the comment, you can always find this code later :)
I tested and it seems to work without naming the instances. Tried running a
external raven from the pubsub sample and it worked. Can you do some
testing and see if there is still an issue?

On Sun, Feb 5, 2012 at 10:20 PM, Johannes Gustafsson <
reply@reply.github.com
> wrote:

> Components registered using IContainer.Configure() should also be named,
> right?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/NServiceBus/NServiceBus/commit/41444d10586be258d1ed40b9941eeb02206c7ca5#commitcomment-934750
>
>  i think she will be a great contribution for trinitycore.

She IS being a great contribution to TC
failed to compile

MOD_M2 = 1, 

is defined in src\server\collision\Models\ModelInstance.h
and not in current .h
I have a fix for collecting attributes.  The keys in original_attributes are actually objects rather than symbols - have to get the name.  Fix is here:

https://github.com/devrandom1/dm-is-versioned/commit/8c001a383a80ed717c0c2e2e663178b716b13497
I reviewed the individual commits so this should be fine.
@metaskills can you provide a test that passed before 2.3.2 that currently fails on master regarding this?

Remember, you *only* have access to the resulting model instance in the `after_*` callbacks. It's always been this way. You've **never _ever_** been able to access attributes that existed on the model instance from within FG attributes themselves, so it doesn't actually matter that these methods are on an anonymous class.

Again, an example (instead of a gut feeling) would be great help here.
this issue has been fixed on r.js 1.0.7: https://github.com/jrburke/r.js/issues/104
Pardon what? Is this a bug report? I'm a bit confused.

The python serialization format isn't a real serialization format, as it doesn't serialise to a string. Reversion requires the serialisation format to serialise to a UTF-8 string or unicode, so it can be stored in the database.
A variant, actually. `userDataErrorString()`, `frontSideTextErrorString()` and so on. And some high-level `checkUserData` to check for string emptyness and, if not, to show a message to user.
Oh! I think I can't do this because we will loose the input, I think!
I have a look.
Also I had to modify your version of TinyWebServer.cpp file to initialize the server_ and client_ like this:

TinyWebServer::TinyWebServer(PathHandler handlers[],
    const char** headers)
 : handlers_(handlers),
   server_(EthernetServer(80)),
   path_(NULL),
   request_type_(UNKNOWN_REQUEST),
   client_(EthernetClient(255)) {
I agree.  I hadn't noticed the version suffix.  When we've decided this release is stable should we remove the suffix?

As for the merge, I used `git merge` but pushed an incorrect version up which was up for a few minutes until I reset and did a `git push --force` to remove it.  If you pulled the merge right after I initially pushed it that may have caused a problem.
I say you make the patch and merge into `strict`.
You added a whitelist? When?
Can you explain why this was necessary?
'{' should be on next line
>= is not compatible with n.* version constraints because it doesn't mean anything. Please use `2.*` alone.
Well, let's see :trollface:
This function doesn't work. If you renamed `nsIFile` into `file`, it might be better.
NOOIT
reply to reply to reply
Please add a simpler version of `safeQuery` that defaults all of the `groupBy` nonsense to null.
sorry, i just pushed fix
tristate "MSM Resource Power Manager Stat Driver"
OK, thanks for imformation.
And how did you otimuizing power-consumption? Can you tell me?
On Jan 10, 2012, at 10:00 AM, bdarcus wrote:

> On Tue, Jan 10, 2012 at 12:57 PM, Charles Parnot
> <reply@reply.github.com>
> wrote:
>> Damn it! I thought that would be caught by the validator :-)
> 
> It definitely should catch that; it's basic XML. If it doesn't, maybe
> you ought to use a different validator (RNV is superfast, compliant,
> open source, etc.).

Weird, then. I am just using the online validator linked in the wiki. TextMate actually had that ampersand in red so it is definitely a basic error that should have been caught. I'll look into alternatives.

> I thought some of us started to work on such a script. Rintze, do you remember?

I would be interested, thanks!

charles


--
Charles Parnot
charles.parnot@gmail.com
twitter: @cparnot
http://mekentosj.com
This line is no longer required.  You may remove it.
great job! :-)
ja het zou goed kunnen dat ie em weer tript op ipad omdat ie lineTo doet naar een punt waar ie al is ofzo?
On 29 February 2012 09:33, Otavio Salvador
<reply@reply.github.com>
wrote:
> But why allow the user to use C+l? in a Kiosk I expect the useful
> links to be on the webpage, no more.

When using a kiosk with the "webconverger" default chrome with the URL
bar, having Ctrl+L shortcut is very handy. Why disable it if there is
a URL bar?

I don't quite understand.
Moved code over from CM7.  Will add back into CMParts if it is added to CM9
@adgaudio - most likely I put it in there originally and then neglected to just remove the line when I was manually testing on batch_get_item.  Should probably be removed from there as well at some point.
Agreed. I would prefer to do that piece in another patch. I'm happy to add a TODO, but the actual splitting out into a helper is a little bit more complicated, as we cannot simply call delete_object, as it crafts the response and the response is structured differently in the case of delete_multiple_objects and delete_object, since delete_multiple_objects attempts to delete each object and returns a list of successes/failures.
Keeping that in one line is imho more readable: `cdef char * type = dat._data.dtype.name + ":soa" if dat.soa else ""`
It is and this definitely needs fixing. There currently isn't a way of telling `inline_with_numpy` to use mpicc other than through exporting environment variables. The correct thing to do is fixing it in instant and send them a merge request, which is what I'm going to do. This commit is for the sake of getting it to run on buildbot without requiring a restart of the build master and needs to be reverted asap.
one more thing: I think it would make messages more easy to understand if these actors had names corresponding to the address they are serving
:-) 28 is a nice number
I changed from 30 to 25 yesterday. Saw a failure today.
30s is the barrier timeout, and I would like to be less than 30, but still long.
I think there is something else that is broken. We have several tests that sometimes fail when we shutdown.
this looks unnatural/unorthodox.

I'd prefer to reuse the nested protocol notation: akka:tcp://.../...
this `@volatile` does not serve any purpose I can see (no threads involved)
please add a comment what exactly is wrong with this address
You can do Failure(new RuntimeException) instead of Try(throw …)
Yes: 'Up'. just to make it clear that it is a command.
Yes, I'll untangle it and clean it up.
Why explicit type here?
I made the change to remove the unicode strings but I still get this error when I try to upload a binary file.  Are you seeing this?
has shape has shape?
Incorrect docstring, a list is returned.
in general, try to avoid whitespace changes -- make all functional changes first, commit, then do white space :-) easier on reviewers
it won't break, just means more metadata flying over the wires. I'm also trying to find the original bug in platinum's filtering.
VS plays nice here but should I declare as volatile to be safe?
Actually, checking the code I realized this was the only instance AddToGroup was called with bSortAndRenumber set to false, so I thought there was some reason for this.
That's why I kept it that way
Mark keys, yes (they're like boxes). Not sure about prompt tags, will think about it (it's a safe choice for now).
if that happens, will the data be stuck in the feeder and block all tasks?
Thank you Bert, for some reason when combined it doesn't display the bg image in Firefox and just reverts to the bg color.
Another idea is for ModelTest to be a friend of Model. It could then provide a method to access a private doSetParent().

Note that we already use ModelTest in every module test.
I think variable should be variables here.
You are correct, that was supposed to check before frame assignment too. Must've undone when I was playing with it. It works correctly in scenarios where there are dynamic numbers of cells; I'm testing it with a different app.  



On Wednesday, October 17, 2012 at 9:10 PM, Jonathan Willing wrote:

> In lib/UIKit/TUITableView.m:
> > @@ -765,8 +765,10 @@ - (void)_layoutSectionHeaders:(BOOL)visibleHeadersNeedRelayout > } > } > > - section.headerView.frame = headerFrame; > - [section.headerView setNeedsLayout]; > + section.headerView.frame = headerFrame;  
> I'm not entirely sure what exactly the problem is. Definitely I see an issue with moving cells between different sections in the current demo, but I also do not see much of a difference with these changes in place. What exactly is the issue here?
> In the demo, there are always 25 cells in each section, but in a real application there will probably be a variable number. Does your patch resolve the problem in a real use case?
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/github/twui/pull/114/files#r1874975).  
>  
>  
>
Probably better for readability to call this snapsToTickMarks
My point is, that somebody did this mistake, because he didn't know, there can't be the trailing slash. The autocompletion then stopped working.
Method name is ok, not returning $this is ok. :) What do you mean with 'coning style' comment? I don't see any  vitiation of Coding Standards (http://doc.nette.org/en/coding-standard)
Really? Thanks, I'll have to remember that.
Is this really what we want do ? I think the MessageToMessageEncoder already has such kind of method. Or I'm wrong ?
This is checked in `ctors.c` line `874`ff and not necessary to check here again, since `NewFromDescr` is called in all array creations.
I think this should be indented.
а когда и зачем они отцепляются?
пробел после &
тут, наверное, тоже имеет смысл передавать координаты линка в createElement()
Agreed with @frodsan, should stick with `spawn` since it's different in collection proxy.
This is related to the other change, right?
Good catch ... thanks.
We rely on `typedRefinement` having been called. Alternatively, I could pull the `OnlyDeclarationsError` out of there and into `typedCompoundTypeTree`. Not sure which way to go here; wdyt?
We might even drop the argument altogether. I don’t think access to the actual function is actually needed:

```scala
def prepare(): ExecutionContext = this
```
so that it matches the format of convolve, convolve2d, correlate2d.  Now it's a definition list instead of an unordered list, and should look better on the web docs after this change: http://docs.scipy.org/doc/scipy-dev/reference/generated/scipy.signal.correlate.html#scipy.signal.correlate http://docs.scipy.org/doc/scipy-dev/reference/generated/scipy.signal.convolve.html#scipy.signal.convolve
Similarly?
You are duplicating logic that is already present in Pow: it knows what to do with the odd and even exponents.
On Wed, Jun 6, 2012 at 4:51 PM, Stefan Krastanov
<reply@reply.github.com>
wrote:
>> @@ -1824,8 +1824,9 @@ def is_right(self):
>>      def altitudes(self):
>>          """The altitudes of the triangle.
>>
>> -        An altitude of a triangle is a straight line through a vertex and
>> -        perpendicular to the opposite side.
>> +        An altitude of a triangle is a segment through a vertex,
>> +        perpendicular to the opposite side, with length being the
>> +        height of the vertex measured from the line containing the side.
>
> The original formulation misuses the word "line" (i.e. infinite line).
> A "segment" is the part of the line between two points.
>
> The wikipedia article uses the terms "line" and "segment" correctly,
> however the meaning of "altitude" changes from "line" to "segment"
> throughout the text.
>
> This method returns the "altitude segment" so I think that the new
> docstring is better.

Yes, I was trying to make the docstring reflect what the function is
doing, and I agree with Stefan's observation about the wikipedia
article.

Does this read better?

```
An altitude of a triangle is a segment that is perpendicular to the
line containing a given side of the triangle and passing through the
vertex opposite that side.
```
This doesn't have to be done now though, I'm just thinking aloud.
str(self) will be an infinite loop...
I don't see that this is the case with ```_eval_evalf````. A ```git grep "def _eval_evalf"``` only reveals one argument, usually named ```prec```. I have renamed ```s``` to ```prec```.
Ah, good point. I think FiniteSet sorts the elements based on their hash. 

Union (after PR 1164) on the other hand would sort this for you.
I'll start with a nitpick, sorry :-) Did you probably mean "exponent"?
Since, trace is under core, it does not have any reference to scipy or numpy related classes defined in quantum/matrixutils.py. If I need to check for the args type in trace.py I will have to add a reference, which probably we should not do at this level.

Any suggestions on how to approach this?  One alternative is to add _eval_trace() and let the doit() workflow take care of it. but that would not be consistent with Matrix behavior.   I will think of something meanwhile, till I get some suggestions.
This one should be where all other "Enh" entries are.
If we're not using the context, why have it as a parameter to the PostCrashReportTask constructor?
This should be on your branch only - Should we bring this back for development? perhaps /sandbox?
I'd do...

    assert_equal(build.get('discovery_failure'), True)
Can we split this message or warn treats new lines specially?
I understand that you want to be consistent. However, I think it's rather useful to have the API documentation for Silex link to the API documentation of the Serializer Component... Nevertheless, I can remove it if you insist :smile:
Are the `-viewDidMoveToSuperview` and `-viewDidMoveToWindow` overrides necessary, given that you're overriding `-setLayer:`?
If I'm storing an encryption key for this user, do I need to store an origin as well?
That is part of the validation that I'm adding - if it is a concern, I can backport it?
"additional caching needs": not clear what that means.
Yeah, I completely forgot that its is already possessive >_<
add -> modify?
This should probably be moved under `if (!sizeof($error))`.
This again is behavior that isn't documented anywhere. 3.1 isn't supposed to break BC, if the guidelines are incorrect then these should be discussed and changed first. Not writing up guidelines that then get ignored because it is convenient
http://wiki.phpbb.com/Development_Process#Step1:_Identifying_the_appropriate_version_for_this_feature
This is not relevant to this PR; it should be removed eventually, but in its own PR.
You can use the driver name as an array key. You're assuming it is unique anyway.
Completely useless diff.
That sounds good to me.
RE mappend point: right. Also, we are already passing it in for inserLabeledRecordP. It's just about keeping the semantics between non `...P` and `...P` functions consistent.
мерси, ще поледна статията. И също ще пробвам да видя пак какво точно ставаше тук.
Иначе точно тази настройка май реши проблема при мен. А е закоментирана, за да не отиде в master.
Oops! That wasn't meant to be left in there. Pushing a fixed version
This will conflict with the YUI 3.5.1 upgrade.
dg nemikone :D
Correct response was ![](http://dague.net/wp-content/uploads/2010/01/welcome-to-the-world-of-tomorrow.jpeg)

You lose 200 points. Also, I don't have 4.5 installed yet, so I can't handle PR's. You'll have to get that other @quandtm chump to take care of it. Nyah.
These were never properly addressed in dependencyManagement
Ah, cool. Thanks for all that you've done to work on this, it's really appreciated.
PEP8 the class name.
Why does `longitude_of_prime_meridian` not default to `0`?
I thought this would be obvious (since it is all under allow_gc and the variable names are similar.
This op has two outputs, a generic variable and a tensor. I know the shape of the tensor. What is the correct output for infer_shape? I've tried `[(), self.shape]` before with no success.
Is this functionally different from instantiate the ACG with (MAX, MAX)?
OK to just drop the scary comment. I'll add the const issue to my long list of things to do next time I'm "in the area".
This TODO should really be done before pulled to master -- it can cause regression.
Spurious vertical whitespace above.
`node = (this['create-' + tech + '-node'] || this.createDefaultNode)(ctx, tech);`
attept -> attempt
So the question now is: what to do when select_related is True? Two use cases:

    # will work ok
    Species.objects.select_related(depth=4).select_related('genus') 

    # should get all immediate relations AND the Order
    Species.objects.select_related(depth=1).select_related('genus__family__order') 

I seems to me we need this split on select_related and we need to handle it intelligently, ie. 'count' depth of select related field and conditionally add it or not.
What about test cases that might come after this one?
Typo - m2m not m2n
Totally agree. I originally had a deletion of all of the notices, but that was way to cut throat, heh.
Yeah I'm not going to fix the whitespace for now.
I mean, there can be no body at all, you can't reuse it for what you do.
To further explain my original question, the reason why I'm asking is this:

If you have anything above a few 10,000s of tasks in queue at any given time (which, if I'm not mistaken, is exactly what will happen when you use a TaskSet), all of these tasks will be enqueued into a RabbitMQ queue named `celery`. Additionally, 1 result queue __per task__ will be created. We've encountered problems with this in the past; if a very large number of result queues are created, RabbitMQ can eat up a ton of RAM and cause failures or freezes. We had this issue with the oq-engine last October or November, and we learned from the error. That's why we ignore results of tasks and instead (https://github.com/gem/oq-engine/blob/nhlib-integration/openquake/utils/tasks.py#L210) and instead save everything directly to the DB.

Does that make sense? I remember this being a really frustrating problem, so that's why I asked about the number of tasks.

(Of course, this a bit of a tangential issue, since there's really no need for distributing these calculations anyway for the first iteration of this feature.)
NP, just wanted to be sure I didn't miss something.
bad. I'd like to use fragment cache then reset it with user observer, but no sure what should do to make it work on heroku
Ah, that module doesn't support 4.0.
And when both parameters are NULL?
I'd like to see an internationalized domain test as well.
use joblib.cpu_count for that
> New in version 2.7.

We can't use this an support python 2.6 testing.
no error checking is done here, package.json is assumed to always have a version field
ok, also geht es darum die Sortier-Richtung festzulegen wenn man zum 1. mal auf die Spalte klickt (um sie zum sortieren auszuwählen). Das ganze ist aber losgelöst von der Default-Sortierung in der die Tabelle angezeigt wird.
diesen threshold würd ich in der config mit reinnehmen. Meistens sind queries ab 0.250s schon als langsam anzusehen
Where does Syte use Bootstrap's modal dialogs anyway?
On second thought, that won't really work because they each need their own sets of fields and bindings.
Yes. I'll write them soon.
Not sure why this caused the tests to fail.  I recreated it though and it passes...
https://github.com/Migrii/blanket/pull/86
File size doesn't really matter in an npm dependency; it's more that every dependency is another moving part that could break. As far as I can tell, this is the only place that uses Liquor, and it would be almost just as easy to inject the content manually.
One more thought on this. I think getMenu() and getMenuItem() should always return the same object. Internally we would maintain the Menu and MenuItem objects for the menu system. Events sent to these objects would have "this" set correctly so that clients could easily change the menu's state based on events.

Does that sound right?

-----Original Message-----
From: Narciso Jaramillo [mailto:reply@reply.github.com] 
Sent: Friday, May 18, 2012 9:32 AM
To: Ty Voliter
Subject: Re: [brackets] Menu API Proposal (for review only) (#911)

> +     *  click
> +     *  rollout
> +     *  rollover
> +     *  hide
> +     *  show
> +     *
> +     */
> +    function MenuItem(id) {
> +        this.id = id;
> +    }
> +
> +    /**
> +     * Returns id of parent MenuItem if the menu item is a sub-menu, returns null otherwise
> +     * @return {string} parentMenuItemId
> +     */
> +    MenuItem.prototype = getParentMenuItem()

In general, do we expect people to be passing around and holding onto actual MenuItems, or onto their IDs? If they're going to be dealing with both, then our naming should distinguish between the two, so these should be called `getParentMenuItemID()` and `getParentMenuID()`.

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/911/files#r846242
We don't need both falsy and undefined checks -- falsy catches both null and undefined (along with empty string and zero).  Ditto for the matching code in removeBinding().
Usage message updated for all four scripts.
@redmunds 
Your suggested code has an extra right paren. Otherwise, it works correctly. 

@njx 
I did some testing with the suggested code and all code hints work except some cases in EWF.
I realize this was here before you, but `height` should be on its own line, consistent with every other property
Are we sure this is correct? Two calls to filepath, not really for this PR, but might be handy to check
should we name it `__pad` ?
Hackish and elegant at the same time. The best kind of solution.
And taking inspiration from `rugged_reference.c` what about something like:

```
rb_yield(rb_str_new2(rugged_str_new2(branches.strings[i], NULL));
```
Or add another section for Ubuntu. For Debian it is only necessary to add `mongodb` to the list of necessary packages.
in this instance, cancel_account is a helper method that does a process, rather than a single action.

i didn't write a separate method for the alert because it would add an unnecessary layer of abstraction between the user and selenium.
Use `pluginOptions.hasOwnProperty( key )`
Is there a point to using `return` here instead of branching with an `else` to call `_makeBubble`? Just a warning, not necessarily wrong.
Given that you do this in 3 places, can you make into a function?
Personally, I'd wait and do it en masse.  I don't think this provides us much value yet.
Is there a better way to figure out the relative path of the worker?
It is indeed as designed. So is this a matter for executive decision making, or something to refer to 'Design'?
This is a personal peeve: but I hate it when hyperlinks just point to `"#"`. They could point to at least `"#edit"` or even `"#edit-project-:id"`. That's just me though. I like the idea that nerdy users can see what action clicking will invoke from the URL.
Please don't use empty alt attributes; they are bad for accessibility.
.textContent = ''
Shouldn't this be `@vcr_recording = !File.exists?(full_path)`?
That should probably be `unstable` instead of `lucid` per the Debian people's request, right? Then there will be some sort of branch to create the Ubuntu source build requests with the Ubuntu release codename embeded in the version number, right?
这块代码跟ngx_http_log_bypass有重复，再用一个函数?
%d -> %i, the type indicator of ngx_int_t is %i.
eshtebah shod sorry
nadidam beyn line 53 va 70 faseleye !
-ms-filter
Can you call puts only once with multiline string?
If we are going to pass @org_label as part of locals, we should be able to define it on 105 as a local variable (e.g. org_label).  Or if it is only used by the edit partial, we could define it where it is used inplace.
Do-end for multi-line blocks.
wrong indentation here and for all method chaining on mock expectations
Based on reference - multiple entries are pointing to the same object.
The above lines are innecessary.  You can simply take only an `image` argument, and use it instead of `water_img` in rest of the method.
Yes we still unlock the lockscreen on incoming call. Not sure what to do about that.
nit: Remove the dump
I've tried to clarify these the version I will push up shortly.  Let me know if you think it's still unclear.
It also works to do ``speed_unit .to([1., 1000., 5000.])``, right? Perhaps another line should be added with tat.  When I first looked at this with just the "Getting Started", it gave me the impression that I needed to use `get_converter` to use arrays
I'll discuss my idea below.
Yea, I don't see a situation in which ModSrcTgt does not uniquely label a transfer.  I guess in future if we do great repartition this would not hold any longer, but I'm coding for the code base as it stands now.
But doesn't this thread *relock* the wallet? So "bitcoin-lock-wlt"
Is there any particular rationale here for the change of type?  I find unsigned types kind of appropriate for bitmasks, which is what this is.
Why are these controls included?
``callable`` is a type, not a class. So putting a backslash in front would be wrong
fix: http://code.google.com/p/gcalcli/issues/detail?id=84
Shouldn't it be `sys.getfilesystemencoding()`?

Also, we should be careful that it is *str*, not bytes, since presumably py3 distutils isn't this broken.

This is a stupid bug in distutils - I commented out the type checking in distutils, and the inlining with unicode paths works perfectly.  Unfortunately, I don't see a clean way of monkeypatching to just fix the checks.
I see that here you are using `sqlite` but below you still test against `sqlite3`.  Are both defined?  Are the right ones used in the right places?
you changed something that was correctly spelled to something that is not :)
The reorder happened when i broke my project file in a odd way. I will just run sort over it
Oh okay, thats quite a bit of work!
I was going to ask you if you would mind sharing your formatter setup. Never mind! ;)
I think this docstring is misleading. Really this resource is just a normal product-versions resource, which returns either a list of productversions or one productversion, but returns them with full environment data. It's not actually restricted to "a list of environments for a specific productversion."
Do you mean "when do we ever have allowable errors"? Things like 'database "%s" already exists', 'language "%s" already exists', etc. Basically a way of saying "Exceptions of type ProgrammingError with an error message like this should be caught and logged instead of raising"
Same thing here as in the "enable" case, the value being put should be in the payload, not in the URI. In this case, it would be convenient to push single properties to be modified, so perhaps a POST request is more in order (eg. "create a new resource" rather than "edit an existing resource"):

POST /db/data/autoindex/node/properties
"my_property_name"
I'm not worried about the implementation itself as the additional layer of indirection shouldn't cause a race condition, but the behavior in GameState might cause unwanted behavior.  This is entirely up for debate, and I don't mean to go all PHP-core-dev-team on you, but the expected behavior of the game is to save stats only after the end of the player's session, not on a per-song basis, as users might play, say, 2 out of 3 songs, screw the second one up, then pull his/her card AFTER the results screen, all while under the impression that the profile was never actually committed to the memcard (the "debate" part of this, I suppose, is whether giving them the ability to do this is right to begin with;  I don't care either way, but it's just the way that itg2/openitg cabs work).  So, Saving it after every song might be deceptive.
What was change here ?
Yes. And if the host is not the real one, then some things will be strange. For example when generating URLs with the new hostname pattern, the genrator will think that the current host differs from the intended one. And thus will generate an absolute url with the hostname instead of a path only.
Also, I'm not sure this is the right thing to do as for instance I have no problem running the test locally with PHP 5.4. Is it related to Travis somehow?
How can I correct the code? Do I must create a new commit and a pull request?
I guess it doesn't hurt, does it?