labofsahil/github-event-dataset-2012 huggingface dataset:
you are right. fixing in 4... 3... =)
how do we ensure that if the parent also needs to generate its id that its generated before?
Could you clarify this comment to be clear on weather you mean [romaine](http://www.youtube.com/watch?v=YrGeX-29AhE) or just cabbage?
It not "not the best", it stinks. I am aware of it, but still looking for a solution. For some weird reason, ios-orientationchange.min.js doesn't work with the path variable alone.
Effectivement, on est quand même dans un cas ou history est un model sans model défini c'est déjà pas idéal. Et c'est une extension du NotesModel.
Faudrait au minimum une méthode de copy/transfert.
The current gem for activeuuid (0.1.1) doesn't include this fix, so people have to pull the source to get the fix.
oyunadi.jpg olarak gelmiyor bilgin olsun direk orjinalresimadi.jpg olarak kaliyor
А это зачем удалено?
We tried that, but can't find the version of clojure io on clojars, and it wasn't clearly indicated in the github page, so did a quick and dirty fix
Right. The "end" callback had the old signature (without time stamps). And
definitely want the target (from the first event) sent to all three. So
might as well merge it. I'll review it in more depth tomorrow and compare
with what I have here. I could have swore there was a typo in there that
had the wrong name for one of the time stamp properties.

Thanks again!

On Sun, Jan 15, 2012 at 6:40 PM, Paul Connolly <
reply@reply.github.com
> wrote:

> Aha now I remember, I never intended for this to be pulled!  Was just a
> branch for some stuff I was doing, but I'll initiate it now. The fixes
> where the time stamps not actually being defined for some touch types, and
> the initial target issue. Have a look though first before pulling though.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/gibletto/My-Library/commit/f11ea1fe7f729a649cac003c482cae4e4f16f950#commitcomment-869446
>
Make this an optional parameter

~~~
FLAGS.AddFlag('s', 'single_album', 'Only upload this single album (by name)', '')
~~~
I agree that it sounds quite longish but we're still in the process of discussing proper engine namespacing so this can change in near future.
@davedash: I needed this change to get uploads to work. Is this ok? How should I fix it?
suundo?
Structural types removed in latest commit. I'm having another think about case restrictions.
I'am geting this error, also sickberad crashes

01:03:39 INFO::SEARCHQUEUE-MANUAL-SEARCH :: Downloading episode from magnet:?xt=urn:btih:045d98c68d78e80b8ca0553b4c150f461208258d&dn=xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx&tr=udp://tracker.openbittorrent.com:80&tr=udp://tracker.publicbt.com:80&tr=udp://tracker.ccc.de:80
terminate called after throwing an instance of 'std::runtime_error'
  what():  locale::facet::_S_create_c_locale name not valid
Aborted
This class is not included in this repository, where can it be acquired? I did not find any source for it yet.

I checked out your work before and was able to succesfully run it, but sadly this change broke it for me :(
Strange behavior:

1. Type Bel
2. Press down or up
than I cannot continue editing anymore.
@toddparker - It was on everything, but Garann fixed it with pull request #3344
My thoughts too (also, figured the overhead lots of single word calls for a table would be a pain); just wanted to check on your preferences.
It's a single line comment. I don't see how's that any overhead in the compression.
why not retrieve this setting from WP itself?
naja und bei cata hast du das nette  treetree problem. der typ der fake cata versionen raushaut und irgwie auch so top beliebt ist :O
Since some users may run this as a cron job, it would make sense to have an easy flag to retry (with the default being to do so).

Many times D2L is just slow and times out.
rename to setTimeStep.
"const int& value" is overkill (and confusing), "int" is enough (a copy of a int is as expensive as copying a pointer)
Awesome!
but the parent of this commit is not the last one, I've merged it with what I had pushed after your pull, rebased and sent again to my fork.
Mind a last look?

I like you change very much, but I still plan to separate the DocumentBuilder in two separate roles as we had discussed.. that will take care of the _propertiesMetadata_ indexes which I was fearing to touch for this fix (thanks for that!)
typo: stopped
Let's leave it for now then...we may want to move these functions in the future anyways. I'm starting to think these should be moved to PersistentList and PersistentHashMap, and renamed...
Sure. Without this cleanups, use of fcntl locking cause panic, see ASSERT(vp->v_filocks == NULL) in vn_free(). I guess no one tried to lock /etc/dfs/sharetab before, but I tried for some reason I don't even remember :)
You should set up a current_user thing for this -- I thought we did that at the coffee shop the other day?

Then, you could do current_user.admin? 

Check out the Railscast about authentication for a guide about how to do that.
How about doing something like this...

helper_method  :title

private
  def title
    @title = 'Accounts'
  end

this method could also be added to base controllers
Versão do git,
Understood. With your vision of this part it looks like this class should be much more abstracted than it is now. Anyways, the situation as it is now presents a bug to the end users and many of them have posted on the forum that the panel just pops up, whether they want it or not. This will at least make the functionality of the panel work as intended.
I haven't tried it yet, but I have some review notes for you.

I like the direction.
Sorry about that.  Try it now.
Ah, right. I forgot to fix that after running .toUpperCase(). Please start an issue about it :)
A ras lfoula ta pas bouge la fonction sort men l controller !!!
In order to get the dependency tree correct, this was necessary. It is in the generic overlay so both SDC and SmartOS use it
很多地方都有这个问题，其实在使用的时候应该把类型放到meta里面，这样才能确保万无一失
但这样是不是会丢掉js灵活？

我这点还在纠结，包括，在做其它的一些操作的时候，到底是严格一点呢（性能有限，使用受限，不容易引起错误），还是非常自由呢？
@dkharrat This is from quite a while ago... have you tried using the most recent AFNetworking?
глупый (мотивчик)
Not a big fan of this. It's too light ;)
All of the deleted tests weren't testing the scenario described by the test name. They were side effects of the method call. In order to test the stated assertion is true these were unnecessary. Furthermore, since now we're not running through the entire board over and over they're not correct anyway.
I think this is the wrong place for this addition. All of the "SetVimXxxOption" functions extract a parameter and operate on it, but this operates on a whole modeline. I'd stick it down around L113.
W000t! Thanks!
----- Original message -----
> On the 20newsgroup dataset, the estimator
> object for a `LinearSVC` with L1 penalty went from 9.5M to 180K.

That's really cool. Definitely usefull.
 
> This just gave me an idea: one could create a smart CSR matrix
> implementation that would uncompress rows on demand (possibly combined
> with mmap to load the compressed rows).

You can have a look at the carray package, that does this and can be usefull to work on really big data. We cannot depend on it in the scikir, but we can try and make the scikit work as well as possible with it.
This is imported here to keep the dependency from blowing up pipeline users who don't need this compressor.
A check for COM interfaces is missing.

In the longer term we should probably switch to a templated opEquals
to support opEquals in Interfaces.

http://d.puremagic.com/issues/show_bug.cgi?id=5459
Wertebereiche müssten eventuell noch angepasst werden...
haha I think you used the exact same commit message a few months ago ;)
I sink the messages to the file because—as you'll see below—we cat the file back out to generate the commit message.
``` php
<?php

$consumer = $this
    ->getContainer()
    ->get(sprintf('old_sound_rabbit_mq.%s_consumer', $input->getArgument('name')))
;

```
apple key + , -> source code -> code formatting -> text file or csharp
Look here https://github.com/MechanisM/django-image-helper/commit/f989affa146e8f3a6e40942e245f178020e25f63 might be helpful.
See what I mean? Where the hell does "login_forms" come from? It's not like it's significantly shorter than "accounts.forms"; It's almost certainly known as just plain "forms" somewhere else; and if "auth_views" is anything to go by, it should be called "accounts_forms" not "login_forms" - except that "account_forms" would be no shorter at all than "accounts.forms"

