labofsahil/github-event-dataset-2012 huggingface dataset:
Since this triggers when a transaction updates the wallet, a potential DOS attacker would need to know both your IP (because no one will relay), and one of your addresses (pubkeys) to do their dust attack.  In my opinion, there are easier ways for them to DOS you at that point.

Also, the fee calculations are not trivial.  Adding that would require either duplicating a lot of code, or moving those calculations into their own function.

Rate limiting would be a bit easier, but the best way to do it would be open to question.  Make a queue to toss them into, and then have a thread poll and dump that queue every 30 seconds or so?

I made this after two people popped into IRC, within an hour or two of each other, both having problems figuring out the "right" way to spot new transactions coming in to their wallets.  This solves their problems quickly and simply.  I'd rather see this get out quickly for those people, along with a note or something about the potential danger (and how to mitigate it for critical nodes) than make them keep waiting while we figure out how to make it safer under circumstances that may or may not ever happen.

Then again, no one comes bitching to me when things don't work, so I defer to the judgment of the more experienced developers.
You can't change the binary serialization of these, as they are part of the network protocol (coinbase transactions use an IsNull() COutPoint as input). Using 0xFFFFFFFF as initialization/test value works fine.
ACK if you add a comment something like:
// height will be < checkpoint only when reading an old blockchain via -loadblock or -reindex
// (skip re-org'ed transactions in those cases)
Should be 'Your version of PHP does NOT have the mbstring extension loaded'
Well if you want to split the regex use string concatenation to join it back instead of using an array and implode.
What file does provide this? Is there an `@include` for it?
Is it really `i18n`?
And in other methods too
`@group.save!` instead of `@group.save`
Perhaps name it `arrow_taper`.
Did you mean to include these changes?

-[Unknown]
I think we should either make them public or remove them - since we're not using this API internally, there's no reason for it to have private parameters.
I tried `DISTINCT` first, but it turned out that it compares whole row (i.e., including `session` and `line`).  I am not SQL expert and there is many option for constructing SQL code so I am not 100% sure, but I think it picks up the latest entry.  I think this choice makes sense.  We can have more precise control by doing `SELECT MAX(session) MAX(line) ...` or `SELECT MIN(session) MIN(line) ...` but I think it over complicates the history class.  You will need to add MAX/MIN in `_run_sql` method.
why change this? the outcome of a failed assertion will be less informative in the new form
'U{{{a} * {b}}}' == 'U2' is just to make sure that the expression in double brackets is evaluated as JS before parsing the whole if condition.

'U{{{a} * {b}}}' == 'U2' >> dust's automatic parsing >> 'U{{1 * 2}}' == 'U2' >> the resolve method evaluates {{...}} >> 'U2' == 'U2'

I admit it looks very messy, but i wanted to have something to test that {{ .. }} was properly evaluated.
I think we can feed these in through the constructor and make _properties final.
Towel is ModelView. Even if the integration with ModelView does not fit from a conceptual point it should be here because it's necessary for search to work.
hm nah, keep it where it is.
Uh... I don't understand this one. This goes from True to logging.INFO?
The more pythonic way here would be `'[[%s]]' % doc.title` to create the string. Or `'[[{title}]]'.format(title=doc.title)`.
This should be `./node_modules/less/bin` or `node_modules/less/bin`
__whimboo__ wrote:
> Which case is it? I'm kinda interested in more details.

STR:
1. checkout 39c36d31f3fc
2. run `mozmill` with test manifest `mutt/mutt/tests/js/controller/tests.ini`
3. get the following, after `synthesize_events.js`

        ERROR |     runtime = test.get('time_end', datetime.utcnow()) - test['time_start']
        ERROR | KeyError: 'time_start'
