labofsahil/github-event-dataset-2012 huggingface dataset:
Pretty sure #include <string> and #include <iostream> will throw errors on compile. Added them after prog1...
By the way, I just got everything working smoothly for the submission form. File upload is working great too :-D
My branch here for EJBCLIENT-18 issue has a slight change in this method https://github.com/jaikiran/jboss-ejb-client/blob/ejbclient-18/src/main/java/org/jboss/ejb/client/EJBClientContext.java#L253. But (like you say in our mail conversation), the associate shouldn't happen in a synchronized block. Furthermore, I don't think it's possible to use a receiver without a successful association, since the associate method invokes the EJBReceiverContext.close which in-turn unregisters the receiver from the EJBClientContext. 

I guess we have missed a case in RemotingConnectionEJBReceiver#associate() where we aren't calling the close on the EJBReceiverContext if the association doesn't occur successfully. I'll discuss this with you in that mail
Ah thanks! Just found the "lhm" gem which seems like a special case of this repo. Also, to confirm, it seems like the journal table concept has been pulled from LHM now?
Oups, faut virer tout le elseif là x) Tu tiens peut-être la cause du bug !
> How would you express that?

I would use italic aka \<emphasis\> in the Docbook world

arg\<emphasis\>PARAMETER_INDEX\</emphasis\>
Woo, docs!
Good job on the update. Are we going to have these in a separate repo? I guess it makes sense, but at least let's put all the style guides in one repo. This one will make a good starting point for the other ones.
What error do you get when you run it against later versions? If it fails, then i guess for the later versions we need to use the GP for address lay out, it could be you have a different one from what i have, i recall adding a comment about this somewhere
This seems counterintuitive to me.  You're not actually adding anything as the method name implies.  You're formatting a string and returning it.  Would it be better if this method appended an optional graph as it implies and the append method only added additional patterns? Rather than having append do both functions.  Is there a need to only format a string?
Ok, I'll revert the API part. They do mean 'country' though, right?
Why did you change the "let"s to old school instance variables? :)
thanks! I had copied the code from someone else, so cleanup needed to be done!
@jeffmccune for example,  I don't know how to define state as "attribute :state" and make it return the value of attribute power_state, so I define it as  "def state; power_state end".   "attribute :power_state, :alias => 'state' " doesn't work as I expect.   Could you give some hints on implement it?

i think after we resolve all comments here, I can make some change to the pull request accordingly.

Thanks!
I had this as dynamic, but left it static because I'm tweaking the completion for the `--profile` and `--host` options to be of type `file` and `url` respectively. I'm almost there. 

If they're dynamic, I have no hook to treat them differently.
Since the goal is to drive low-level DOM and graphics color with this module, it may be worth simplifying the API rather than complicating with more options.  Minimal function and object/array overhead should also be goals.

In order to fast-path the public API, we may want to consider streamlining to string in/string out, and leave the config/return wrapper for `Y.Color.convert(config)` and expose `Y.Color.toArray()` for converting to array.

It would also be nice to have a way to test the type, for both internal use and developer convenience; maybe `Y.Color.isType('#000', 'rbg')` and/or `Y.Color.isRGB('#f00')` etc.
I recommend changing this to:

    typeof c.extra === 'string' ? c.extra : Y.QueryString.stringify(c.extra)

Checking for a string is simpler and means we can avoid calling a Y.Lang function. Also, you may want to verify that `c.extra` is defined before passing it along like this; it should almost certainly be optional.
Don't need this limit when count is specified.
Just add:

```javascript
window.Wikia = Wikia;
```