In fact, if this was a simple "import accounts.forms" this line would be about 20 characters shorter; so calling this by a "short" name has actually made the file *longer* that it would otherwise by.

*really ending the tanty now, I swear*
Pra que que você tem um projeto todo em Rails, vazio, só com um html estático que redireciona para um site no about.me?
:D
this broke installation
You seem to have created this branch from master and not my development branch. Vertex color import is already working with the alpha feature list. You seem to be going down the same route I went down, on my second iteration using setattrib. Using int() was not really required in the previous scripts and wont work at all with the new api as it store float values. 
Here is the stuff related to your commit using the seperate color & alpha layer  - https://github.com/neomonkeus/blender_nif_scripts/blob/development/scripts/addons/io_scene_nif/import_nif.py#L2175
Hello!
How do u undervolt RAM?
If you're just directly instantiating this, why not make it reusable and have the backoff value parameterized?

That avoids the appearance of 2401 elsewhere in the code, which is fragile.

(One could also say "why not have a ResponseBuilder parameter?", or otherwise parameterize your mocks, which would allow you to reduce some code duplication.)
Der Codestyle ist klasse ^^.
Der Code wurde schön mit der autofunktion in vc++ formatiert -> beschwerden gehen an microsoft.
Also added min-width for body and reorganized the footer for a more responsive layout
can you remove it ,since it has been tested
Hm, I wonder If you can get the page surrounded by ♥
I agree, it seems that it should be a success, but I also don't want to break a otherwise consistent behavior in the OS
NOPE...
What does "native access" mean? Accessing the `ValidatorFactory` instead of a `Validator`?
<label> mismatch =/
Yes I can definitely vouch for that after yesterday's experience. Containers all the way


------Original Message------
From: Juliano Schroeder
To: Martin Balmaceda
Subject: Re: [GameDevGroup] added tile matrix to world class (c43e05b)
Sent: Jan 12, 2012 15:55

Dude, arrays are evil!! http://www.parashift.com/c++-faq-lite/containers.html#faq-34.1
=D

--- 
Reply to this email directly or view it on GitHub:
https://github.com/AbsolutelySmokinSoftware/GameDevGroup/commit/c43e05be7b511626f6f5d9f1e055d134510ceb0a#commitcomment-860912


Sent via my BlackBerry from Vodacom - let your email find you!
Or have I completely misunderstood the purpose of device-vendor-blobs ?
Anyone who knows, please share it to me, because it's really frustrating
для nodeToJson()
Done. Thanks for the hint.
If this isn't needed, can it be removed?
I agree that `ReadySet` is the correct account to pick in that situation, but that logic should be contained within the `find` method.
We should never have to set inline styles like this.  Why can't/didn't you do this in the .css?  I will merge this but would like to see you fix it in another commit and PR.
It used to be documented that we used this for everything but log.  If that got reverted in our documentation, we should revert that :-)
I'd prefer this function to look like this:

```python
def _aos_to_soa(self, data):
    if self.soa:
        return data.T.ravel().reshape(data.T.shape)
    return data
```
More generic event can be triggered from RTE.tools.insertElement - this one is used by gallery and polls editor as well
That's The Line :)
I saw that the differences in delays was by design, good.
This config property is a duration.
should be `30 s`
default case here?
Added check now.
Generally better to use string's Equals or Compare with StringComparison.*IgnoreCase
There's no decorator, so does the method simply fail when mpl is not available?
os.path.is_dir would be more specific
We have to put it somewhere.  It's either expose the seek handler in some way (owned by the app) or expose the 2 functions we need from it.
Eh, why is it using abort?
You're right, there probably should be. I added that, and in doing so also checked if that protection ever was triggered, and it wasn't.

So yes, it should be there, but no it doesn't matter.
This is comparing a tuple and a float, result will be true until Pymongo 3.*.
The whole first argument to eunit:test/2.
This is a good idea. Something similar is already done by the `wheelEvent`, by disabling anchoring and by adjusting manually the center using `centerOn`.
May be I can extract and reuse this bit of code when dealing with a pinch gesture event.

As a side note, I see that `setTransformationAnchor` accept `QGraphicsView::AnchorUnderMouse`.
May be we can just switch to this mode while scaling (wheel or pinch), and return to `QGraphicsView::AnchorViewCenter` when the zoom finish.

Anyway, I will play with this a little and see what gives the user the best experience, do not expect me to push this change in the next couple of hours :)
Borrar esta "f" XD
Ok, I realized that the supertype defines the getters for the common fields, @BeanProperty was only needed for the cause, for which I added a getter now.
I had real pain here.
What exactly is happening here? What does this empty case statement even do?
No space before the colon?
Feel free to drop the extra whitespace around `ZCtx`.
Sorry, I typo'ed. I meant `sysconf(_SC_NGROUPS_MAX)`.
But split out the calculation into a `size_t used = sizeof(data_) - offset_`. DRY.
Wait, does that mean you are fixing it? You commented on your own pull request. :-)
Absolutely psyched to pull in this change and take it out for a... spin...
Print statement should be removed.
Created an OpenLayers pullrequest  - https://github.com/openlayers/openlayers/pull/776
`JSON.stringify` can fail in case of circular references in the object. Since params data comes from the engine, we can't assume that there are no circular references. One work-around is to pass a replacer function to `stringify`. Here is an example.

    var seen = [];
    JSON.stringify(data, function (k, o) {
        if(typeof o === "object") {
            if(-1 !== seen.indexOf(o)) return '[Circular-' + k + ']';
                seen.push(o);
            }
            return o;
        }, 0);
файлики из ресурсов грузить лучше через ":/filename". путь к файлу как в ресурсе указано (у вас, видимо, просто будет ":/settingsDefaultValues")
Нет.
У дерева есть метод takeChildren, возвращает список
Дебажил:
int c1 = item->takeChildren().count();    = 3
int c2 = item->childCount(); = 0
В этот список детей кладется еще  что-то, поэтому foreach не покатит.
Whitespaces :scissors:
Or use `regex === text`.
fix punctuation :smile:
Thanks for the comment.
I was thinking what is the proper test name if MySQL default sql_mode would be to `STRICT_ALL_TABLES`. but it may be too much worrying about future.

I like your idea. `test_mysql_strict_mode_disabled_dont_override_global_sql_mode` shows its purpose correctly. I'll update my commit.
Possible to rename, but it's public API, so you'd need to deprecate the old constant.
I think we should. The HTML5 spec is pretty clear about the value sanitization algorithm for each field.
I think it would be faster.... Unless I'm mistaken and `.seq` isn't generating a new collection...
can you get away with calling _logpdf so we don't have to go through all the generic checking again
This change seems unrelated. Also, you've effectively changed it to `/^[a-zA-Z0-9_]+$/`, which means that event names can start with a number (which is bad).
`self.date` defaults to today … is this intended?
http://www.python.org/dev/peps/pep-0008/

*While sometimes it's okay to put an if/for/while with a small body on the same line...*
The difficulty with applying an operator to a density matrix is that it has to be applied to the bra and ket in the outer product form.  It really looks like this:

`rho -> O*rho*Dagger(O)`

This code handles that.  I am not sure how we would teach `qapply` to do this.
use elif on 2nd test. Also, is it possible that it could be neither (e.g. a coefficient that is vanilla-symbolic)?
Why are you sympifying `other` twice?
We make changes to gitignore that are shared - ignore this on your branch will keep it not ignored in the master.
In the interest of keeping the README concise, I'm wondering if the code block is necessary at all. The instructions seem pretty self-explanatory.
I believe this should be `(@stack_usage...num_locals)` if you want to support multiple locals in a code block.
Don't really need this line. The name will change anyway when we change the name to Lists.
Does the `domain` parameter refer to the space path?
You don't need to tell people that the description is detailed.

"MongoDB host" is not a term that appears in the Manual. Use :program:`mongod` (I think) in these instances.
Yes, the main change is switching to section logic rather than plain ID lookup. I could write a view test, but it would be somewhat redundant. We know ID generation works, because there are tests for that. So, I'm just testing at this level in the code because it's simpler - ie. *if* an ID were present, does this work.
Personnellement, je n'ai jamais aimé les formulaires en ligne pour contacter une compagnie. Le message que ça envoi c'est: *on ne veut pas vous donner notre courriel parce qu'on veut pas être dérangé*. Pourquoi ne pas afficher notre adresse courriel dans un élément `a` comme ça:

    Contact us at: <a href="mailto:info@ooda.ca">info@ooda.ca</a>