oh sorry you do use argparse. so I think these two lines are very suspicious
Do you remember why this @add_person javascript method is added here? (I just want to make sure that we aren't breaking anything else on the site.)
I wanted to have the initial module i created then optimize it later. I know it is far from perfect as it is
Insert $name as second parameter?
and the summary must be ended with a dot.
@jfcixmedia If you run PHPUnit in strict mode, a test without any assertion in it will be marked as a failure
> I vote for removing this method, to avoid that a bundle remove the config set by the end user

yeah. i think there are only very rare edge cases where it might be necessary to modify the user set configuration because the Bundle cannot sensibly merge multiple overlapping configs for a setting (we have this case for Security firewalls). but this is a rare edge case we likely do not need to consider for this use case.
the curly brace should be on its own line for all methods
seems to be http://twitpic.com/99jcvh/full
Why using such high priorities ? and why using different priorities for json and xml ? They don't support the same formats anyway
properties should be camelCased, and you need to declare it in the class
The copyright line was cut and pasted from another file.  You are saying I need the project name under the filename?  I didn't catch that earlier when I made the change.
Indenting wonky here. I would put new opening tags on new lines (e.g. the \<strong\>) tags)
yeah! going for it :)
Idem as above
I think that most of OIIO code supports BOOST_FILESYSTEM_VERSION < 3 (or not), so this code will not work in all case (still it's better than now;)) - any solution for such users?

What about using MultiByteToWideChar - you can convert path to UFT-8 with that and use it with _wfopen.
parent_wth_children => parent_wth_child
why not extends AbstractRemoveStepHandler
This should be read-resource-for-update
so each module depending on server has it available ? It this a "no go" way of dependency declaration?
The validator should be "new IntRangeValidator(Thread.MIN_PRIORITY, Thread.MAX_PRIORITY, true, true)"
Is this still needed? Should be indented if so
no need for quotes on these
As above, shouldn't have to call `uniqueId()`.
This looked elegant enough, without being intrusive in keithamus's style.
> I heard that we won't have a USB notification overlay, maybe from @jcarpenter ?

As per our IRC chat, we will have a prompt appear when the user plugs in their device. @caseyyee is working on this.
Also why don't you use the 'hidden' html5 attribute?
please add a name to the anonymous function
why defer here? thanks
This seems unnecessary. You set it to true and then you set it to false. But you never read it.
I don't think it is `!==`.
Whenever an app is closed, `ModalDialog` should check if it is the current displayed app.
If an app is closed at background but `ModalDialog` close the foreground prompt UI which belongs to the foreground app...it would be sad :-|
+1. Scope is fine in this scenario, so I've updated 'Recents' to 'this' in both cases ;)
if (origin === target)
  return;
I think you should remove this line and #18-#21 and #33-#41 and #54-#46 also.
If you do not have any inheritance from borders. remove the entire rule please!
Why is this check needed? Is this for load order issues?
var entryPoint = app.manifest.entry_points[origin.split('/')[3]];
    if (entryPoint)
      return entryPoint.fullscreen;
    return false;
unable = unavailable?
@albertopq  this should be 3.6rem, if not we will have an UI issue, and unwanted space at the bottom of the document (between the scrollbar and the document end)
This type parameter still needs to be updated.
* Why a new method? We have  RepoInfo::getLicense for retrieving the license, returning an empty string if  there is none and also RepoInfo::hasLicense testing whether a license is available.
 * Apart from this the name is IMO misleading. I'd expect 'bool acceptLicense()' to ask for confirmation rather then to check whether a license is present.
Btw, tests still fail. You can run tests by running `npm run-script test`.
Ideally this for loop would be written similar to

```
extractedAssets = extractAssets doc
extractedAssets = extractedAssets[0..extractedAssets.indexOf(assets[-1..][0])]
```

But I don't know how to do cross-browser `indexOf` cleanly without jQuery or underscore.js
`Shared.ModelStore.load` returns a object, e.g. `{ id: 2, cliendId: 4 }`
This will clash with one of håkan's faces. Strictly it shouldn't matter, but it'd be nice to be didactic and separate the colours
This should really only accept "true" and "false"
Scratch that - tuple conversion gets delegated to mapPlusMap, of course
no shit.  I got sick of doing this to see the real stack trace.
? dont use Date.now(), hrtime is designed for this sort of thing, Date.now is unreliable for benchmarking
Use 0.0.x
Awesome, I hacked this out quickly at defcon and didn't have a test environment, I couldn't see rvmrc get loaded anywhere before this call though.

I'll rebase and resubmit.

The exit was purely because I was calling it all in a subshell
Follow proper Python naming conventions for this function name.
Alphabetical imports, please.
Remove break.
then i'll do it later. i'm not going to reference code that only exists in a separate branch
Nope, by default it should return an empty list as if there are no elements. the EMPTYLIST constant is immutable so I returned an empty Array List instead.
Pareil, pas de filtres ?
Also the variable name `$array` is totally wrong. :) Please use something *semantic*.
this `if` sentence is not needed, you've checked it already.
Hrm, that would get tricky with the current code, I think ... because there are assumptions in the test setup that b_b knows about all of the operations that will take place, via the `operations` list.  That's used for resetting ETS counters periodically (which we could implement differently, easily) and file naming & file handle opening at init time.  And if those naming & file opening things were truly done on demand, then it's possible that a CSV file could have "holes" in it: no lines between the start of the test and the first instance of an `Op` that b_b sees. I suppose is also possible to work around all those complications, but implementing it all would  get a fair bit messier.
this comment isn't quite accurate, since it's not applying info from a setid anymore
I'm just putting it out there, because if you're changing the schema, then maybe it's easier to change it to be i18n friendly today rather than later.... but maybe it's not an issue.

My thought was "128 ASCII chars" becomes "32 complex Chinese chars", which is that really enough? I didnt know if there was a "right way" to make a string column i18n-proof. But, it might be too early to think about this.
I'd suggest you not use macro to declare variable or member function.
Because it is hard to debug.
If you want to use macro, i think you should use CC_SYNTHESIZE here.
May be can do it like this

    CC_SAFE_DELETE(this->mBytes);
We need to set ReservedSlot for the case where the user passes in an anonymous function. Which if not rooted, or added as a reserved slot to the node, will be collected. We can do something like the following to overcome the subclass issue (Taken from cocos2d-iphone):

```
JSClass *klass = JS_GetClass(p->obj);
int slots = JSCLASS_RESERVED_SLOTS(klass);

if(slots >= 1 ) {
     JS_SetReservedSlot(proxy->obj, 0, argv[0]);
} 
```
No problem--let me get rid of those three.
This is exactly the point: you shouldn't allow `normalize("something")` when you obviously really want to modify a string using your normalization routine. What you should be supporting is:

    template <class T> void normalize(basic_string<T> &path);

If anybody wants to use this function, they should do:

    std::string path = "./foo";
    normalize(path);
    assert(path == "foo");

Also, this functionality sounds like it's already in Boost.Filesystem and is very specific to filesystem paths that I'm having a hard time seeing whether it's worth having in cpp-netlib.
The notion of free used to be `owner == CAP_FREE`.   That won't make sense with your next changes.  We can call it `destination == NULL` instead.
Sounds good.