at the bottom.
Absolutely, and I'll open a ticket to add that.
this definitely needs testing.
What should this function return?
I don't like initializing RPM_BASE_DIR with respect to BASE_DIR actually and this not going to work for anybody who has non-standard git repos setup. I am still thinking of a better way to handle this. Suggestions welcome. It's not going to affect rpms though, just the dev setup.
Yup.  Same mistake copied & pasted from pulp_rpm changes.
Fixing.
I just use github editor xD
This is how the account (User model record) and the card are bound now.  The @card.save makes it permanent.
I wonder if using SELECT to switch between the two would be more obvious?
Use camel case -- "createFlag".
Ok, so have a look at it.  You'll have to make sure that the hardlinks field in the catalog is set even with the "normal" file inserts.  Please also adjust swissknife_check (catalog verification utility).  The special treatment for reading the hardlinks field (if (hardlinks==0) return 1) can then also be omitted.
See above about return codes
we could also match on number only by adding `?` here : `/(pr|pull)`**`?`**`(-|\/)?(\d+)/i`
Ok. It seemed pretty innocent in this case, but it is simple to remove it.
I ported many of the changes, but I will recheck again.
Yeah, I was writing it just now! I'll add it to the pull request shortly.
I have no idea how to re-insert it. Would you please explain? :[
Style: needs to line up with arg 1.
Can you make this method private and call it only when this class is first loaded?  libpd_init should really be called once and once only.  The other language bindings call it implicitly.
It would help to add a short Javadoc above this and other classes. Also, can you put ExternalBucket and InternalBucket into their own files? Actually I'd suggest making a spark.shuffle package and putting them in there.
Wrong method name, should be removeDuplicitCookies. Also, any reason why not to return ```$this```?
remove references to PBS with SGE
This one is correct, but could you write it as ``dtype=np.int32`` for clarity?
еще лишняя пустая строка
у нас геттеры по традиции пишутся без get
/// ...
а это сделано специально. потому что если во время mouseMove делать сразу arrange and adjust получатся лаги. Там стоит счётчик. на каждое 14 движение - я могу сделать таймер, если надо. через каждые 0.3 секунды
Okay, anyone can tell anything about this pull request? Tell me what I have to fix please. 

/cc @dhh, @tenderlove, @steveklabnik, @sstephenson, @rafaelfranca, @jeremy, @fxn
Ah, I see.  Sorry for the confusion.  I want the second one.  We should just add a class method on the mailer to return the queue.  This I can mix in a module to return the right queue depending on where I want to go.

Something [like this](https://gist.github.com/2981106).
Put a blank line before this comment. I think it make easier to read.
s/must read with/can be read using
`params.permit book: [ { :authors => [ :name ] } ]` works too?
This will just crash on anything but a string literal; even if it isn't intended to be called by anything but, it should issue a reasonable error message if it happens.  It's definitely possible because it's a public method, and there's no way to write a type signature like "LiteralString => ..."
If you want to go for a test drive, I just threw away a few hours of struggling and pushed what I think is the last good-working version.  paulp/topic/unchecked.
Is there one of these for reflect too?
I think that if `b.ndim == 2`, then all we need is for `b.shape[0] == A.shape[0]` and all the matrix stuff below would work.  The resulting `x` is the same size as `b`, whether `b` is a matrix or vector.
I wrote a C-function to do the casting and my timing test is now 4x slower instead of 2x faster.   Of course this is in a loop of 100 calls to the integration routine for a simple function.   

This is really unfortunate.    Let me see if I can play with it some more.
*docstring
The wild stuff in particular is difficult. The old idea of SymPy wilds only work if there is a standard basetype that supports all the operators. This really isn't feasible for a general system. This wild implementation is completely general but creates patterns that can't be manipulated by SymPy (it injects large non-basics into the tree). It is, for example, often impossible to print a pattern.

If anyone would like to comment on how this could be solved I'd welcome ideas.
by -> be
This seems redundant , since you have ``ask(Q.real(expr.exp),assumptions) and ask(Q.real(expr.base),assumptions)`` at the bottom of this method.
Ok, cool. I just wanted to make sure.
Is it possible that `ob_clean` will fail as well?
Tree logger doesn't need to be passed down
None of the tabs are showing up. Hold on..
it's shorter, at least..
Probably a more elegant solution is to clear the selection when the selected item is no longer in the domain (can probably be handled from within the TreeModel or the Tree widget)
No, it isn't boolean.
OK, seems reasonable.  You couldn't just specify a width and height on the green div in the background?
C# also does not execute the conditions after a OR condition that is TRUE, but this wasn't what I meant. If you do int foo; and then Console.WriteLine(foo); you will get a compile error for uninitialized local variable. But in this particular case not... that's why I am surprised...
c/are in/are written to/
You're right, I left it in from a copy paste, will remove and rebase.
Do we need to mark these tests as :unix_only?
Wrong parameter order, optional parameters should come after required parameters.
As I said above, default to 0.
Use sql_query_limit.
Why the == 0 check? If it's a boolean == 0 can probably be omitted.
It was another bugfix needed to get installing and updating working. I can split it into a separate ticket and PR if it really matters.
Cast all variables to integer.
And this should be a no-op because you inserted the user with 'user' as username instead of $username.

You need to have assertions that users are actually created and destroyed. At the very least compare count(*) from users before and after.
Hrm, I didn't understand this change at first but see that it is only used in tests. In a new PR we should add a table of all of the events this class can emit.
fixed in 7642482
i removed the 500 millisecond fudge... since we schedule the check after the timeout or after completion we don't need the offset.
This was the best I could come up with.
This file should not be changed.
rather than adding this file in by hand, might it be wiser to a) include his library (https://github.com/elmasse/Ext.i18n.Bundle, https://github.com/elmasse/Ext.i18n.Bundle-touch) as a git resource, so if updated we can access the updates? b) make a fork of his library, and include that as a git resource, so we can update/modify on our own and take elmasse's updates only if we want
Sounds promising. yes please.
You've inherited this one, but while you're here, would you mind chopping the line length down? Thank you!
return empty string instead of false?
On Thu, Aug 2, 2012 at 12:50 PM, lamblin
<reply@reply.github.com>
wrote:
>>              {
>> -              assert (self->n_output_vars >= self->n_updates);
>> -              Py_DECREF(rval);
>> -              rval = PyList_New(self->n_output_vars);
>> -              for (int i = 0; i < (self->n_output_vars); ++i)
>> -                {
>> -                  Py_ssize_t src = self->output_vars[i];
>> -                  PyObject * item = PyList_GetItem(self->var_value_cells[src], 0);
>> -                  Py_INCREF(item);
>> -                  PyList_SetItem(rval, i, item);
>> -                }
>> +              Py_ssize_t src = self->output_vars[i];
>> +              PyObject * item = PyList_GetItem(self->var_value_cells[src], 0);
>
> I added that check, and 29/36 tests on theano/compile/tests/test_pfunc.py fail, because the compute map contains 0.
> I'm not sure how to find out why and how it happens.
>
On the surface of it, this sounds problematic. Are you sure that you
are checking correctly? The assert should match the way the code
updates the compute_map after evaluating each node. How are you doing
it?
PS I changed the imports in this file
Consider changing the name of batchDiff and removing its arguments -- it can access m_pathRefPlan, m_savePlanPath, and m_numSQL directly.
It's confusing to pass members to methods as arguments that are never passed any other values.
Why doesn't AdHocSP use the same queueSQL/executeSQL pattern as the multi-partition variant? The code here is more direct, but is a whole separate code path that might break independently. We also need the queue/execute pattern working for running adhoc in procs.
Check whether the downloaded diskshare is the same you added in setUp()
good point. I'll add a check.
also assert that all values of the content length header is greater than or equal to 0, list_to_integer accepts negative values, better to catch that as early as possible
Nitpick: There's a mix of single quotes and double quotes in a single statement here. Please choose one.
All these `_` patterns suggest that the grouping of these arguments together might not be the best possible one.

I'm really keen on making progress on this issue now to meet our promised end-of-year release date, so I'm inclined to leave any non-essential work as future TODOs, including this.
Yes, i know.
Before it was like that – 
<pre>
complete: function( jqXHR, status, responseText ) {
…
self.each( callback, [ responseText, status, jqXHR ] );
</pre>

You actually change sequence of arguments for callback function.
But now, always() is already give us arguments in correct order
And what about scrolling a few pixels at first (so toolbar disapears from the screen)... ?
I didn't realize how close your regex was to the one I was originally thinking of adding.
I'm not sure what you're trying to do here. Won't $sub always be a closure reference (and so always true)?
Looks like a typo: You want to do:
`(m1 * w1 / w2  + m2)  * w2 / (w1 + w2)`
I removed the dtelog, which is only useful for debugging purpose.
I would like to close this pull request.  The better place for changes is [this page](http://meteor.com/examples/todos). What about adding
two new section *Make some changes*, *Things to try out* and moving your comments there?
Right, but in the future, they won't be trying to do this with the new system. They're doing it this way now because it was that or us spending a bunch of time adding an API. With the new system, if someone gave us this requirement, we'd build an API of some kind which would return a 400 because that's what APIs should be doing.
We can't do this. Nuking files outside of the installation prefix, especially silently, is a big problem.
We should probably update to 7.6.1, pull this patch and see if the testsuite situation has improved.
Maybe just personal preference but I think `unless positive` is nicer than `if negative` (if that makes sense).
That could be a useful thing to comment in the code I suppose.
It this is loaded from a YAML config file these won't be raw strings, can that give us surprises when trying to escape & expand stuff?
^subreddit's
после "А вот" запятая явно не нужна.
this seems unnecessary since all the code inside is commented out.
We only want to jquery.apply_stylesheet if there is no conflict error from save mode or if the mode is preview.
Shouldn't this be `{self.param: _('the name of an existing wiki page')}`?
a --> the
I think this wants to be flipped with 236
Not sure whether the mv is a good idea - imagine the script aborts somewhere in between, the file will still be moved away
count ?
Can we write these tests in a way that doesn't end up with the lines so long?
This actually changes the outcome in some cases, where the hashCodes were equal, but the names were not.  Hopefully rare, but we should still get this right.
I'd like to control the `this` context and set it to the parent record of the association.  As far as I can tell, this is going to require a bunch more refactoring to make it work but I think it's the most intuitive.  Will discuss IRL
JavaDoc should probably be changed :P
add returns either the component if already existant or the component created. No sense in doing a get on further lines :p.
By the way, did I mention that the spaces are still wrong?
Inadvertent. Fixed.
Also, does delaying filling this in until runtime mean that there's some period where the title bar will blank out (between when the native shell shows its title and when we fill it in the DOM)? That might look a little funny.
hintProviders is initialized with { all: [] }. Then, in the if branch guarded by registerInAllModes, the provider is added for all modeNames, including the modName "all".
based on the above i don't think this can ever be false
Sorry I don't know what proper formatting should be for if statements in the views.
I tried following armaan's example but sorry if they're still wrong
Положено положено, просто пока не написано
Indentation. Please indent for any block (HTML or template).

Also, this form template is not the best one. It is missing for example display of errors not related to any field (general form errors).
what about this:

```ruby
@redcarpet_extensions = {}
@config['redcarpet']['extensions'].each {|e| @redcarpet_extensions[e.to_sym] = true }
```

From my point of view it's easier to understand.
i would be fine with putting this in the restmail module, although in that case it'd require the regex as well as the email address and message count. if that is the case, i also think we should put the restmail module in a place that's easier to import by other projects. (like in Mock, since it is essentially a mock webserver)

i think that the direction the team as a whole is moving in is that they will be using freshly created users for each test. if personatestuser.org doesn't require the step of getting the url out of the email and loading it, then this could go away. in the meantime, everyone who is creating a user for a test needs to do this step, and they are a lot more likely to do it with restmail than try to automate gmail (eek!).

the fact that the user might not be using restmail is why i provided this as a separate method rather than rolling it into some larger process.
Please include a link to the commit (i.e., version from git) we have in our tree
Whitespace after container.
Lint is complaining about the fact that you're not including -o- and -ms- prefixes
This should be in an external .css file so it will a) cache and b) get linted by csslint
Same as https://github.com/mozilla/butter/pull/1316/files#L8R108
Specify a mime type here (can be borrowed from the script tag)
uh, this indentation is off.

Also, I guess a comment wouldn't have hurt here.
Porquê removeste essa linha? Não poderia estar no header do arquivo?
The version of archlinux's default python is 3.x, but many users will create a symlink `/usr/local/bin/python` pointing to `/usr/bin/python2`.

In my first commit py3k compatible problems has not been fixed, so I change this line. I will update it.
You might want to do this after sending the text announcement but before sending the functional announcement -- then have the payload constructor just take in a user and use that to populate the payload nick / userid.
Shouldn't this also mention id when the ID is not "*"?
No big deal, but you can call apply by just adding (). So instead of .apply() just do ().
filename should be read from an attribute of the Page object persisted in the db when the original static file was created.
if args are not passed correctly, you should be printing usage and exit. Currently it would fail with ArrayIndexOutOfBound Exception
Was this intended to be in this commit?  Doesn't appear to be a build issue.
it's already pointed out in the PR's message, "how to properly test it?"
Make this internal.
Paul,

> To me, a higher priority is either error handling or disabling "inferred
> SP" support. Or BOTH error handling AND disabling -- try planning with
> inferredSP and on failure retry without inferred SP -- I THINK we do
> something similar with multi-statement batches. What we need to avoid is
> cases like "SELECT ID, DATA FROM T1 WHERE ID=1 UNION SELECT ID, DATA FROM
> T2 WHERE ID=2" where each SELECT would normally run as a single-partition
> plan and contain a single fragment, but the PartitioningForStatement state
> would indicate that the single-fragment plans are only valid when run each
> on their own partition.
>
 Could you please elaborate a little bit. I am not sure I am following you
there. What is the "inferred SP (Single Partiotion)?

Mike

>
>
Where is this file come from? We cannot use an external file without checking it's license.

If it's a simple shape, you can try to draw it yourself with Inkscape.
Please change "==" with "==="
Do you mean the sub-classes should use this API too? I find it strange to have to pass something (especially e.g. ``lefterror``) that are already available in the class instance. What if the user passes a value that is inconsistent with the content of the instance? The way I tried to do it is to ensure that the information is only passed in one place. The other reason I thought it would be a good idea to do it this way, if we ever need to pass more information for any unforeseen reason (e.g. mask), the propagation methods have access to the entire data object for the two operands. But I need to think about this a bit more.
That's why I used both:

#define INVALID_SOCKET (SOCKET)(~0)
#define SOCKET_ERROR	(-1)
That syscall with that parameter is Linux specific, additionally the PR_SET_NAME checking-part is because it's only available on Linux upwards of 2.6.9.
That line just became unreadable
This looks like a weird test to me. I wouldn't rely on a translated string to test a geometry type.
you have an extra space here.
Instead of making this a dict, I'd prefer to just have skills as a list and `skillpoints` as a key in the top level dict with the total.
It's funny, yesterday I was staring at this exact line for a couple of minutes wondering if I should make a note or not, but I was a bit pressed for time and github's diff view wasn't very helpul here, so I just merged it right away. But now that you mention it: It's a bit awkward to pass both type and subject, especially because subject is present in data as well (this might even lead to strange errors when subject in data and the passed subject differ now that I think about it. Not in the current implementation, but at least theoretically whe looking at the function signature).

About the metadat loader I'm kind of wondering if this should somehow be linked to the rdfMapper. I mean, they both translate between the storage backend and the rdf representation, maybe the mapper should have a getLoader() method. But this needs to be thought out a bit more fully..
The Map implementation should be UnmodifiableMap
Okay, thanks for the update.  Is there a kill or terminate method on Popen objects in Python 2.6 in Windows?

Might be time for me to dig out an old copy of windows xp for testing....
A try block can have multiple except statements (or also an except statement can catch more that one error/exception)

```
In [1]: x = 1

In [2]: try:
   ...:     if x == 1:
   ...:         raise ImportError("oops")
   ...:     if x == 2:
   ...:         raise Exception("eek")
   ...: except ImportError:
   ...:     print "got an Import Error"
   ...: except Exception:
   ...:     print "got an exception"
   ...:     
got an Import Error

In [3]: x = 2

In [4]: try:
    if x == 1:
        raise ImportError("oops")
    if x == 2:
        raise Exception("eek")
except ImportError:
    print "got an Import Error"
except Exception:
    print "got an exception"
   ...:     
got an exception
```
Actually, I have been quite frustrated on numerous occasions by this general shortcoming of the display system.  I think we should revisit the whole `_repr_foo` machinery and consider allowing it to return *containers* of a given mimetype, rather than a single entry.  Once that is done, then it might make sense to include display_data outputs in these reprs.
You can install matjax in your profile dir now. 
Webserver search files first in the `current_profile/static` then in the install dir. 

This allows you to overrite some IPython files on a per profile basis.
If this method won't be used. Should be removed. What was the intention behind it anyway?
This and the other functions bindingly assume that there is a local level, which may or may not be true. It may also not hold that what we've defined as local level is the highest one, as the user might define new bunch of config levels that look up in Redis, LDAP, Riak, SQLite and MongoDB, making it webscale++ and this hard-coded value wrong.
Anything between double brackets will be evaluated as JavaScript.

{a} + {b} == 3 is testing that {a} and {b} are replaced with their values

'U{{{a} * {b}}}' == 'U2' is testing that {a} and {b} are replaced with their values AND that the resulting "2 * 1" string is evaluated as JS. 

Edit: i initially posted in the pull request, moved it here.
It might be a performance hit to touch `absolutify` twice, maybe save it in to a var.
Do we need to support browsers that doesn't support these transforms (I'm assuming older IEs)?
csrf seems to be disabled on the whole site. I tried adding `{{ csrf() }}` to the form and it didn't output anything. So I decided to make sure we knew that it wasn't enabled by adding the decorator. I saw it used in other forms in other apps in bedrock, so I figured this is just how it works for now. A comment I saw somewhere indicated that it was off b/c of the lack of a cache backend, but that doesn't make any sense to me.
Makes sense. Probably start out inactive, then activate since that is what actually happens on the site.
+1. I'll grep for it in the rest of the code.
what about: initial "loadFile" call with URL and params, and each piece of data from then on is sent in a "fileData" event, with the chunk of data, loaded, and total.
Maybe I'm getting old but it appears that this exception is defined in many places.
Be gone with you `EXT`! That has been deprecated too :)

One thing I've just thought of, is that $this->load->vars() is going to potentially override other stuff as they are made global for ALL views - not just the main one.

I know the way you were doing it before was technically working, but relying on "protected" methods for functionality is never good, as I said they shouldnt even work and some odd merge error is the only reason it does work.

SO, solution? Dunno, but we cant rely on the old solution and this one has problems too...
Spent some more time debugging. topnavigation.js has the following:

            $(".topnavigation_trigger_login").live("click", forceShowLogin);


        var forceShowLogin = function(){
            if (qs.get("url") && sakai.api.User.isAnonymous(sakai.data.me)) {
                $("#topnavigation_user_options_login_fields").addClass("topnavigation_force_submenu_display");
                $("#topnavigation_user_options_login_wrapper").addClass("topnavigation_force_submenu_display_title");
                $("#topnavigation_user_options_login_fields_username").focus();
            }
        };

forceShowLogin is also called in doInit(). I removed all of the code like this from the 404.js file, and put breakpoints in the above. It is not getting past the qs.get("url") check, so it is never going into the addClass code. I tried commenting out the if check, and the problem becomes when you load the page it displays the login submenu (since forceShowLogin is called by doInit). If I create another function that only does the 3 lines in the if block, and change the .live() call to use it, it works. What is qs.get("url") for, and is there a way for me to setup the link to utilize this, or is the better alternative to move those 3 lines into a separate function (like I have done locally)?
@com.sk89q.minecraft.util.commands.Console makes it work for both console and players

so, why do you make a separate command?
you're apparently duplicating most code anyway
Do we have a preferred `toString` format? "key: value" or the more traditional "Header[key = key, value = value]"?
To be honest I'm not sure if we need logging on that level as well, that's why I've had the $debugMode variable but even so, I could remove this logging for now.
This is incorrect as your returning an (```Doctrine\DBAL\Statement```?) object not an string
I fotgot. The latest one should not have it.
Indeed. I have removed it.
Is `work_off_delayed_jobs` the exercise in this test? Newlines around it?
FWIW, if the windows installer becomes truly supported, we need a windows build box to do this. otherwise, just keeping the key files secure and in place is a gong show.
Brian, the pushed changes should be better than before.  If you feel strongly about creating/destroying a new thread per deployments persistence unit, lets discuss on IRC.  David mentioned that perhaps we could use a thread pool, so I know that he is open to the idea.  I think it really depends on our needs.  

If we continue to use short lived thread pools in various areas (in AS land), perhaps we could combine some of these pools (so that we don't grow them past an OS limit).
This should be context.revertReloadRequired();
Just an FYI: `path.gsub(/^/, prefix)` is probably better written as `"#{prefix}/#{path}"` or perhaps better `File.join(prefix,path)`
see comment on L406
Should probably remove that comment :)
please, don’t use any arbitrary class name or data attribute unless absolutely necessary.
Why don't use `h1` tag?
Oh, ok. I did not realize. Open an issue if you want. This way, I wont forget it.
@alivedise  +1. I missed to delete it. Now we dont use this var anymore. Removed! And thanks for the tip.
Is exactly the same check than before. Could we refactor in a separate method?
BTW, using currentContact.adr.length without knowing if currentContact.adr exists could create errors. In the previous check you already added it.
On 09/18/2012 05:37 AM, Fabien Cazenave wrote:
>
> In apps/email/locales/email.en-US.properties:
>
> > @@ -102,3 +102,12 @@ message-contact-menu-reply = Reply
> >
> >  folder-list-header=Folders
> >
> > +reply-quoting-wrote={user} wrote
>
> if you want |{user}| to be magically replaced, it should be |{{user}}|.
>

In this case, the goal is basically to provide a string-bundle to the 
back-end so it can re-use the string when replying to messages. The 
back-end does not necessarily have access to mozL10n.  It got angry when 
I used {{}}.  I figured single {} was better than falling back to "#1 
wrote".
You have not fix this function yet.
Never mind, greg clarified this.
Line 406 and line 407 should join?  Maybe my display splits the two lines, if so please ignore.
Please prefix your function names with something appropriate so that they become a little bit more descriptive. Eg. continue/1 could be anything
Either ```bigJoiner```, ```smallJoiner```
or ```bigMode```, ```smallMode```
Requires investigation
Could we get rid of this format change too?
Could break f out into a second parameter list?
I think this should perhaps be called `resolveTag_thd`.

@djmitche ?
It's probably related to the regex on the gsub
```
@theme: blood;
```
should suffice?
Ugh, it was supposed to skip those. It did in in a first version, then I fixed a problem, and now it doesn't anymore ... Will create a new PR.
are you sure that it will work in production mode?
setting to define the distance of swipes
DetectTlsFingerprintFree(tls); on it's own line
Need to add $text into the docblock.
No, I mean subobject. There aren't any classes.
Do you really need an iterator here ? I think plain arrays map / reduce would work just fine here as you iterate over each tab anyway.
Also it won't require non-standard features making it easier for newcomers.
@sawyerh - the table/table-cell allows the content to be vertically centered.
If addressCache is no longer used, it should be removed from the other areas as well.
great approach to make this thing testable.
Does the site TOS/PP show on the upgrade screen? Will this already be displayed from the main authentication or pick email screen? If so, does this need to be here?
Could you wrap these sorts of status messages in a `settings.DEBUG` conditional so we can turn them off in production or when run on a cron easily?
Mistake! There were the original extra spaces... I'm changing them.
This would be better handled by allowed `expand` to take varargs and joining itself. Otherwise we leak out the knowledge of these being directories (which isn't much, but it is something).
Wouldn't it be nice to use vsprintf() here? Would allow for people to easily change the format by simply overriding this class and changing the property containing the string used. An addon would be that this method would become more legible I think.

Opinions?
I think this should be `AcceptedViewModelSelector`
You're right. The meaning of the code is a bit confusing, though.
@prolic I fixed this test in #1109 Can you remove this line? My apologies
previous exception
I wouldn't add these methods. If you need to use message-id, access it via Headers.
What are `$hash` and `$header` for?
I'm sorry, can you give me a bit more to go on? Are you referring to the else? I thought that was correct.
Do you plan on updating this BTW (Asking because you closed pull request) ?
This not exists.
Click on badge button to grab code to embed it on your site!
Wouldn't abbreviate it here...
As with `nom/fresh` I think we should just call this `tie` and users will write `nom/tie`
this looks like a proprietary extension? if so it should be has-bracketed.
Please add an empty phpdoc line between the param and the return, and same between the return and the throws
He he good catch. I'll fix this.
Yeah, I'm guessing this should be in local devs' .gitignore and not in the repo's.
I think soon we will simply remove the external library dependency (since really its only $.each and $.ajax).  

1.5+ is currently needed because $.ajax didn't support the headers attribute until then.  

On Sunday, November 25, 2012 at 2:34 PM, Andrew McCloud wrote:  
> In src/raven.js:
> > @@ -49,6 +49,14 @@ > } > } > > + // XXX: is there a better place to put this? > + if (typeof($) === 'undefined') { > + throw "Unable to find required library (jQuery, Zepto, ender)"; > + } > + if (root.jQuery === $ && $().jquery < '1.5.0') {  
> Why is a version > 1.5.0 needed? Would it be future proof to do some feature detection instead?
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/getsentry/raven-js/pull/35/files#r2217409).  
>  
>  
>
afaik, <= and >= are the only supported operators by LDAP for the modifyTimestamp (eg < doesn't work). check this again and maybe raise an exception if it is an unsupported operator (or log warning and convert eg > to >=; although i prefer exceptions)?
if your code is to sort them alphabetically, windows_live should come after vkontakte
Mark with @Valid, lest validation will not be cascaded.
Two spaces for indentation please.
My sense is that models are probably reusable between the pages, but views not so much. I think we should separate them out.
Calling play here only works if you run the test alone.

If you run it with othe tests, unfortunately, it does not reset state, and the video's time was at 14. Playing from 14'll never hit the range 2-5.

Should change p.play() to p.currentTime( 2 );

Might even be able to get away with calling pause too, as we don't need to be playing to run these tests.
no space between '}' and ')'
I'm not sure how thorough we want to be (anyone?), but the first case would exclude the following case:
Just assume, a vote is getting registered twice for one click of the button, the test would still pass because post_click_count = 2 (for example) and initial_count is 0.
Ah, okay.
I just haven't found many more scripts with this format since there isn't that many of those using context (and considering previous version of this one, which contained "/server/map/blah" as context, I didn't think I should really care about what to put there, as long as it's different for every server.)
Will fix ;)
Is the variable `dist` necessary? would it not be better to do:
```python
return sum(char1 != char2 for char1, char2 in zip(str1, str2))
```
Delete test, dont skip
If you change the db from 
"ActiveRecord::Base.establish_connection(:sfx4_global)" to 
:sfx_global_test as you are suggesting, I think that's probably good 
enough.