Va voir cette [page](http://www.bullbear.ca/about) et clique sur le lien de l'adresse courriel.
We can remove this debug message
its actually greater IE7 not 8 ;)
Vars at top of function
Yes it was. There was a missing tab, so I added one so all these lines lined up nicely.
use the fireOnce misc function here?
should be function(test, asserts), and the assert calls in the block should call the asserts library functions.
Nice!   Something like this makes me want to get that test suite working soon!
w00t w00t!
I figured there was something along those lines -- just didn't know what it was called! Thanks!
It's horrible because we're letting browser inconsistencies leak into our API. There's no easy way to avoid this.
What is `this.translationData` good for? Maybe you could add a comment?
Good point.  I'll try fleshing this out.
Exposes the extensions for the view engine from the ViewEngineStartupContext in case view engine components need them. The extensions could be fetched from the actual IViewEngine, but this way all the information is in one place (and the ViewEngineStartupContext had the extensions anyway, they just weren't exposed).
Eventually we need to refactor this function....reusing a lot of code. I will most likely have Shivam do this as this is his code.
Should abstract out 2000 into a variable. Should be a variable in Util.js that is used to update most of the UI components (updating other titles, updating stores, etc).

Make a variable in the top of resources/scripts/Util.js called updateUITime, set it to 2000. Also make a function in Util.js that returns updateUITime, and use that function here.
You could have hit the eof and not have a string of length 0. e.g. read() may have managed to get 3  of the requested 4 bytes. Regardless, let struct worry about whether you've got enough bytes. If you haven't it'll raise ```struct.error: unpack requires a string argument of length 4``` or similar.
@pelson Anything that's not the *actual list* in the other cube.
Tuple = goodness.
Here the terms are theano variable? You can remove the useless function nonempty_sum and replace it by:

reduce(lambda x,y: a+y, [a, a])
self.purchase_date and self.deprecation_kind
``min-width``
Resest -> Reset
constants should be defined before properties
Yeah, I'd pull it out of dispatcher just to avoid confusion.
Instead of not using it here can you remove this argument entirely? I'll merge when that's done. Thanks!
I'd normally agree, but `parse` could also be generalized to be variadic and parse multiple objects like `scanf` or `istream::operator>>`, as in `parse(s, Int, String, Int) : Int, String, Int`, and it seems more natural to put the output types after the input stream in that case.
Yes it is.  What fix would you propose?
It's due to the way AnyType objects can currently be constructed. Currently (for the PostgreSQL port) there is one template constructor, and it will eventually be missing an implementation for `TypeTraits<Reference<...> >`. One could improve by using SFINAE (i.e., prevent type `Reference<T>` from being instantiated and force a conversion to `T` instead) or changes in the design, but so far I have not given it priority.
It could be, but what is the benefit of having (essentially) a global variable if it can be avoided?
the recoveredQuestVar function
Do you mean if an app developer intentionally changes the value of `Accounts._loginButtons.loginServices`? I don't think that's a real concern. Similarly, someone can also do something like: `Accounts._loginButtons.loginServices = "WON'T WORK"`, if they choose to shoot themselves in the foot.
MD may have done. I would have thought that VS would ignore any tags it doesn't understand.
Suggest this be called `:head` or something, we eventually might want to record the formula's blob hash as well; useful for saying "hey, the formula you used to build this is different from what we were shipping at [head revision].
Does this require x11 or just libpng?
Looks like `--disable-debug` isn't a supported option with its configure script.
New style:
`depends_on "metis" if build.include? 'with-metis'`
Begging to be a one-liner.
I assume these are debug puts? ;)
Yup! So I was just teasing you that the comment is now no longer accurate. ;)
LOL! Been writing faaar too much javascript of late!
A la limite, supprime le du dépot celui là, il sert à rien

git rm  data/GUI/Theme/projet.xml.old
component.id and component.version are not URI encoded
Erwartet man bei filterDirs() nicht eher, dass man nur die Inhalte der Ordner bekommt, die dem pattern entsprechen?
Also angenommen es gibt noch zusätzlich den Ordner "ordner" mit der Datei "datei.txt". Wenn ich nun sage filterDirs('dir*'), dann will ich doch weder den "ordner" haben, noch die Datei da drin.
Well, ideally instead of testing that an option is assigned internally, I would check that it actually times out.  Since that's impossible to 'really' do, I think the next best thing would be to check if the client (net/http etc) receives the expected option.
Admittedly, it may just mean that "create at inject binding" shouldn't be in linker.  I was tempted to go there, but this change was already huge, and I didn't want to fundamentally re-design linker here.
If we stop using the word `root` to mean something that isn't necessarily a root, then we can refer to the top-most linker in the chain as a `root`.
Unintentional. I should use 0x1 since I don't intend to imply that these flags have a 0-255 range.
I think it isn't important because I have tested it with a / and without: Both had worked with Zikula 1.3.3 but if you think a relative path is better I or you can change this.
I agree that "base directory" is probably clearer, though "Clear base directory" is a little scary.
Again a lot of duplicated code. You might even be able to handle writing to binary and text files with a single helper function.
Should probably cache `@constructor` too, I recall it being a bit slow to lookup in some browsers.
I thought of that too and I found a number of ways to do it on stack overflow, but it wasn't simple so I went for the current solution since it's stop gap until we have performance reporting
Style nit: we don't use singe line `if` statements. The `return;` should be on a separate line.