On the validation problem btw, we could for TCP, UDP at least do a checksum validation. Future enhancement I guess :)
Can you imagine any other possible locator for this kind of controls which would actually make sense? If not, then ````:text```` and ````:index```` seem to be sufficient.

Anyway, merged changes in for now, so let's iterate in the future :)
why not 'separator' ?
It's not clear what `s` stands for. Consider using a longer name.
dec2a9279922bad1e2eafee98a9062a48d2ea585
Wow, great find. Router to the rescue.

r+
Nice cleanup to remove the dups
Nice catch on deleting from the primaryAuthCache.
followup to my followup, what's the utility of setTimeout here?
and finally here.
Just call this host and not aHost
What is `CB`? Do we need this?
I don't think this is the right place to check this. If the check is moved closer to the function itself, then the arity check will be done for other calls to the function as well (such as from other functions or from templates).
One of the mid-term goals is to get rid of all the thread-local values, so don't invest too much time in fixing this.
Moved to upstart - this should be addressed in 3980db8 (.sh was already in the base provider).
Can you remove your initials from the version. I'd be happy to put a contributors list somewhere so you get credit for your work.
typo s/destory/destroy/
Is there a risk of redundancy here? In situations when most specs require user to be logged in, the all the files will be prepended with `user_`.
remove the double quotes
there more hardcore!
The dependency must be declared in the "require-dev" section too
Fine with me, will change the fix accordingly.
missed `@`
The doc2rst convert a DocBook file, with .xml extension,  in a reStructuredText file, with .rst extension. If you want to convert file that are not related with ZF you have to omit the -n (normalize) parameter. In this way the file name and internal links will not be converted using the xmlFileNameToRst.
Sorry, THIS shoudl be enchants, not enchants1.
These should not be changed manually, but changed by the current hash path. (eg. /#active).
but thats no possible, since its a passed in object
yes. Dropdown adds `dropdown-menu` (if menu mode is enabled) and, if the user want, a custom css class. Do you think that it is not necessary ?
ok, docblocks do equally well also, but i'll add them.
@samiluukkonen Simple and clean! Käytetään jo olemassaolevaa infraa.
Hi @Ocramius 

Sorry I don't  get this ...

The named native queries follows the same behavior of name queries.
A query name should be unique in a entity and could be override in a sub class..

please, could you explain ?
Will fix later today, but at least this is done :)
You can inline this call
Would you recommend using ```Enum.all?(kw, match?({x, _} when x in kw_keywords, &1))```.
Which one is better call to a function or using partials?
you require a directory with an empty name?
The Aerogear HTML5 quickstarts use this term too. Should they change it there too?
A neater way to write this is:

```ruby
source_url =
  case source
  when 'github' then "https://github.com/#{recipe['repo']}"
  when 'wiki'   then recipe.key?('files') ? nil : "http://www.emacswiki.org/emacs/#{pkgname}.el"
  end
```
too many blank lines here.
And remove this one right?
Probably due to me doing a bad update with master.

I'll rebase.
This is all the metadata that is being persisted. Added a timestamp to provide a simple one-dimensional metric.
Should be replaced with "It depends on what you use but here are some base prices from Amazon."
Should not have title attribute.
Honestly, I wasn't sure if this was the best way to do this. But, it was the only way I could find. If anyone has a better idea how to differentiate between the two buttons 'Update' and 'Decline All', just say so and I'll fix it.
What's the difference here?
I prefer the shorthand.
Right, though I can add a check for "is None" or something similar before we get to "min."
definitely.  I tried several ways to send this information back through the exception pipe to the driver but couldn't succeed.  IOError turns it all into a string regardless of what it is.
Should be `if (null !== $this->_redis) {`.
this was mainly because 
a. hostgroup_puppetclasses is really long :)
b. we already have host_classes
c. maybe fit better when we have chef? ;)
begin statement without a rescue statement is useless? did you try to catch an exception?
Yop, rake can be used for loading the environment, but I wanted better options parser. Rake also produces some logging output about loading the tasks etc that is useless here (and would pollute the output showing the configuration).
unbind more or less means `stopListening` as of 0.9.9
hmmm, I see. OK that's out, too!

Do you know who the dev for this is? It's a pretty poor ux as it does not give a message of success nor failure (and fail it should if you don't have an internet connection). Do you think we could get them to enhance it for the benefit of UX, which would, coincidentally, assist us? ;)
Looks like there is two things happening inside here. The node scanning, could possibly be it's own thread, scheduled at it's own interval?
[See this comment](https://github.com/cwhelchel/SteamBot/commit/e41c66c1de91706890242a9e72b5c76c6fffad2b#steamtrade-trade-cs-P9). I still do not understand that either.
Then these could be `setHighlightStyle()` ... since functions should really be verbs
You can use <code> tags to make this look a lot better:

    <code><var>AREA</var>\ \text{cm}^2</code>
Not yet, but I'm on it for the next pull request.
Yeah, thanks, I know what the schema.rb is... :) Just wondering why there'd be a change to it that'd be required for this pull request...
Where would we want pages with ignored?
These should be removed. :)
Definitely not.  Just something that might be cool to have as an option in the future. :)
I'm not quire sure I follow what this is trying to do. Converting the timestamps seems like a useful thing, but why hyphens-to-underscores? If we need to do that purely for database storage, then I think that part is better placed in storage.clj. I think that the functions in this namespace should be about parsing reports, transforming them from wire format to internal object representation, and validating that they're well-formed.
I've added `@api private` for all of the added private functions
Can you document the types allowed?
constant for 1000?
True. Give me a few seconds to change this.
is plain() needed?
Given that I have agreed that you can return a string, there is now no need for the buffer argument. I know it was in the original spec, but it serves no purpose in the revised spec, so remove it.
Hang on, I just realised I got my ends inverted (ie. my backs and fronts).