Want to hop on IRC this afternoon to discuss some of this stuff 
synchronously?

On 10/10/2012 11:39 AM, Scot Dalton wrote:
> In test/dummy/db/migrate/20120927163304_sfx4_global.rb:
>
>> @@ -0,0 +1,15 @@
>> +# Create a test SFX Global DB.
>> +class Sfx4Global < ActiveRecord::Migration
>> +  def connection
>> +    ActiveRecord::Base.establish_connection(:sfx4_global)
>> +    ActiveRecord::Base.connection.initialize_schema_migrations_table
>> +    ActiveRecord::Base.connection
>> +  end
>> +
>> +  def change
>> +    create_table "KB_OBJECTS", {:id => false} do |t|
>> +      t.integer "OBJECT_ID", :default => 0, :null => false, :limit => 8
>> +    end
>> +    execute "ALTER TABLE KB_OBJECTS ADD PRIMARY KEY (OBJECT_ID);"
>> +  end
>> +end
>
> Yeah, I definitely need to document it. I added a travis flag to the DB
> config which I subsequently forgot to check. I'll check that (maybe
> change it to mock_database) and raise an error if it's not set. These
> migrations are only in the test/dummy app and the db names in that
> database.yml should indicate test (sfxlcl41_test, sfxglb41_test).
> I'll additionally comment the test/dummy/config/database.yml with the
> purpose of these.
>
> Other thoughts on ways to make this less harmful?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/team-umlaut/umlaut/pull/2/files#r1808030>.
>
not required if its a virtual machine
It appears that net is never used.  I have removed the requirement.
we should probably cast strings to regexps here.
Good comments. Should I implement the changes?