I'm surprised JSLint didn't complain about this line :-)
So it turns out I can't change the ``#about-build-number`` handling since that's populated dynamically based on what's currently on disk. That can change if you run brackets and then later decide to do a git pull. I've left that untouched but I did factor out the ``{{VERSION}}`` string into ``{{ABOUT_TEXT_LINE1}}``
ok, i have removed that code
Oh nice
that comment only applies if it's not something you're going to address in the short term
Yes, case._id's
This is just switching to the `semi-1_14-wl` branch, right? Then you would just want to put `:branch "semi-1_14-wl"` in the recipe instead of adding a `git checkout` command to the build process.
Tried doing this, but I think that would have to be in a sessions functional test no?
I tried writing one but couldnt get it working. 
These tests make sure that the create_with_omniauth and update_with_omniauth methods are working.
I agree it would be good to have additional methods to make sure the callbacks are being sent to the right place, but tried for the past hour and half to write one unsuccessfully. If I have time later I'll try again but at least we have these unit tests.
You going to write unit tests for your stuff?
This should be in coaster, not here. The env variable may have `production` instead of `prod`. Don't try to track the options here.
hasjob shouldn't need it's own app as a parameter. It should pick up `app` as a module-level variable.
It may be nice to explain a bit more what it intends to do.
Just remove this middleware, don't comment it out. (Especially without the explanation why are you commenting this out?)
Still this is not good. If you convert to date then only dates (and not hours, minutes, ...) are compared.
Remove this on fire?