Since the basename gives us the last component of the pathname, and forward slashes are acceptable on Windows, it seems like this should pretty much just work, even if we do get some funny URL or our filepath library has different behaviours on Windows and *Nix.  No?
What I would recommend is creating a new local branch or tag before removing it, so that later you can see the diff and put it back in, or continue development after we merge this request into master.
langere variabele namen zijn meestal op de lange duur meer leesbaar. Zou er dus "mouseX` van maken.
Er ... I might be talking about saving a few bytes, but how is:

    if ( ! isset($route[1]))
    {
        $route[] = $class;
    }
    if ( ! isset($route[2]))
    {
        $route[] = $method;
    }

... more readable or clearer than:

    isset($route[1]) OR $route[] = $class;
    isset($route[2]) OR $route[] = $method;

Second approach looks way cleaner and easier to read to me. The first one is simply ugly.
Other driver(s), uniformally return `string` for `max_length` property instead `integer`, so this pull obey to that standard.
Интересно, как же тогда указатели сравнить.
Всё хорошо, просто на заметку: если переименовать название метода в description, то можно будет выводить в лог следующим образом:
`NSLog(@"%@",cell); // cell = [[Cell alloc] init];`
в этом месте переменная newCode принимает значения от 0 до 2 (остаток от деления на 3), то есть значение @"G" будет передано ТОЛЬКО в том случае, если до того в ячейке было @"C". Я вам советую переписать так, чтобы значение генирировалось каждый раз снова при совпадении random() % 4 (от 0 до 3)
Ах, и вот еще, вы использовали отдельный метод, ваше право. Только получается, что я могу использовать этот метод в main перед вызовом метода hammingDistance. Так, например:

`NSArray *someArray = [NSArray arrayWithObjects:@"H",@"J",@"K",@"P",@"O",nil];
[Cell1 setDna:[Cell1 setRndomValues:someArray forDna:[Cell1 dna]]];`

Как итог - Cell1 мало того, что будет состоять из 200 элементов, так еще и половина будет из непонятных значений.
Единственное замечание есть. Если мутировать 1%, то нулевой элемент никогда не сможет мутировать.
А так да, очень интересное решение.
lifespan of file cache in localStorage in hours
Ah yes right. I was thinking of the build_mac.sh script using brackets-shell as the current directory.
On sqllite testing, the test passes without the `.replace(microsecond=0)` on mysql I needed to add it for the test to pass. But with it the sqllite test fails. There must be a better way.
please use 4 spaces per level for the indentation, not tabs or 2 spaces
Yep, it does. Jeff told me not to fix lint errors in other people's code but I can't help myself! :D
I don't understand why you changed the way variables are defined.  I prefer one var statement to multiple.
Given we have a deployment infrastructure which makes it very easy for us to deploy and run a Git repo, I'm tempted to say let's just rip out this compile/minify business and save ourselves some annoyance.
Could you add something about how the main thread is kosher?
why not define `uint8_t *bmptr` and get rid of (dummy anyway) typecasting below?
For the lines above, @tuhoojabotti might have something to say. This part of the pull request is not made by me.
add a note that raws[0] is modified inplace (if True)
looks like a broken rebase (why are these lines removed?)
I'd change the param to be "newOrPersisted"
I would do `params.fetch(:answer, {})` so that this line will never fail. If you get posted something with no `answer` you'll get a `NilError`
You should end all array items with a comma, even the last one.
`depend` should be `depends` with an S
This empty line should exists
默认页 -> 默认主页
Hi, thanks for the prompt replies. I have updated my commits as per
your suggestions. To keep the history clean, the commit ids are
changed, so they will look like fresh commits.

On 12:37 -0700 / 30 Sep, HyeonJe, Jun wrote:
> > @@ -79,7 +80,7 @@ var IRCLink = function(hostname, port, ssl, selfSigned, nick, realName, password
> >        for (var i = 0; i < instance.sockets.length; i++) {
> >            instance.sockets[i].emit(event, args);
> >        }
> > -      
> 
> I think this line doesn't have to be changed.
> Trailing spaces are not good, but a patch must only include the things concerned with the patch itself.
> Please remove this change.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/thedjpetersen/subway/pull/170/files#r1724964
This reads oddly when capitalized.
Eventually we probably want to replace this with `compass-bootstratp` or some similar, and cleaner, piece of code. Compass can handle all these background image things for us, and save a ton of time and code
Ughh. That was ugly. I wonder why I wrote that. (All comments so far have
been addressed.)
Please remove commented lines.
no end necessary
Je n'aurai pas le temps ce soir Steph, mais je devrais pouvoir faire ça demain soir! D'ici là, je vais ouvrir des issues sur GitHub et peut-être que quelqu'un le fera...
Witam,

Moim zdaniem "rok temu" brzmi lepiej niż "1 rok temu"
I do not understand, if I hide some columns how can I set visible columns after that ?
You can't call this two methods on the same grid. It's your goal ?
Wouldn't hurt, but considering the content can't be from anywhere but the TT error I took it for granted. I get a 500 there though, which makes sense IMHO; 404 should only occur if there was no corresponding route defined.
This use of iterator() doesn't save anything here, since the queryset is not kept anyway. Might as well loop directly over it.
> it's a port from C

That was obvious.  ;-)
s/trhow/throw/
So the reason we're sticking with <code>&</code> is for the sake of not touching the frontend? I wish we could easily use <code>/</code> - I fear <code>&</code> may be misconstrued by CloudFront somehow (and the leading <code>&</code> feels odd).
Darn, you're right. Switched it back to save it originally.
I certainly hope so - for all different transaction modes. Hence my question to @mirceamarkus. 