On Oct 22, 2012, at 5:48 PM, Scott Devoid wrote:

> In lib/ModelSEED/MS/Factories/SBMLFactory.pm:
> 
> > +use XML::Parser;
> > +use Class::Autouse qw(
> > +    ModelSEED::Auth::Factory
> > +    ModelSEED::Auth
> > +    ModelSEED::MS::Model
> > +    ModelSEED::MS::Biomass
> > +    ModelSEED::MS::Annotation
> > +    ModelSEED::MS::Mapping
> > +);
> > +use Try::Tiny;
> > +use Moose;
> > +use namespace::autoclean;
> > +
> > +has auth => ( is => 'ro', isa => "ModelSEED::Auth", required => 1);
> > +has store => ( is => 'ro', isa => "ModelSEED::Store", required => 1);
> > +has auth_config => ( is => 'ro', isa => 'HashRef', lazy => 1, builder => '_build_auth_config');
> 
> These don't seem to be used. Can they be removed?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
@ZNickq GenericCustomItem#getPacketType
Apenas espaços mesmo.
Why the blockquote? It just needs  a new line before it (get rid of the break tag).
Unnecesary space between parenthesis and the first parameter, please check the code style, we are using Zend code style
Cheers that you did it all the same ;)
i'm adding a cs remark here, before $limit parameter only one space is necessaary. 
Cheers.
I removed the div class="posted" because it's not needed. Let me know if there is a need for that and I can put it back in
Please make this a doxygen comment, with the appropriate info.
See http://paparazzi.github.com/docs/latest/styledoxygen.html
Nope, in this case it is using `select *`, and it is specifying a where clause. The result is that you have a row[0] entry with one column, similar to node-cassandra-client did.
Helenus is returning a column 'tenantId' and we don't necessarily want that in the returned config.
Mostly just thinking out loud, it just didn't feel right as you're actively using the 'optional properties' property. That's probably an indication that it shouldn't be optional.
Modified this to throwing an error
Are you referring to the for-comphrension in the above ``askOption``? If that is the case, the whole expression should go away as soon as symbols are initialized by ``GlobalIndex``, which is something Mirko (@misto) was willing to do (maybe it's not already there?!).

About the complexity, I share your fear, but I don't know of any better way. Mirko, WDYT?
Is the ``listenMode`` argument mandatory now? (it looks that way) If yes, couldn't we have it optional (with a sensible default)
There is no user role I thought?
did mention zeromq
OK, I'll change this if you say so, but I don't see the point in require_once for these files. The Loader already tracks what's been loaded, and we've already verified both files exist. Am I missing some reason for having PHP do the extra work of require_once over include in this scenario?
Замечательный подход, но почему ж тогда не добиться до пользователя правильного введения? И всё-таки правильность передаваемого процента нужно делать именно в методе mutator.
Пока не до конца разобрался с типами в obj-c. Исправлюсь по мере развития :)
Does this return an Array or a String?
很有趣的标注~
De acordo com as best practices RESTful show deve ser GET e não POST
I think we should always default to a global queue, and allow them to pass a scheduler instead. I can't think of any other GCD queues one would pass here, really, besides main (and that can be done with a scheduler).
No problem, I just wanna keep it focused!
lastBuild may be from another branch and don't make senses for revision range. You have to use getLastBuildOfBranch with the current branch
Plan to do that to echo/remote directories?  I change mine to whereever you want to go with the other 2.  For now it's add-server.go and add-client.go in the basedir.
layout and pep8 :)
2012/10/10 Martin Luessi <notifications@github.com>