closeButton.addEventListener( "click", function clickHandler( e ){
this.removeEventListener( "click", clickHandler, false);
...
Indent this by one
I tried playing around with `trackeventrequested` but kept hitting walls. It seemed like the more logical approach. I can give it another go though
don't know if were going to follow any style guide or not, but whitespace after the `i++`
This will always be true, and doesn't look right
Wrong CSS Property. `border-radius` is standard now.
Even if prefixed, then it's `-moz` & not `moz` - also, this will make it not work except for Firefox.
ah yeah, was just looking at this, and thought it would be more consistent, but you're correct that it just reports that you scheduled an update, not whether it actually updated the table.
Is there a reason not to use the `redirect_to` RSpec matcher here?  The generally accepted good practice with RSpec is to use the most specific matcher available.
Why are we tracking `ssl_version` separately both in `HTTPSConnectionPool` and `VerifiedHTTPSConnection`? Shouldn't it be one or the other?
The announcement of the nick change (e.g. "slifty is now known as dschultz") is an IrcMessageOutPayload
The actual functional nick change (e.g. updating the user object on the client / updating the alias list) should be its own type of payload -- I would suggest calling it an IrcNickOutPayload.  It shouldn't require a message at all, but rather you should be creating it the way you would create and send a "join" payload (var nickOut = new payloads.IrcNickOutPayload;   nickOut.userId = user.id; nickOut.nick = nick)
dso failed to find position (%i)
The idea was that we'd populate this once at the beginning of postprocessing, then we'd never edit it again. That way we'd save some time (instead of populating it every time we postprocess some job) and we'd also make sure that the standardized names are consistent throughout all the jobs, once we start using standardized names.
GoalS ?
rtl kon va no-js ro hazf kon
Thanks!

I see he now added the 1.0.4 tag and even a newer 1.1 one.
At first, lines 25 and 30 are the ones that might possibly change from file to file, so the 19 and 20 could be run somewhere else. Anyway I don't think this makes the code ugly or slow, as this script is called only for the given platform you chose on froxlor. :)
Perhaps.  Assigning to undefined always feels weird and deleting the properties has always been the standard.
I don't like it either... I needed to check if exactly one of the providers resolved the given hub name. Will change that to Count() instead and compare to 1.
"Assuming you're on the default..." what?

Should this also be a function call to get the value, a la `fits.EXTENSION_NAME_CASE_SENSITIVE()`?
@iguananaut - FYI, the change I just pushed up modifies this function a bit (it also had a sign bug) so you'll want to rebase if you've already started working on this.
What I'm thinking is, you might get here sometimes after the nodes have been stopped and fail your test. So coordinating between the main process and this might be good to avoid a sporadic failing test. What do you think?
It should check that a relationship exists at least.
That would use the set proxy, so no need to check if it's a GUI option or comman-line option as it IS the current one, sounds good to me. But stilly my IPv6 question :D. Is Qt ready for it?
There is no FAR in /usr/include/sqlext.h
Then shouldn't we remove them everywhere?
okay, I got it
tested ok on Tom's mac

> In core/src/script/CGXP/plugins/GetFeature.js:
>
> > +                }, this);
> > +            }
> > +        }, this);
> > +
> > +        return {
> > +            internalLayers: internalLayers,
> > +            externalLayers: externalLayers,
> > +            unqueriedLayers: unqueriedLayers
> > +        };
> > +    },
> > +
> > +    getMessage: function() {
> > +        var tpl = new Ext.Template(this.queryResultMessage);
> > +        var key = 'CTRL';
> > +        if (Ext.isMac) {
> > +            key = 'META';
>
>     also I dont understand what you mean by "This also must be taken
>     into account in the control. This doesn't seem to be the case
>     currently." can you explain ?
>
> You should use this:
>
> |box: {
>      keyMask: Ext.isMac ? OpenLayers.Handler.MOD_META :
>          OpenLayers.Handler.MOD_CTRL
> }
> |
>
> in the |handlerOptions| passed to the |GetFeature| control.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/camptocamp/cgxp/pull/308/files#r2447265>.
>


-- 
Camptocamp SA
Oliver Christen
PSE A
CH-1015 Lausanne

www.camptocamp.com

+41 21 619 10 23 (direct)
+41 21 619 10 10 (centrale)
+41 21 619 10 00 (fax)
Indentation too far to the left again, for lines 152 - 154.
here you did $proxyNamespace see? So I would want no $proxyNs, it is misleading, we want readability. I know you have said that Ns is standard but i would still respectfully dispute it
Not sure I understand that comment TBH, $filters[?] can only be one of what we defined.  Could put an else for the if on 49 such that if a $_GET['filter'] was set but not in our valid array that it gets unset and filter is set to array()
all this ifs looks ugly, how about a dict?
item_id : method

then you just catch keyerrors for methods which haven't been implemented yet
Yes. lol. I'll add this to my next pull request.
It's the 'smart' quotes on `‘nosuchlibrary’` in the error message that is evidently messing it up.

Ultimately, this is a bug in rpy2: it should be safe to call `unicode()` on an exception, as we are doing. I've [filed an issue about it](https://github.com/ipython/ipython/pull/1780#discussion_r911934). But at the same time, IPython should protect against that failure, so I've [opened an issue for us](https://github.com/ipython/ipython/issues/1827) as well. I'll be sure to get it fixed before the next release, so don't worry about the extra exception.
Can anyone see a simple way to resolve this, or are we going to leave it for now and accept that quotation marks don't work in the same way as the shell?
There should only be one `env` instance that is created by the application object and used by the different handlers.
Marking my code review progress.
Thanks. What do I use to run the tests? nosetests complains if I write "ip = get_ipython()".
mke/make
sure.  I have no problem with that.  Is this ready for merge, then?

On Tue, Mar 13, 2012 at 10:26 AM, Aled Sage
<reply@reply.github.com>
wrote:
>> +      private Boolean allEULAsAccepted;
>> +      private String transferFormat;
>> +
>> +      /**
>> +       * @see InstantiateOvfParams#isAllEULAsAccepted()
>> +       */
>> +      public Builder isAllEULAsAccepted(Boolean allEULAsAccepted) {
>> +         this.allEULAsAccepted = allEULAsAccepted;
>> +         return this;
>> +      }
>> +
>> +      /**
>> +       * @see InstantiateOvfParams#isAllEULAsAccepted()
>> +       */
>> +      public Builder allEULAsAccepted() {
>> +         this.allEULAsAccepted = Boolean.TRUE;
>
> Changing this properly involves changing the super-type ParamsType, and thus its 13 sub-types. That includes code already in master, so I'd prefer to do those refactors in a separate pull request that cleans up all (?) the builders in one go. Is that ok with you, to take this code into master and then fix it subsequently?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jclouds/jclouds/pull/431/files#r552824
Please revert this change. I'd rather sharejs still uses my version of node-browserchannel.
Based on the original test ( this is one converted from the xpcshell test ) this is just a check to see if it throws. 
See original here: https://github.com/lightsofapollo/ical.js/blob/master/test/xpcshell/test_parser.js#L289
lgtm.

Few more unit tests

1. if there is context override on the partial

>partial:ctx name="joe" 

and ctx also has name ( the expected behaviour ?) can we add a unit test for this.
`@returns`?
Are redirect responses cached by Zeus?
These could be merged into one file.
I still don't understand. Is this specifically about the logging module? Can you show me what the import lines should look like here?
In the instructions that create settings_local.py, there is already a REDIS_BACKEND section, so this is redundant.
That was a library we used for sphinx search. It is gone now so maybe the doc string can be updated.
What if someone uses ``'report'`` as the value of ``$dante_range_type`` here? I think you should verify if it's ``'build'`` or even simply pass the value without checking it.
Why drop stderr? We can capture it as a separate channel using the `subprocess.Popen` function.
Thank you for that! It helps.
What is the point of this method actually? It does nothing compared to its parent class. Right? Can we delete it?
Improve the docblock with params and return value (see parent docblock)
Style: Split this and the line below into two lines?
Why can't you use the exception types instead of Map.class? If you want a 'don't know' type, use Object.class? Map.class is weird middle ground.
best would be to use a ``@deprecated`` annotation
My mistake (think we were in RouteCollection here)
As Larry suggested on the mailing list, let's forget about output_format and text_file_path for now.
Docs out of date.
Sure, it's done!
Yeah, this doesn't need to be private. :+1:
Please use PEP8 style multi line statements without backslashes. either using parenthesis or by using the parenthesis of function calls to start a new line.
Since the `replaceIn` functionality is tested here, can the [replace_test.js](https://github.com/pangratz/ember.js/blob/master/packages/ember-views/tests/views/view/replace_in_test.js) be deleted?
This file is indented using tabs
added a new classname so we can change the icon anytime we want @mecathcart you can change the sync button ifyou dont like that little graphic with a central server and team members...
Looks like a copy paste error with the blank templare text
I don't see InfinispanLogger or -Messages using 10221. They seem to be using ids in the range 10290-99 and  10280-83 respectively?
`bind(this, request)` or using `evt.target` both looks good to me. As long as we use the same method for both callbacks :)
ha! thanks.
class=hidden on the body is weird, but we do it for our localization, so that we don't show default strings and then have the UI flash when the strings get localized, so please put that back
Yeah, some king of inherited code... :-)
I've just realized that the airplane mode does not have any UI representing it's state.
remove the redundant line feeds.
It could be too late but why do we need to prefix this variable with a _ ?
Sigh. I don't like this but it seems we don't have any other way to do it ...
I don't think you're using artist handle and album handle, so I think you can remove those.  Then you could rename searchTitleHandle to searchEnumeration or something without "Title" in it.
Maybe we could leave a comment :), like:

font-size: 2.26rem; /*-moz-calc(10 * 0.226rem)*/

Just an idea, so far the performance is not that bad, but the sooner we do the better ;), if you planning to change this once the value is finally decided I'm ok with it.

Thanks!
Couldn't we relatively easy check this with the FPS counter? IIRC the GIF was a little slower than the transform when I last checked. Also, the transform can be moved off main thread which means it doesn't stall even if the browser is busy.
Should we check if contact.length > 0 also here?
see above ^
Is this not a change in behaviour?
```
>> foo = {:foo => 'foo'}
>> bar = {:foo => 'bar'}
>> foo = bar.merge foo
 => {:foo=>"foo"}
>> foo = {:foo => 'foo'}
>> bar = {:foo => 'bar'}
>> foo.merge! bar
 => {:foo=>"bar"}
```
Again, you remove the "fix this" flag, but I don't see one change that actually fixes anything.
I think it'd be nice if users of this library didn't even see the yaml support if they don't have the pyYaml library available.
Seems much cleaner to isolate this change to the merge case.  Moot point without a reproduction.
Changed to def.
nice contravariance!
Ah, I did not notice that in the screenshot, nice.
Lets keep the current row top and bottom border colors.  This helps distinguish rows easier.
Tried cloneNode() but it didn't work out as I wanted. probably because I overlooked something.
Pull these constants up to the top of the file.
Parts of this method are repeated over different classes. Maybe something to refactor and move into one dedicated place?
You used to have to right click on Anathema.app, Show Package Contents, then browse to `Contents/Resources/Java/English` to read the changelog, custom charm creation info, etc.  Removed those files from this location.
I got following msg when tgtd service not started on RHEL/Fedora host. Is it safe to check cmd's output only?
```
# tgtadm --lld iscsi --mode target --op show
tgtadm: can't send the request to the tgt daemon, Transport endpoint is not connected
```
should this be include in import part? It seems kind of reduplicate.
`s/notrbe/not be/`
One of my recent favorite functions is partition. The body of your for loop can be replaced by something like:

    _, found, branch = row.partition('\trefs/heads/')
    if found:
        branches.append(branch)
when you extract the loops body into a private helper method you could do this, without the variable variable.

replace the ifs' body with:

``` php
// handle array packages
if (is_array($candidate)) {
   $candidateName = $candidate['name'];
   $candidateVersion = $candidate['version'];   
   $provides = $this->buildLinkFromCandidate($candidate, 'provides');
   $replaces = $this->buildLinkFromCandidate($candidate, 'replaces');
} // ...
```

and add the following helper

``` php
private function buildLinkFromCandidate($candidate, $linkType)
{
   $candidateName = $candidate['name'];
   $candidateVersion = $candidate['version'];   
   $linkVal = isset($candidate[rtrim($linkType, 's')]) ? $candidate[rtrim($linkType, 's')] : array();
   foreach ($linkVal as $target => $constraintDef) {
      if ('self.version' === $constraintDef) {
	     $parsedConstraint = $this->versionParser->parseConstraints($candidateVersion);
      } else {
	     $parsedConstraint = $this->versionParser->parseConstraints($constraintDef);
      }
	  return new Link($candidateName, $target, $parsedConstraint, $linkType, $constraintDef);
   }
}
```
Oks. I'll work in that direction.
no i didn't, unless you asked...
interestingly the code using bitfield is over 500bytes smaller in my setup
* 29307 uint8_t :3
* 29819 uint8_t
May be you should check not only unit's ID, but also protocol's one?
Adds `config` so you can configure from the constructor **optional**
Why you use old syntax?
These gems and `bootstrap-sass` should be in assets group. 
Also, I don't see any need to specify strict gem versions.
I did these changes for clarity purposes. The variables `transformed-pose-stamped' and `point-stamped-msg' were necessary for debug output and may be important for that in the future again.
After being able to understand the whole `defer` function, and understand the main usage of these two methods, I think `resolvedPromise` and `rejectedPromise` would make more sense.
You are clearly returning promises in both cases, but it sounds really weird to use `promise` for such special case (immediatly resolved promises) so your initial proposal make sense, but it becomes clear when you:

* use `promise` word for both functions,
* add a second term to distinguish them,
* use past participle that makes it clear what they really are.

Current names are okish,  but they are hard to understand in `defer`.
BTW original implementation details also we're not quite compatible with E10S as messages had to be send in sync.
I'm thinking that we should use `"error: " + require('self').name + ": An exception occurred."` instead, and below in similar cases.
It was useful while bootstrapping the loader when access to other modules is still not available. I could either copy code or export it from loader. I favored this option instead.
@wbamberg will most likely rewrite this through: https://bugzilla.mozilla.org/show_bug.cgi?id=787035
Yep, this should be more modular. Currently each module specifies if it writes to the database or not, so it seems we can build a map from that. But each module must be read and loaded, so maybe it would be better to explicitly declare that info somewhere?
The result is correct but the comments are not. Missed the first .write which was overwritten by GC: 6 line.
But if we don't handle an error, the file is also wiped clean. Together with every other file the provider is handling (if it is possible to specifiy different targets, e.g. cron)

To be more specific: The error is then handled in transaction.rb

    def prefetch(provider_class, resources)
      type_name = provider_class.resource_type.name
      return if @prefetched_providers[type_name][provider_class.name]
      Puppet.debug "Prefetching #{provider_class.name} resources for #{type_name}"
      begin
        provider_class.prefetch(resources)
      rescue => detail
        puts detail.backtrace if Puppet[:trace]
        Puppet.err "Could not prefetch #{type_name} provider '#{provider_class.name}': #{detail}"
      end
      @prefetched_providers[type_name][provider_class.name] = true
    end

Because puppet currently ignores the fact that prefetching fails, the parsedfile provider will work with an empty `@records` hash. With this change we at least populate the `@records` hash with the targets we can retrieve.
I think we can omit this final paragraph on new phases. Even though it's possible, it's not really a convention and it'll probably be best not to encourage this practice.
give some name like tmp("readilnk")
They are not inside of another `spanx` element so they do.
I agree. But, what if in the future, someone else uses something `hidden`? They would have a hard time discovering why their item keeps appearing.
If you loop over all the `hiddenElements`, you don't need recursion.
Why this blank line is necessary?
I'll fix that now. A question regarding the [import statement conventions](http://framework.zend.com/wiki/display/ZFDEV2/Coding+Standards#CodingStandards-ImportStatements), though, which state:

> Import statements MUST be declared once, at the top of the file, following the namespace declaration. The use statement MUST be issued once, with one class (and optional alias) per line, separated by commas and terminated on the last import by a semicolon; class names MUST be aligned.

I took at look at some of the recent classes elsewhere in ZF2 (DI and Form components) and `use` statements are on their own lines. Are the conventions simply outdated in that respect?
We should be able to specify the format. I can't ask my users (that are French) to enter a date in a format that we don't really use (here we use day/month/year).
Uhm if there's no method to call, you should remove the UiHandlers from this widget.
```
.popup button {
  padding: 5px;
  font-size: 14px;
}
```
;)
Nitpick, but `//edit an item` -> `// edit an item`
change happens here
Just test with DOM fragments:

```
    var empty    = $('<div class="empty"></div>');
    var nonempty = $('<div class="nonempty"><span></span></div>')
```
The logic we have in composer is:

- If it matches [a-f0-9]{40} (git/hg hash) => show 7 first chars
- Else show entire string

This might be better done in a twig helper.

Also you can just display it for dev packages but not for others. I think that's the best way.
No, the mapField must raise an exception on duplicated field ...
I know that's what you said yesterday, meant to ask you about that today since I believe heroku won't work unless it's `true`
Yes, Windows to UNIX as my IDE is set to do, both are handled so this should be fine.
Ok. But user using PostgreSQL driver will not be able to modify the RevisionSequenceName if you set the value.
I agree with the reasoning of this patch, but why ``shutil.copy`` is outside the try/except block? ``shutil.copy`` will fail if ``os.makedirs`` fails.
I don't know why you changed `assets_env` to `assetsenv` but doing this you must also change lines 164/165 in `__init__.py` which use `assets_env`
Is a nested `if` preferable to using `and`?
add comment: # check for use of os.putenv and os.environ adjustsments
I saw it expanded in the branding image, so I assumed it had to be spelled out.  I've changed everything back to JBoss AS 7..
(this variable name is fine)
この空行消してください
ah, good call. r+
That's out of the scope of this bug
Could this be
```python
eq_('{0} {1}'.format(user.first_name, user.last_name),
    pq(rnv.content)('#profile-info h2').text(),
    'Should be redirected to a user with the right name')
```
The dash isn't displaying correctly because it was changed to a smart em or en dash. It should be replaced with "-" or "--" to ensure that it displays correctly.
spec expects camelcase `MyValue` but now, snakecase is fine  too..
What happens if you say "Cancel" on the confirm dialog? The form isn't submitted but what else happens?
Don't we need a redirect or something?
Yep, good idea, will do ;)
Does this log message work? IIUC, this will get forwarded through the ExtensionLogger, and a ExecutionLogger.I18n instance will be created that uses `StringUtil.createString(String,Object...)` to format the string. However, the `StringUtil` method expects the variables to contain integers (based upon its regular expression), doesn't it?
According to the [README|https://github.com/RiotGames/berkshelf/blob/master/README.md#running-unit-rspec-and-acceptance-cucumber-tests], you should copy `spec/knife.rb.sample` to `spec/knife.rb` and adjust appropriately to run the specs, so `spec/knife.rb` would be present, right?
Makes sense, I will adjust the tests to reflect this.
Normally C++ programmers would use mySurface_->getWidth(),  but I think that looks confusing due to the underscore. The (*x). instead of x-> does the same thing.
wow! unused code? 
Great!!! less lines here :)
No problem. I changed it back.
I suggest to rename that function to "setServiceInformation".

It should just call:        this.$.service.setServiceInformation(inData);

This will help adding new services like dropbox, ...
Same as above, this should be moved into a utility fn.
Something is wrong on your environment.  Remove this so I can apply this patch
Updated it to be 

$this-> instead of static::