Sent via the InterWebs

On 29 May 2012, at 18:36, Sanne Grinovero<reply@reply.github.com> wrote:

>> @@ -214,8 +214,6 @@ private Object enlistWriteAndInvokeNext(InvocationContext ctx, WriteCommand comm
>>          if (ctx.isOriginLocal() && ctx.isInTxScope() && !ctx.hasFlag(Flag.FAIL_SILENTLY)) {
>>             TxInvocationContext txCtx = (TxInvocationContext) ctx;
>>             txCtx.getTransaction().setRollbackOnly();
>> -            final LocalTransaction cacheTransaction = (LocalTransaction) txCtx.getCacheTransaction();
> 
> Cool to see you checking it all :)
> yes those - among others - were the doubts that prevented me to make such a pull request myself as I don't knot how this fits in the greater design.
> 
> Aren't there tests already checking for proper cleanup?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/infinispan/infinispan/pull/1147/files#r893972
it isn't really (group == functional), it just stress the conditional operations. I'll rename it into ConditionalOperationConcurrentTest
@Sanne I've added comments to all the methods that are not synchronised describing why.
This class was introduced in 5.2
On 25 Oct 2012 13:11, "Galder Zamarreño" <notifications@github.com> wrote:

> In
> cli/cli-server/src/main/java/org/infinispan/cli/interpreter/Interpreter.java:
>
> > @@ -95,9 +97,13 @@ public void stop() {
> >     }
> >
> >     @ManagedOperation(description = "Creates a new interpreter session")
> > -   public String createSessionId() {
> > +   public String createSessionId(String cacheName) {
>
> What is the effect of this change in old, 5.1.x, clients?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/infinispan/infinispan/pull/1411/files#r1939412>.
>
>
This is inconsistent with enumerating over Cocoa collections, which use `BOOL *stop` instead. I think we should match that behavior.
Well, thanks for doing such boring reviews :)
Pep8 patches are always long tedious.
The comment doesn't really help, nor does the docstring. This code is not very pythonic (I would guess this was written by someone who comes from , and I think a good explanation is necessary in case of more code "cleanup".

Also, this code will break if the function called returns the integer ``0`` (if that is ever the case..)

My 2 cents,
N
In the loop starting on line 8002, I want to iterate over the datasets in reverse order so that the first input goes on top. Note that I put them back in the correct order at line 8027.

Now that you point this out, it might be better to reverse the loop than the vector. This would result in the first dataset being plotted at the bottom, which might be better.
Ok, the variable naming here is ridiculous.  We have 'x', 'X', 'x0', 'x1', 'X0', 'X1', and their 'y' equivalents.
Yeah. Minor nit though - you could merge with this issue.
The test fails.
@miketheman it is possible with the way the provider was reworked with COOK-1521, that it may not update an Amazon-provided repo file.  If that is the case then it may be a sign the provider is becoming over-engineered for edge cases like this Amazon one.  I'm in the process of signing up for AWS to see how this affects that particular platform.
"Classes et espaces de noms entièrement qualifiés doivent"
If these keys will be the same, can you remove one or the other?
@nicolaasmatthijs Can we remove the 4 extra spaces here & on the lines below?
@bp323 see comment above
I'm trying to find a better solution than the current switch - it's only the first time you see it that it is confusing. Would it be ok to change "Draft site / Published site" to "Draft / Live" ? might be able to find a way to show both labels then.
well now that enabled? depends on the state of collecting?, this shouldn't be an issue. Or maybe I'm not fully understanding your concern? I can walk you through the chickenscratch on my notepad as to why I think it should work ?
I'm stashing the local btree handle a bit earlier, so I'd rather if you do the merge.
The gitingore is just for project files.

.DS_Store, destop.ini and files like this have to be in the .gitignore_global
https://help.github.com/articles/ignoring-files
Этот файл удалить - не используется
He put a `<br>` but didn't put it in code things so it parsed it. He's asking if it makes sense to put a new line here, since `<p>` pretty much does that for you.
The class name is not autoloadable using a PSR0 autoloader. It should be in the sub-path "ZendTest/Mvc/Service/ServiceManagerConfigTest.php".
Are you sure ``$request`` has the correct type as ``object<Symfony\Component\HttpFoundation\Request>`` is not a sub-type of ``string|array``?
Are you sure that the property "hash" exists?
Not that I'm against, but I have a feeling that this will become multiple functions instead of one -- In which case, would we want to replace `createProject();` with subsequent calls and move them to the outside scope?
Судя по плагину к jQuery у вас есть эта библиотека, у нее есть полный набор инструментов для работы с ajax, почему его не использовать?
Went and looked at the spec, a HEAD request by definition won't return a body:
<blockquote>
The HEAD method is identical to GET except that the server MUST NOT return a message-body in the response.
</blockquote>

So we really shouldn't return an exception here.
I don't think this commit is useful, as the VERSION file shouldn't have changed. So it wouldn't happen.
Might steal the phpbb git hook that prevents whitespace from being commited and then fix all titania whitespace at some point.
Not really to do with this commit, but I don't like these calls to get interface and dynamic cast, there should be a either a specific queryDFS[impl] call to get the instance or an extended interface that it gets.
I investigate that a little bit more and I honestly I do not see any reason why someone would like to use it. 
Could you provide some scenario for default_service option?
What does the $syntax do - does it have a special meaning?
This is not right. We should be checking equality of the map, not of the list:

http://docs.oracle.com/javase/6/docs/api/java/util/Map.html#equals(java.lang.Object)
Да, согласен. Экспериментировал, а потом забыл убрать его.
:) I was really genius when I was writing this.
Please be consistent. We are using "login" elsewhere. Not "log in".
I did this because we are comparing dates. I can change it back.
This should be probably case-insensitive search. Fix or add TODO.
Oh well the idea is it grabs a list of the members of the house and
populates the box with those names.

On Sat, Nov 17, 2012 at 2:09 PM, Conor Reisman <notifications@github.com>wrote:

> In fullhouse/templates/dashboard.html:
>
> > +					</p>
> > +					<p>
> > +						<label form="time"/>
> > +						Time: <input type="time" name="timedue" class="time-due"/>
> > +					<p>
> > +						<label form="assigned"/>
> > +						<p>Members: </p>
> > +						<select multiple="multiple" name="members" class="member-list">
> > +							<option value="1">Janette</option>
> > +							<option value="2">Conor</option>
> > +							<option value="3">Brandon</option>
> > +							<option value="4">Jeff</option>
> > +							<option value="5">Cornel</option>
> > +							<option value="6">Shahaf</option>
> > +							<option value="7">Nat</option>
> > +							<!-- TODO: Implement members, above are just placeholders
>
> I meant the commented out part. We should not have hardcoded values in our
> code anywhere.
> … <#13b106b9738372bf_>
> On Sat, Nov 17, 2012 at 2:06 PM, bkgomez <notifications@github.com>
> wrote: > In fullhouse/templates/dashboard.html: > > > + </p> > > + <p> > >
> + <label form="time"/> > > + Time: <input type="time" name="timedue"
> class="time-due"/> > > + <p> > > + <label form="assigned"/> > > +
> <p>Members: </p> > > + <select multiple="multiple" name="members"
> class="member-list"> > > + <option value="1">Janette</option> > > + <option
> value="2">Conor</option> > > + <option value="3">Brandon</option> > > +
> <option value="4">Jeff</option> > > + <option value="5">Cornel</option> > >
> + <option value="6">Shahaf</option> > > + <option value="7">Nat</option> >
> > + <!-- TODO: Implement members, above are just placeholders > > It just
> makes the select list with the pre-populated names, the type of > list that
> you can hold ctrl and select multiple items. > … <#13b106859f400e7b_> > On
> Sat, Nov 17, 2012 at 1:32 PM, Conor Reisman <notifications@github.com>wrote:
> In fullhouse/templates/dashboard.html: > + </p> > + <p> > + <label
> form="time"/> > + Time: <input type="time" name="timedue"
> class="time-due"/> > + <p> > + <label form="assigned"/> > + <p>Members:
> </p> > + <select multiple="multiple" name="members" class="member-list"> >
> + <option value="1">Janette</option> > + <option value="2">Conor</option> >
> + <option value="3">Brandon</option> > + <option value="4">Jeff</option> >
> + <option value="5">Cornel</option> > + <option value="6">Shahaf</option> >
> + <option value="7">Nat</option> > + <!-- TODO: Implement members, above
> are just placeholders What does this code do if run? — Reply to this email
> directly or view it on GitHub<
> https://github.com/Full-House-UW/fullhouse/pull/121/files#r2160285>. —
> Reply to this email directly or view it on GitHub<
> https://github.com/Full-House-UW/fullhouse/pull/121/files#r2160336>.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Full-House-UW/fullhouse/pull/121/files#r2160341>.
>
>
break missing ...
sisTimeout? Typo?
Well, I'm on mac and I'm using postgres.app (postgresapp.com) which I can just start and stop. By default it only listens on localhost and doesn't have a username or password.  

On Linux, you'll probably just setup postgres to listen to localhost only and remove the username password requirement.  

On Friday, October 26, 2012 at 9:30 AM, mnewsome wrote:  
> In chipy_org/settings.py:
> > @@ -30,7 +30,7 @@ > > MANAGERS = ADMINS > > -DATABASES = {'default': dj_database_url.config(default='postgres://localhost:5432/chipy_org')} > +DATABASES = {'default': dj_database_url.config(default='postgres://postgres:postgres@localhost:5432/chipy_org')}  
>  
> I'd like to set up a dev version of Postgres app...never done that before.
> … (#)
> On 10/26/2012 08:49 AM, emperorcezar wrote: In chipy_org/settings.py: > @@ -30,7 +30,7 @@ > > MANAGERS = ADMINS > > -DATABASES = {'default': dj_database_url.config(default='postgres://localhost:5432/chipy_org')} > +DATABASES = {'default': dj_database_url.config(default='postgres://postgres:postgres@localhost:5432/chipy_org')} Here you're setting the default username and password to postgres. This will break other installs. Development versions of postgres, like postgres.app default to no username password. You'll want to set this in your .env if those are you postgres credentials. — Reply to this email directly or view it on GitHub <https://github.com/chicagopython/chipy.org/pull/6/files#r1952964>.  
>  
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/chicagopython/chipy.org/pull/6/files#r1953448).
Yep, you're correct. I wasn't paying attention closely enough.
Revert the lines that are the same. Perhaps the difference is the end-of-line?
Pass in the node-auto-index (which should probably be stored in a varaible in RestIndexManager
Yup, thanks for this notice
A bit opaque when templates don't return to each other but work on something like `.flat_list` instead. Not sure we ever do that elsewhere.
If we had `$0` and `$1`, we will execute in `$1`'s runtime? I think `$0` should take precedence, that's the most recently selected element, I wouldn't expect overriding because the previously selected was on some other runtime.
Can remove this blank line.
What about a different name for this function if there could be more appropriate name?  remove_extension and remove_suffix   sound like they are doing the same thing.
Use

```ruby
Time.zone = time_zone
Time.zone.parse ..
```

instead of `parse().in_time_zone` because the latter would parse into UTC (or whatever the default timezone was) then convert that time into `time_zone`
I know Ryan, beginners mistake, won't happen again!
Can you change this to an absolute import?
That's a great idea…I actually started implementing something like this, but let it go early. In my version, `for_code_point` returns a hash of values; something like:

```ruby
{:codepoint=>"1F3E9", :name=>"LOVE HOTEL", :category=>"So", :combining_class=>"0", :bidi_class=>"ON", :decomposition=>"", :digit_value=>"", :non_decimal_digit_value=>"", :numeric_value=>"", :bidi_mirrored=>"N", :unicode1_name=>"", :iso_comment=>"", :simple_uppercase_map=>"", :simple_lowercase_map=>"", :simple_titlecase_map=>""}
```

Wouldn't that be simpler than returning an instance of `TwitterCldr::Shared::UnicodeData::CodePoint`?
All strings should be [translatable](https://docs.djangoproject.com/en/dev/topics/i18n/).
This comment is redundant with the docblock comment on Pheanstalk_Command::wantResponse() in the interface, so it should be removed.
If more clarity is required, it should be added to the interface docblock.
ok. we'll stick to lowercase.
> According to this, you can write:
> 
> My Main Page                         {#mainpage}
> ============

You are correct.  Now that I did this, the dox show up and it's somewhat less ugly in markdown mode (though the mainpage tag is never going to look pretty).  I think I prefer this version.
> Also, I think I probably prefer the underline notation for the level 1 and 2 headers. It makes them stand out more in the text file version. It might be worth switching to that for both this and INSTALL.md. For the level 3 (and higher) headers, the only option is ###, but I think that's fine for them.
> 
I have to run to a meeting, so I will have to play with this later.
Here's the Python syntax:
```
try:
    result = _galsim._...
except RuntimeError as err:
    if strict == True:
        raise err
```
It's ```RuntimeError``` rather than ```HSMError``` because we haven't wrapped ```HSMError``` for Python (and we'd have to do a little extra work that I forget how to do right now because it's an exception).  If you don't do that, C++ exceptions get turned into Python's built-in `RuntimeError`.
I am that good.
this changes behaviour and might break things.  is there a good reason to do this?  better to leave behaviour the same and issue a warning.
Useful to include more details in log statements, such as the app that is about to be loaded.

Think about what it will look like when you have a massive log with log statements originating from hundreds/thousands of classes, and you've been sent this log by some customer who has encountered a problem. You want to figure out what the customer was doing and where it went wrong, potentially in subtle/complex race-conditions etc. So thinking about what to include in the log statements is a really important habbit to get into.
maybe name it "javaType" to avoid confusion with catalogItemType?
It's not touched in this pull request, but just noticed that `waitUntilInterrupted` is wrong (it doesn't have a loop around the wait() so the thread could return from wait spontaneously to return early (and thus potentially exit the app).
Would be nice to use \@throw in javadoc, to say exactly what will be thrown when
hopefully this isn't needed
Is this going to work? If you get two requests for two different type T's, the as check will return null, and as a result the old factory will get overwritten with the new one - which will result in the factory eventually being garbage collected. I think we may need a Dictionary<Type, ChannelFactory> instead and store them all by type.
safe, safe_pp and safe_str seem to be obsolete now that w is working fine?
you don't need to use global here, you can change it inplace:

```python
SERVICES[:] = []
```
Just a note: This doesn't pull in the arguments from the file.  A quick:   `CONFIG_OPTS=$(cat sbtconfig.txt | xargs echo)`  should do the trick though (with blank line removal and # removal....
This seems weird to hide the page header.  It makes some pages quite unclear as to what is going on, especially since the breadcrumbs have gone too.  If you looked at `/offers/metadata/` without knowing how you got there, it's hard to understand what the form is for.

It seems that we've got to have at least one of breadcrumbs or page header
remove oxid scope
I'm fan of having css properties sorted by alphabetical order. Why did you move it here instead of leaving it where it was?
这里也是，要区分显示“我” 和其他用户的区别
It would seem that way. I don't know why these particular values were chosen. Maybe that's where the Mercator projection breaks down too much, although that only makes sense for latitude, not longitude.
It is just confusing a basic example, and making it seem like PSR-X is the ONLY WAY
В случае с ```\n\r``` у тебя будет лишняя пустая строка.
Ты закрываешь здесь System.in
usage - это описание, что и как делает программа.
I've seen code like this in other test files. Is there way to factor this into a utility that you can reuse across all your tests?
sorry I didn't notice that :)
run pyflakes on this file you have a lot of unused imports

please put pyflakes in your editor as it's the first thing I try on a file. It saves time to both of us
I'm not sure if it will matter, but there is a vectorized way to express this if `classes` is an array rather than a list (and I think `np.unique` always returns a list):

    weight = (y == classes.reshape(-1,1)).sum(axis=1, dtype=np.float64)
    np.reciprocal(weight, weight)
    weight *= classes.shape

I skipped the division by `np.sum(weight)` here, since (unless I'm mistaken) that should always be 1.
BTW, have you tried to check if the dense and sparse RP yield to visually similar outcomes on this example?
separable
It does not raise any warning or exception with for example-2
(which might mean n_estimators - 1).
Well, I agree this is overkill especially if there are numerous forms. It works, though it needs to be optimized. Will look into it.
Ouch, sorry, I had completely forgotten to clean-up code. Will fix.
it doesn't look like you've pushed all changes? - this pickfirst is still here!
This is legal:
```
namespace bb
{
    namespace cascades
    {
        class Container;
    }
}
```
isn't it?
TiLogger is not a TiObject and I don't see any reason for it to be.
DocEtude est ds le package models.
Je sais pas ce que ça faisait là ^^
Don't look yet, I broke directory recursion... :S
Provide *a* simple method *to implement* spatial index via annotations
Don't really understand why Roxie is special
No, n (scaled) is based on elements in stream.
I'm following the definition of 

ILocalGraph::getLinkedResult (eclhelper.hpp)
Yes - they already are for the version of CHOOSE() that returns a scalar
The class is responsible for two things now `Fiddler` and `MockServer` and from cohesion perspective, a two separate filer classes have to be maintained (i.e. `FiddlerFiler` and `MockServerFilter`)
This code was building the links from the front page - I am not sure where the replacement is but it's building links like this:

```/home/products/topcrasher/byversion/WaterWolf/5.0a1/7```

That should instead be:

```/topcrasher/products/Firefox/versions/17.0a2?days=7```
Hah, of course. No that's fine.
seems like this should be something like
 ```log 'token endpoint', :level => :error```
I originally changed this to check replies for a few reasons:

* Multi-window sync
* iframes
* It now waits for dependent assets, including JavaScript

Unfortunately, the `pageLoading` signal is just _differently_ reliable; for example, you'll get multiple signals for a page with iframes, and the number of start signals doesn't always equal the number of finished signals. I think we're going to need to deduce the actual page state based on a combination of reply and page signals. That's sort of what we're doing now, but there are clearly holes in the existing logic.

Let's merge what you have; we can change the way we handle replies in a later commit if we need to. The UnsupportedContent changes are nice, and this at least reduces the possibility of a deadlock.

Related: I'd like to build up a suite of tests that reproduce these kinds of issues, similar to the `malloc_spec` we used for finding the segfault in 4.8. That way, we have tests we can run before release that reduce the possibility of reintroducing race conditions. I'll give that a shot this week.

Anyway, nice work on finding the finished signal edge case.
Isn't the "local cache" becoming too much intrusive? Have to think about that.
Чиний good, bad гэдгийг зөв, буруу гэж орчуулсан байсныг харсан л даа. Надад бол муу, сайн гэж орчуулсан нь зөв санагдаад байгаа юм. Буруу гэдэг чинь ажиллуулахад алдаа заах юмыг хэлэх биш үү, энэ кодуудыг бол ажиллуулахад алдаагүй зүгээр ажиллана шүү дээ. Ингэж бичвэл сайн гэж зөвлөж байгаа болохоос буруу зөвийг нь шүүж байгаа хэрэг биш шүү дээ. Тиймээс сайн, муу гэж орчуулах нь зохимжтой санагдаад байгаа юм. Чи юу гэж бодож байна?
the generated `ThumbnailFile`, not `Thumbnailer`
Unfortunately not @ctweney 's scripting skills but just copying some files around.
I think `PUT` would be a better method for this. Even if we don't want clients to have to send the full representation we should still use `PUT`. A PUT w/o a complete long form representation is not a "partial put" unless the server is unable to take that representation and convert it into the full representation using the existing state of the system in an idempotent manner. So PUTs with a range header are partial and not ok because if you repeat them they are not idempotent. PUTs with a "lite" representation of the resource are *not* partial puts, even though that "lite" representation does not include all the resource information.
just <hr> in HTML5
`(> (author-count book) 1)` palauttaa sellaisenaan booleanin, `if` ei tarvita.
Should that now be 

if ( result != null )

??
Can you use relative namespace ? `new Queue(...)` is enough.
Wilco.
I don't think your timeout argument is really used in this function, since no function actually uses the 'to' variable.
do you really need to serialize the finder ? It is not necessary to rebuild the resource as it is instantiated inside the class anyway. So it could be recreated in unserialize(), making the serialized data smaller
I tried doing this, and think the if statement for multichannel is needed - otherwise a 3d gray-level input will result in parsing each 'slice' as its own channel.  I can simplify it slightly by getting rid of `if not multichannel:` and replacing `else` with `if multichannel:` in the current form.

Alternatively, we could use the simpler version if the data parsing of `random_walker()` is changed slightly such that when `multichannel=False` we change the shape of the array with something like `data.shape += (1,)`, guaranteeing the last dimension will be channels.  This might require other modifications, but might be more elegant going forward.  

I could go either way; thoughts?
Hmm, I don't understand this stop condition. I would've assumed that it would check for that one of the first 2 dimensions is equal to 1.
What about `it "can test the negative case"` or similar?
ok let's start the good practices here!
Broke this out into a standalone function (and new tests) with my last submit.