> In mne/artifacts/ica.py:
>
> > +        if raw._preloaded:
> > +            self.raw_data = raw._data[picks].copy()
> > +        else:
> > +            start = raw.first_samp if start == None else start
> > +            stop = raw.last_samp if stop == None else stop
> > +            self.raw_data = copy(raw[picks, start:stop][0])
> > +
> > +        if noise_cov != None:
> > +            assert self.raw_data.shape[0] == noise_cov.data.shape[0]
> > +            self.pre_whitener, _ = compute_whitener(noise_cov, self.raw.info,
> > +                                                    self.picks)
> > +            del _
> > +
> > +            self.raw_data = np.dot(self.pre_whitener, self.raw_data)
> > +
> > +        elif noise_cov == None:  # use standardization as whitener
>
> hmm.. if (noise_cov != None) is False, you know that noise_cov is None, so
> there cannot be any silly values, no? (I might be wrong, it's early in the
> morning and I haven't had any coffee)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/141/files#r1805705>.
>
>

Ok, I was wrong. But there remains a silly values point. What if we:

is instance(noise_cov, mne.cov.Covariance)

and then below:

elif noise_cov is None:
The above commit really seems to be the problem and afbc21b7b7c65ff6e69cd2a072cfbbb7acc3ef00 doesn't fix it.
why is this statement here?
I have tested that evicting just person works try this:
sessionFactory.getCurrentSession().evict(Context.getPersonService().getPerson(patient.getPatientId()));
Please use `&&` over `and` for conditionals.
Does this mean that if we use feed then we can't use keep alive connections?
We should check http specs for SSE for example to see how to end a stream without closing the connection if possible. Or verify that SSE (or any other open socket protocol) are not compatible with keep alive connections.
This should be, I think:

    And also create a ``HashedResponse`` object extending ``Response``::
no need to use a ``configuration-block`` if you provide only one language in the example
It would lose the meaning though - this is showing that this is equivalent to the ```Request::createFromGlobals()``` method. I'm not that happy with it being formatted like this either, it now takes up too much vertical space even if it does avoid the horizontal scrolling.  Something like:

```
$request = new Request(
    $_GET, $_POST, array(),
    $_COOKIE, $_FILES, $_SERVER
);
```
would avoid both of these but not be ok CS wise.
...it would mean you wouldn't need to use `puts` or the backticks, and would mean it would print the output as it did the work.
good point I didn't noticed the `,`
I've got it `test/unit/**/*.coffee'  works :) I guess it failed the last time because there where old and new tests in there.
Is this idempotent? (Since it will be called every time the Actor is Restarted)
Емнип, это говно нельзя передавать по конст рефу, поэтому я так и писал. Но надо проверить - уже не помню
ahh, very true. i file my comment under premature optimization...
no, but this is something we need to figure out, I don't want this branch to get sucked into the composer website.  And any branch that has a composer.json file automatically gets included in packagist.
pep8: please use a whitespace around the + sign "self._get_prev_sibling_count() + 1"
+1 to checking the behaviour in rhsmcertd.

I think I'm Ok with it being checked after SSLError as long as it is handled in the GUI/CLI.
Sorry to fuss, but I think it'd be better to adjust len down to zero here and let the outer while loop terminate it.  That's less brittle in case anyone else modifies the code.
This technically wont happen anymore since the uuid of the guest is passed at startup. Maybe we should punt w/ some other error message. This brings up the PREPARING race condition again though!
there's one empty line too far here ;)
Indeed, we gain nothing from genericizing the return type of read...
When would this fail to instantiate and not be an error?
@dannygreg Sorry about the state of the doc here. Your good work will drive some improvements, at least. :kissing_heart:

So let's see... The libgit2 `interhunk_lines` value is equivalent to `git diff`'s `--inter-hunk-context=<lines>` setting which is documented as:

> Show the context between diff hunks, up to the specified number of lines, thereby fusing hunks that are close to each other.

With the default `context_lines` setting of 3 and the default of zero for this parameter, the 3 line contexts must overlap for the two successive hunks to be fused. With a value of 1, two hunks would be merged if they have less than 7 lines between them (i.e. they will be merged if the 3 lines context abut).

So, @jspahrsummers is right (as if there was ever any doubt). Perhaps better docs would read something like:

> A hunk represents a series of changed lines along with some stable surrounding context.
>
> `context_lines` controls how many lines of stable context will be included above and below the hunk. There must be at least `context_lines` sequential unchanged lines to mark the start and end of a hunk.
>
> To further clean up diff output, if the outer bounds of successive hunks have fewer than `interhunk_lines` between them, then they will be merged into a single hunk with a longer group of unmodified lines in the middle (e.g. if the 3 `context_lines` of two hunks would overlap, then the hunks will be merged with an internal run of 4 or 5 unmodified lines).

Or more succinctly:

> `context_lines` is the number of unchanged lines that define the boundary of a hunk (and to display around it)
>
> `interhunk_lines` is the maximum number of unchanged lines between hunk boundaries before the hunks will be merged
Put a space after the `,`.
if we add a indent here this page will remain the same
@gsilver Can you fix the 2 spaces in here?
Please add the license header.
it's now using the same raw query for aggregating in postgres
OK, understood.  I don't think this code is going to survive long-term (maybe the worker threads need to choose from a pool of sessions?), so I can live with it for now.
We jump to err in paths where pool_name has yet to be initialized, it needs to be initialized before that.
All this is just to save the minecraft.jar data to the file
It isn't clear to me at all what remoteID has in it, it seems like it might be more suitable as a struct, since it seems to contain multiple things.
I think you could write the real @robot name here
Gibt es **EBEN NICHT**? Oh Java.
you left out a semicolon
I think I see where you were going now. What is the default for this?