labofsahil/github-event-dataset-2012 huggingface dataset:
According to Arne, the right way to do this is to QI the pDist object to IWebBrowser2 rather than using the m_pWebBrowser member. See e.g. https://github.com/salsita/aji-ie/blob/libprothandlers/libsalsitaextension/HtmlToolbarWindow.cpp#L22
I kind of don't want to support /projects/:id since we don't need it
What do you think of Request? I'm not a big fan a base...
The previous code here was clearly wrong, so I've updated the change to reflect the actual intent.
addColumn is correct
I am not following, CDN url can be changed in ~/.katello-disconnected/config.yml and with katello-disconnected setup command (see the options I think its --cdnurl). This is just a sane default when you dont set it explicitly.
typo in nows
I decided to leave this at 1 but just include the individual pages from the developer docs in this TOC, otherwise the single top-level bullet point was a bit lonely too, and it was redundant to have the intermediate index page anyway.
This is nitpicky, but is it necessary to put an extra newline here?  Usually if a docstring will fit all on one line, I keep it that way.  Does the extra newline do anything in the documentation?
I think the use of WIDTH was probably deliberate - given that we are talking about tables, the term COLUMN might be confusing.
alright, then I'm satisfied there's no reason to change it unless we later change something in the parsing (and only maybe then)
Routines like this would also be much more readable with spaces between the operators. That may sound like nitpicking but it really does make a big difference in readability.
PEP8 (spaces after commas)
i'm fine with backslashes
I tried removing that, but the editor just puts it back in. I'll try hand-editing and see what happens...but I'd like to avoid a situation where people have to manually edit the file if they touch it.
Why the indenting change? And is this part always necessary?
OK, but you could at least remove the variable I guess.
I would just make it a public method on the ModelIntegrationTest class.
We should use GeoExt localistaion
Wouldn't it be better if the variable is called $type here for clarity's sake?
Check the discussion in the companion pull-request, I'd prefer to keep the handling logic up to the dev.
Well... I guess no, because if one condition is satisfied the second doesn't need to be evaluated... the expression wouldn't needed to be evaluated (saving some performance)

but that brought me attention the expression is wrong.. I will simplify it
There should be a space after a comma (`(args, filename)`), to be consistent.
`read_py_file` when it's for local files (Unix-style paths seem to work as URLs, but Windows paths don't). `read_py_file` is also slightly more efficient.
this should be a test dep, right?
AFAIk the major change here was the formatting from the correct spacing (3) to the wrong one (4) :) please revert!
Nitpick: `None` is better than an empty string for empty fields.
Again, this will break if the page name changes, and seems a bit premature anyway.
Don't split the  'We are mozilla' string, depending on the grammar rules for the language,  the image could be at the end, the beginning or inside the sentence, the image tag should be a variable in the string.
I don't understand this. I thought we group all the stdlib stuff together, then group things by library. What should the import lines look like?
ahh, yeah.  same bug.  Should be created_by in both cases.
These are the ones to run a full Gogo shell for the main (), will factor
them out. Thanks!
On Apr 29, 2012 3:34 PM, "Michael Hunger" <
reply@reply.github.com>
wrote:

> > -        test(player, 13);
> > +        test(player, 19);
> > +    }
> > +
> > +    public Option[] testOptions( )
> > +    {
> > +        String gogoVersion = "0.8.0";
> > +        Option[] options = options(
> > +            repository("https://oss.sonatype.org/content/groups/ops4j/
> "),
> > +            cleanCaches(),
> > +            frameworkStartLevel( 10),
> > +            mavenBundle().groupId( "org.apache.geronimo.specs"
> ).artifactId( "geronimo-jta_1.1_spec" ).version( GERONIMO_JTA_VERSION ),
> > +            wrappedBundle( mavenBundle().groupId( "org.apache.lucene"
> ).artifactId( "lucene-core" ).version( "3.5.0" )),
> > +            wrappedBundle( mavenBundle().groupId( "org.neo4j"
> ).artifactId( "neo4j-kernel" ).version( NEO4J_VERSION )),
> > +            wrappedBundle( mavenBundle().groupId( "org.neo4j"
> ).artifactId( "neo4j-lucene-index" ).version( NEO4J_VERSION )),
> > +//
>  mavenBundle().groupId("org.ops4j.pax.logging").artifactId("pax-logging-api").version("1.6.1"),
>
> why are these commented out?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/neo4j/community/pull/474/files#r748940
>
:+1: I have same problem
I see no other solution than injecting the generator too in the RouterListener. As this PR removes the RequestContextAwareInterface from the RequestMatcher (which makes total sense, don't you think?), it means when using a RequestMatcher it cannot initialize the context for the generator.
So we need to inject the generator, which would also remove the confusing and interdependent implementation.
``php
foreach ($keys as $key) {
    $this->delete($key);
}
```
You should re-add the original authors here too.
Does this test will work correct and this assertion will executed?
Hehe oops :) not the first and properly not the last grammar error I make.
To be honest, `getFlashBag()` is only named as such because it conflicts with a method kept for BC.  While this is a bag, it's also not really a bag in the sense of being optional. I agree that maybe `Metadata` is more explicit but `meta` does actually mean the same thing, `abstract information` which is what meta-data is.  I would feel more inclined to remove Bag suffix in all references to this particular bag and just call it `Metadata`.
Just pushed some more withValue blocks. Error output is slightly more specific now (was previously reporting line x column 1, now reporting line x column y).
Style: avoid one-letter methods/variables when possible. `c` becomes `config` here.
Not positive, but I think `@event.groups?` might work.
happy to change it to package. In fact, the whole paragraph could be made better if you have a suggestion :-)
typo, but i think resolving if successful is implicit and redundant. maybe just talk about what it resolves with?
mysql 已经被作者放弃维护了，rails 3建议使用mysql2
makes sure page preview urls in admin are constructed compatible to what multilingual.py does...
Yay!!! Unit tests!!!
Just a minor note - IMO it would be much more precise for this test to catch the EJBException and then make sure the enclosed exception is indeed a IllegalStateException.
This should avoid going through the driver manager
AIUI, this will detect a problem starting the DeploymentUnitPhaseService, but not any problems starting any services *started by* that service. Which would include the DUPS for the next phase, and transitively any services started by that DUPS. So if 3 phases down there is a failure, e.g. a missing dependency for some service added 3 phases later because a datasource the app injects isn't registered in JNDI, that won't be detected.

That's what management ops do. The OperationContext tracks all service installations and removals that take place in the context of the operation. (The OperationContext provides a custom ServiceTarget to track service installations; removals are done via the OperationContext API directly.)
If you check master, both ClusteringImplMessages and InfinispanLogger are using message IDs starting with 10280.
Previously ClusteringApiLogger/Messages also contained a bunch of messages that should have been defined in ClusteringImplLogger/Messages.  So, to avoid mixing ranges in different modules, it seemed best to do some minor message ID shuffling.
Given that we don't have any message translations yet, why is this such a big deal?
You're right, it works without this sleep. I'll remove it. However, another two Thread.sleep down in the code have to be there, because synchronization is only partial.

----- Original Message -----
From: "Brian Stansberry" <reply@reply.github.com>
To: "Ondrej Lukas" <olukas@redhat.com>
Sent: Thursday, August 2, 2012 3:15:33 AM
Subject: Re: [jboss-as] JBPAPP-9506 [7.1] regression test (#2780)

> +            op.get(OP_ADDR).add(SECURITY_REALM, "JBossTest");
> +            op.get(OPERATION_HEADERS).get(ALLOW_RESOURCE_SERVICE_RESTART).set(true);
> +            updates.add(op);
> +
> +            // set up new security realm
> +            op = new ModelNode();
> +            op.get(OP).set(ADD);
> +            op.get(OP_ADDR).add(CORE_SERVICE, MANAGEMENT);
> +            op.get(OP_ADDR).add(SECURITY_REALM, "JBossTest");
> +            op.get(OP_ADDR).add(AUTHENTICATION, JAAS);
> +            op.get(NAME).set(getSecurityDomainName());            
> +            updates.add(op);
> +
> +            Utils.applyUpdates(updates, managementClient.getControllerClient());
> +            
> +            Thread.sleep(3000);

Why? The updates should be applied synchronously.

---
Reply to this email directly or view it on GitHub:
https://github.com/jbossas/jboss-as/pull/2780/files#r1289403
It is still there, I remember David explaining this few weeks back...
See below comments about type
Ok - had a good go at the method - this is a lot neater. Still need better names surrounding the lambda, but hopefully it's readable.

We've now got some quite powerful methods to do with scope: we can find the scope for a position, we can traverse and only change a particular scope, and we can choose whether to enter an inner scope based on a block.

We could go super-functional at this point and generate the transform method from a two 'strategy' lambdas, but I'm not sure how to do recursive lambda generation in Ruby... :) Perhaps we'd be better off sticking plain strategy composites to handle the scoping.

Any reason not to merge this into master soon?
Airplane mode should turn off

- Radio ('ril.radio.disabled'`)
- Data ('ril.data.disabled'`)
- Wifi
- Bluetooth
- Geolocation

It should also save the status of the latter 4 items so when leaving the airplane mode we could know which one to turn on.
What about using the html5 hidden attribute?
`unread` sounds likes a Boolean. Use `unreadCount` instead.

(not-blocking comment)
hmm, conceptually I like that idea but the nit here is more that we may want to load real views with the DI functionality but because we set `jsBase` in helper.js all tests now have that as the default. I would prefer a setup/teardown in the tests that actually need the modified path rather then mutate the default in all cases.
This sorting was needed when `concat` pending messages + messages from Gecko. Currently we have to remove this code.
It will if I bind it to "this", and yeah that looks cleaner
Yep, you are right, the problem is that we don't know if more headers have been added.
Another option is to create a 'refresh' method and call it from outside every time a new header is created/removed.
Makes sense for me.
Everything here is also exposed as a global (tests run as top level files). If this is what you intend (to expose this content to other tests) I recommend creating a helper file then including that like you do your other implementation files.
Add/update the test for this the modfy_account_test is now failing
two space indent is enough
encoding issue? (Modalità)
there is already a variable named _ to point to L10N API, thus this is not needed.
Yep! totally right. I'll fix it
How do you feel about using %s instead of string concatenation?
Maybe we should create `utils` module for general purpose functions like this one?
As I commented before, `id="title"` should be `class="hide-element"` if you want to make this heading and the legends hide. 

In this case you where you have the legends below you'll probably need more modifications to get this to work right...
we should encapsulate such logic in the model e.g:

```Track.random_track```
I'm suss on what typeset CDPATH will actually do. Ill try to wire up a test
harness for major versions of bash and zsh today.
On 14 Dec 2012 03:23, "Michal Papis" <notifications@github.com> wrote:

> In scripts/functions/support:
>
> > @@ -128,6 +128,18 @@ __rvm_sed_i()
> >    fi
> >  }
> >
> > +# Drop in cd which _desotn't_ respect cdpath
> > +__rvm_cd()
> > +{
> > +    typeset old_cdpath ret
>
> try simplified code:
>
> typeset CDPATH
> CDPATH="."
> chpwd_functions="" builtin cd "$@" || return $?
>
> this works well with IFS - we have this trick in few places in code
> already and for now nobody complained
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/wayneeseguin/rvm/pull/1383/files#r2409269>.
>
>
Not an orchestration class. field name should be changed, as well as setter method. This will also cause the property to change on cxf-servlet.
app-manager branch has constants needed. Will fix when they are all on the same branch.
Sorry. I see what you're asking now. For the checkbox in the general tab, the widget is an actual QCheckBox while the others aren't(they are some kind of group widget which happens to include a checkbox). Therefore they do not behave identically (or similar in any way really).
returning false even when the delegate sets the matadata ? really ?
@zml2008 
I didn't make those protected because I though they were worth overriding, I did that because it would enable them to be called in children classes. That way, if I have a third registery process that must be called after methodRegistration and nestedClassRegistration, or in-between ( whatever ), I could be able to still call those two without having to rewrite them...
Won't this make lines one character longer than they should be if there are no spaces in the line?
I'm honestly not sure why nobody ever wrote this class.  The backend work was mostly good to go, it just needed a tool action to add the Equipment Database to the toolbar.
I haven't checked, but you might be able to increase performance by using x*x instead of x**2
Since `bool([])` means `False`, the above 4 lines could change to
`return bool(re.findall("%s$" % self.target, node, re.M))`
or just:
`return bool(re.search("%s$" % self.target, node, re.M))` (re.search method returns None if no matched string found.)
This should be master branch.
Yikes, good point!
So, it isn't this way be default (for security purposes), but the
repository of the sourcestamp (which this will feed) is often used for
the url passed to the source step. This will only work if the repository
is not system local (except in the case where things are mirrored
locally). I agree that it often isn't possible to detect this
automatically, but it should be straightforward to provide a
configuration option to do it.

It might be reasonable to *also* have a way of getting the local url,
but I think that most people would find that a poor default.
Before 5.2, the community war used to be named "drools-guvnor.war". Since then, the are for example called "guvnor-distribution-wars-5.5.0.Beta1-jboss-as-7.0.war" (which should probably be renamed to "guvnor-5.5.0.Beta1-jboss-as-7.0.war"):
https://repository.jboss.org/nexus/content/groups/public/org/drools/guvnor-distribution-wars/5.5.0.Beta1/

Anyhow, mind if we replace "drools-guvnor" with a docbook variable in all these url's so it can automatically fill in the correct version?
Well, both project_url and the tag are coming from external parameters right? So why the interpolation can't happen outside exdoc?
'Source' is the first one here, that's not self-explanatory. A quick description in parentheses could be decent. "Sources (for team inbox messages)".
Can thd ever be null here?  Should just be an assert(thd) instead?
add a utility function for checking active so that if we change the implementation, we can change it all in one place:

if (xcore_fn_check()) ...
i'm going to switch this to a regular ole Popen when I get back to my laptop, forgot check_output was introduced in 2.7
If this only unclaims a single row, it should be called `unclaim_row` not `unclaim_rows`.

But really it should be called `complete_row`, because it doesn't reverse the effect of "claim" (which is what "unclaim" implies), it moves it to yet a different state, complete.
Regarding coding style for comparisons... We should also change all of those instances and make use of the triple operator to check for the type too and kill the currently necessary type conversions. That means in this case use `!==`.
Oh, cool, I didn't know it could take a block like that. I need to familiarize myself with RSpec a lot more...
Ahh I see. I think we should track it. That way if we add parameters or
modify the names, we will still get the changes. If you want, you can
commit the parameters you use and I will stash a set of parameters for
myself.
On Oct 20, 2012 5:51 AM, "schasins" <notifications@github.com> wrote:

> In scripts/common/params.js:
>
> > @@ -1,106 +0,0 @@
> > -/* -*- Mode: Java; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
>
> I just stopped tracking it since I figure we're always going to have
> different settings. We can keep tracking if you prefer.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sbarman/webscript/pull/4/files#r1897143>.
>
>
Mark the test as Incomplete. We are using Skipped when a third party dependency cannot be satisfied.
Same remark, in this case 0 is not an empty name (I mean, the index 0), but is considered as empty with empty function.
CS - move the opening brace to the next line, please.
An alternative to throwing the InvalidArgumentException would be to allow the ElementInterface, but use default values here for the checked, unchecked, and hidden element values if the element is not of type CheckboxElement.
We usually do that... but it's not required, super.onBind() does nothing
See the comment about string.split() on line 30
This is pretty useless on the server. Maybe a fire and forget background thread? Do you need to know when it's done?
Old habit (for when the "ul" would actually be changed inside the foreach loop)... I fixed it in both places.
This is still referenced by ZfcUser/Mapper/Exception/RuntimeException.php
why are you unsetting the var?
Hi @franek!
I understand that is possible to set revision_sequence_name from config.yml; but to maintain the cross-platform feature should be possible to use the bundle without setting this parameter, unless the user needs to change its default value.
This is really weird. You validate the connection when connect_confirm is *false*
Idiomatic R would be

```
if (!missing(...)) {
  stop(...)
}
```

but to make it parallel the other checks, I'd default it to `NULL` too.
http://mandrivausers.org/index.php?/topic/10771-cl-10-basic-shell-scripting/

"The wonderous semicolon ( ; ) - Like many programming languages, BASH can use the semicolon to separate statements (NOTE: The semicolon is only needed in BASH if you have more than one statement/command on the same line, otherwise it is optional, unlike many other languages where a semicolon is needed to separate every statement.)."
I'd rather merge a human's contributions than some stinking bot ;)

This is on my TODO list, I'll get to it soon. Thanks!
I dislike redefining the name of the function within the function. I know it's OK because this would never be called recursively, but still...
Maybe wrap to avoid too long lines?
Yeah, I was finding the privious function annoying to debug, I needed to constantly change it to this to add breakpoints, logs, etc, and felt this was easier. I left it, because I felt it was more explicit and easier to read. I am not really attached to it, though, so totally have no problems reverting.
I would find this fairly unexpected. If my program calls a function named `removeTrackEvent` and the track event still exists, I would likely report it as a bug.
remove this end bracket kthx
Soundtouch is not really involved, I just forgot to change branch... While testing I found out that soundtouch doesn't handle videos with sound so I enforced audio only in the type (see just below).
can you explain what this file does, what it's for?
I used pry to debug a failing test, so it makes sense for me to have in the testing env, but no problem to remove it
Does this really look better than a bullet list once formatted by a Markdown tool?
I think adding a color to this output would help make it nicer. :blue should be good IMO.
with static import:

assertEquals(queueResponse.getPrepareResponse().getStatus(), ActionResponse.STATUS.OK);

Is much better. Also, if it fails, the error message tells you more.
We want users to provide a value for it. Assignation is just to provide a default value.
We can not use final here :S
Parens can (and should) be omitted from functions with no parameters.
лучше не разделять методы одного класса встроеными типами.
Okay, I am going to play a bit of devil's advocate here: what value does this factory bring?

I do somewhat understand the point, but as I see it now there is no direct benefit to separating this code out. If there's a unit test written as well, that would be direct benefit -- but that also implies that the test CAN'T be written on the existing controller. Unless that's true, it seems to me like this is extra code for extra code's sake. YAGNI, unless you have concrete code that proves you do.
I am contemplating not doing email verification altogether.  I think we want people to get up and going as quickly as possible, and avoiding the trip to check their email will help us with that.
Github seems to be wrong here. There is a newline for me.
```ruby
@string_attributes ||= attributes.select {|a| /string|text/ === a.type.to_s}.uniq
```

:D
It looks like this globally exposed function does no checks as to whether the current user is allowed to update the tenant info ?
This can be collapsed to `Cassandra.runQuery(q.query, q.params, callback);`
Also, in line comments should begin with a capital letter. I've made the changes required for this.
`rows[0].count <= 2` I believe this will need to be, to account for principal_id hanging in there as a column.
What happens when I only supply one of the `identification_columns` to `#fill_in_with`? Does it still work? Does it raise an error?
some of these comments are misleading/wrong

* i'm not sure but useacl isn't important for an auth plug-in.
* disableactions should also be irrelevant due to "cando[useradd]"
* autopasswd, authtype and passcrypt are irrelevant for the plug-in.
perhaps the method should be called `confirmation_message` not `notification_confirmation` or at least `notification_confirmation_message`
I am even more confused why you are including these. They are low level socket headers for unix systems. Given that you don't need these block of includes at all, you probably don't need the "config.h" include near the top.
Ik zou dit hernoemen naar `EntityHealthBar` zoals ik verderop suggereer. Met de aanpassingen die ik aanbeveel word je health bar onafhankelijk van de Entity die hij volgt.
Some new dependencies I see here - what do we need so that devs and build server can run these tests?
Needs to be:

    if (strlen($extra)) // strlen() really isn't useful here though
    {
        $extra = ' '.$extra;
    }
Wouldn't those be better with `$this->set_header()` instead?
не, если короче, то не сработает :))
Возможно. Я исходил из того, что более чем АTGC не будет, поэтому массив решил не трогать.
Справедливости ради нужно сказать, что весь мой код не будет работать в случае если 100 перестанет быть 100, а станет 150. Я даже хотел вынести это значение в настойку, но не являюсь сторонником самовольного додумывания разработчиком ТЗ. В нем нигде не сказано про то, что количество молекул в ДНК может быть отлично от 100. Вообще если бы там было бы упомянуто о том, что количество может меняться, скорее всего алгоритмы обработки были бы другими.
К тому же, вы вычисляете неверно случайно значение индекса для замены: m меньше, чем размерность DNA, таким образом, ячейки с индексом выше m никогда не поменяются.
But there is a typo here ;)
In the context of Rails (so both sidekiq and raven being require'd by the big Bundler import) are there load ordering issues to worry about here?
Hi Elias,

Thanks for this idea and code. I'll pull it in once you're happy about the CMUCL / SBCL idea that Stas brought up.


On Sep 3, 2012, at 10:01 PM, Elias Mårtenson <notifications@github.com> wrote:

> In dev/cmucl.lisp:
> 
> > \ No newline at end of file
> > +	      ext:*environment-list*)))
> > +
> > +(defun %exit (code)
> > +  (unless (zerop code)
> > +    (error "CMUCL does not support exit codes."))
> > +  (ext:quit))
> Should :RECLESSLY-P be T here?
> 
> Also, when googling I found a reference to UNIX:UNIX-EXIT here which does support exit codes. Should it be used instead? http://comments.gmane.org/gmane.lisp.cmucl.devel/10946
> 
> I haven't been able to install any version of CMUCL so I haven't been able to test this.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 

--
Gary Warren King, metabang.com 
Cell: (413) 559 8738
Fax: (206) 338-4052
gwkkwg on Skype * garethsan on AIM * gwking on twitter
I think this is easier to read:

```ruby
def edit
  if params[:step]
    step = session[:topic_step] = params[:step]
    send "edit_#{step}"
  else
    redirect_to edit_topic_step_url(@topic, step: steps.first)
  end
end
```
Again a leftover, will change it.
fix it + the mysterious titles param and I'll merge
smaller or equal to wsize/2 (otherwise the default value would wrong)
Just define:

    def confirmable_attribute
      send(self.class.confirmable_attribute)
    end

And friends.
You're right. Put it in an else
It's forcasting, i'll move it down.
updated description, and added `long_desc` :point_down:
You shouldn't include the above 2 lines
Register an Extension as a Service
same issue: it should be on line 234
Isn't this a regression?
`auto_reload` is now passed two times: here and also by **kw.

Also, users can now set it as `jinja.auto_reload` and `templates_reload`, while at the same time all other renderers rely on `pyramid.templates_reload`.

If we decide to support multiple ways to configure this setting and keep backwards incompatibility, we need to handle both settings and raise conflict when they have different values. Maybe a word from @mcdonc if we should switch to `pyramid.reload_templates` instead of `jinja2.auto_reload` (from current `reload_templates).
I recognize that these are "fields" on the data structure being passed in, but did not realize there was an attempt being made to maintain consistent data structures between various (possibly unrelated) functions, especially those in an external API.  This requires a much more intimate familiarity with the extended code base and the inference of patterns and the designer's intent.

This is a benefit of OOP, in which objects have explicitly defined structure, enforced by the compiler/interpreter.  Design patterns are much more apparent in OOP languages.  Free form data structures that rely on implicit conventions are inherently prone to these types of API inconsistencies, especially in an environment with multiple contributors.
Please revert, p4string::jsonencode is deprecated, use ['Core']['Serializer'] instead
nit: pull out some of these parameters to local vars to make this line easier to parse
yea booooy
Looks fine! :class does sound better
Btw, commons reminds me of Apache Commons or JBoss Commons, both really dump grounds...
Ah sorry, one sec, you're adding these cache managers directly to the cacheManagers collection in each test. A bit off but that should be fine, assuming tests pass :)
This cast is unnecessary, since `void *` is implicitly convertible to any C type.
Yep, with the current impl. also, if the guesser is not in the guessing_order, he will not be called.
Actually, I have found it to be better to end lists and dictionaries like this with the last element having a comma and putting the closing brace or bracket on its own line.  This makes for fewer merge conflicts as it is easier for patch to handle any edits to such a list of things.  Of course, that is merely my own opinion.
Hah, too much Javascript coding the last days. Thanks for spotting that.
Please see: https://github.com/mozautomation/mozmill/pull/45/files#diff-16

It has been renamed and updated. That why it is shown as new.
Whups?
In-line with @jds2501's note above, can we word this as the recently received message?
This should probably be a choice parameters, allowing 32 or 64 bit regardless of your current platform.  The default can come from mozinfo. (E.g. parser.add_option("--bits", choices=(32,64), default=mozinfo.bits, ...)
btw, the method name does not really state what the method does
Your config is named database.rb, but the server is named server.rb. Either need to rename one of them or pass the config file name in line 18 to make the example work.
Removed it as it wasn't necessary.
@bp323 Can you convert to single quotes?
Why are you using Integer.parseInt()? Why is this different from everything else?
You may use dsymm
currently it's based on content introspection, i.e. matching that we've received a log line saying "Done. Build script exited with:"

it'd be better to pass an explicit flag along with the last log message from the worker, that could be a normal log message with an empty string but a flag like `:final => true`
These 4 lines are copy and pasta for every method in this class.
shouldn't this be done after the ``before`` event instead, to have first the global ones and then the route-specific ones ?
Should be DISTINCT * to be in line with the rest
Are you sure ``array('compound' => false, 'data_class' => 'Symfony\\Component\\HttpFoundation\\File\\File')`` has the correct type as "array<string,Boolean|string>" is not a sub-type of "array"?
hmm, and ``set`` too
This is something that we're not doing, but I'm also not following the example that's given. Does it mean to say that `Large-Scale` should be enclosed as in `Large-{S}cale`?
We talked to Marc and decided there's no reason, since a node server hosting an app isn't the same thing as a device using an app hosted somewhere else.
This has also changed. The new spec:

Evaluation of an equality expression ee of the form e1 == e2 proceeds as follows:
The expression e1 is evaluated to an object o1.
The expression e2 is evaluated to an object o2.

If either o1 or o2 is null, then ee evaluates to identical(o1,o2). Otherwise,
ee is equivalent to the method invocation o1.==(o2).
I didn't want to confuse the sample here. :)
this should be removed. It does not make sense as Twig does not use SVN
看一下send的实现，很简单的。
Kokoelmia käpistelevät funktiot toimivat yleisesti ottaen kivasti `nil` arvolla, esim. `contains?` palauttaa aina `false`:n.
No need to do this manually/memorize it - just import Services.jsm, and use that (Services.prefs).
You know, you really should have a return; here but meh
Changed to:

    TcpInput input = service.getInputs().get('10000', InputKind.Tcp);
I'm confused. What is this new support for a resource scheme "splunk_search" for? And why is it here?
Can you clarify what this will return if no judge has yet submitted a response to this task?
No, `>` in CSS. jQuery abstracts differences so there should not be any issues. But it works in the same way.
Use jQuery `.each`.
This is too error prone. You will have to capture date with regex.
How should i do this using Tastypie? I don't get data in bundle like post does. In bundle are data from comment not from notification.
Ah, yes.  All is well then.
make sure that pol_gens ends up as a list of strings; Symbols will work if not processed, but they are significantly slower:

```
>>> timeit("'%s'%a","from sympy.abc import a", number=10000)
0.14560361250843351
>>> timeit("'%s'%a","a='a'",number=10000)
0.0040921665412323449
```
This should be --worker-class now.
While the list that stores each returned number is cleared after each range is exceeded (to minimize memory consumption), in the range of Int64 (9,223,372,036,854,775,807) this is going to hurt performance after some point.

I would suggest that we leave that as-is.. After all, when/if run out of numbers we start from the beginning. Furthermore, one could manually pass Int64, if required to serve so many unique, random, numbers (9,223,372,036,854,775,807) using a single instance :)
Wouldn't it be simpler just to inject a RandomNumericSequenceLimit into RandomNumericSequenceGenerator?

If we did that, would we need the RandomNumericSequenceLimitGenerator class?
Awesome - we always prefer to ship working credentials. Makes it way easier for future maintainers.
corrected, anything more? :)
tab here, and there should be a new line at the end of sql file
@PuniCZ i was using that but it was giving compile errors.

@warpten this actually makes sense as we should never target something that is in an invalid position in the map.
bluffing?
How would you feel about namespacing the ajax urls so that they are also prefaced by 'project-hours' just for consistency with project hours?
Something like:

^project-hours/ajax/

and one with the <pk>
Right now it is `nil` by design. I could make it it the ID of the current user, which might make more sense.
> still if your goal

**however** if your goal
Interface used to SNAT inline traffic
Cheering for the progress indicator?
users are not supposed to use this method
however, do we really want to hard code the version ?
Looked better before
Should use double quotes as primary, throughout the file.
You don't need here the outer brackets AFAICS.
Because when the code is used internally at Google it's a typedef for something else.
why are we demeaning?  shouldn't acf return the correct confint?
Same concern about array creation here. It might not make a big impact on performance, memory or smth, but I think it's unnecessary here. We can check that `for_code_point` returned an array and get its 3rd element, but if it returned `nil` simply return an empty string. What do you think?
Ah: `apf.escapeXML(filename)` will do this.
Brian - We're going to submit a pull request to connect integration so there can be a full review of the feature-direct - I added your comment to this ticket so we can track these things for review fixes:

https://issues.connectopensource.org/browse/GATEWAY-3190
super() is unnecessary.  I've seen it in multiple places.  It's just clutter.
i added this enhancement issue to CONNECT backlog: GATEWAY-2951 (extract mysql connection URL as a maven build property)
when fire'ing eZ template operators, it's nice to use brackets, even if there is no params. I mean ezurl().
I removed 
log4j.logger.brooklyn.SSH=DEBUG
and replaced by:
log4j.logger.brooklyn.util.internal.ssh=DEBUG

In all log4j property files.
Jaurais precise que c'est un domaine dans lequel le cerveau humain excelle slors que les ordis sont notoirement mauvais.
Is it necessary here?
I know, I've changed. I hope don't skip none
액티비티 바로 바꿀께요.
**Error:** A closing tag is not permitted at the end of a PHP file
you have to require 'csv' instead of 'fastercsv', but it's in the standard library so it's not an external dependency in the gemspec
お..
確認して修正します><
On Mon, 17 Sep 2012, Jeremy Mikola wrote:

> > +		add_assoc_long(server, "port", port);
> > +		if (database) {
> > +			add_assoc_string(server, "database", database, 1);
> > +		}
> > +		if (username) {
> > +			add_assoc_string(server, "username", username, 1);
> > +		}
> > +		add_assoc_long(server, "pid", pid);
> > +
> > +		/* Grab connection info */
> > +		add_assoc_long(connection, "last_ping", ptr->connection->last_ping);
> > +		add_assoc_long(connection, "ping_master", ptr->connection->last_is_master);
> > +		add_assoc_long(connection, "ping", ptr->connection->ping_ms);
> > +		add_assoc_long(connection, "connection_type", ptr->connection->connection_type);
> > +		add_assoc_long(connection, "max_bson_size", ptr->connection->max_bson_size);
> > +		add_assoc_long(connection, "tag_count", ptr->connection->tag_count);
> 
> What is tag count? The number of tags associated with a mongod?

Yes

> I assume we wouldn't have access to any of this information if 
> connecting through mongos, correct?

That's an interesting hypothesys, that needs checking. I assume we 
don't have access though.

> Otherwise, is it the tags for our read preferences, that the 
> connection presumably matched?

It is the tags that have been assigned to the server for which the 
connect element is. It doesn't clarify whether something got matched or 
not.

> I think tags would be more useful than the count, which the user could 
> calculate if necessary.

Indeed.

cheers,
Derick
fetchとmergeの違いが分かりました。
Where is this new partial actually used? :)
I have no idea of what you mean ? :)
Is it just removing all migrations and creating a new initial one or is there some special way of doing this?
If you want to keep all your dump, can you do a LOG(str) method an enable/disable it with a |var debug| variable. I would like to not pollute the log file too much, it's already a pain :/
now that we face this pattern a lot, it may be simplified as;

var params = goog.array.map(this.parameters, function(p) {
    return p.paramName
});

return params.join(', ');
i'll remove file_get_contents, asec ;)
OK, but if it's PSR-0, we should put it in the right place and let it autoload, IMO.
This var is no longer used.  Just revert the throwpass variable additions, and I will make it work right. Open an issue for me, kindly :P
these also need to be node['cinder'], not node['openstack']['cinder']
What would it take to insert without reloading?
так мы же здесь ничего не пишем в лист1 и лист2, только читаем из них, вполне укладывается в "запись производится редко, а чтение - часто."
Ок, вот тебе задание. Напиши тест на сортировку (последовательную) 10 тысяч случайных строк (где будет половина одинаковых). Один вариант - сортировка, а потом фильтрация на выходе, а другой - фильтрация, а потом сортировка. И сравни времена. Используй PerformanceTest. Тест закоммить.
Please try webViewFrame = self.view.bounds and consider navigation bar height.
It's supposed to be .* = %

.* is regular expression
% is JDBC syntax for the same thing.

is only used in file name globbing which is not what was intended here.
If you replace this with an org.jboss.spec dependency, no need to specify the version manually
It appears that this would either return True or throw an exception. It would never return False. Assertions should be in the test, so when we break this rule we should add comments to explain why.
This indeed is failing. It looks like it's a Marketplace issue. Search suggestions are not displayed on other browsers except Firefox on mobile. while we are waiting on an answer from @krupa on this, I'll go ahead and merge the pull since it has other tests.
Can you throw an AndEngineRuntimeException here?
``-window_ext``
Yeah, I am wondering that, too ;)
I would rename `transf` to either `transformer` or `grp`.
I am just being very very very dense and did not notice when looking at the patch. My whole thread of comments here is just pure noise :(

I think that I try to be too "efficient" when reviewing patches, and end up in such situation. I should rather accept that slow is better than fast.

:+1: for merge. Thanks.
just built the doc. It seems fine. Don't bother
Do I understand that correctly that this is the effect of ``from __future__ import division``?
shouldn't it be < 0 if the real concern is overflow?
Why don't use VARCHAR here to not make key fixed length? Could there be cases when App have multiple values for key? What about unicode key/values?
I'd remove a lot of stuff here since it is an example :)
Thinking of it, configuring CLI is a single operation. Could the `loadCliHelperSet` and `loadCliCommands` be unified?
I'm not, I was meant to ask you this directly, but I forgot...
Note: I would expect there should be some 'reset' function where these should be freed rather than the destructor. 

@jakesmith this pull request is so I can carry on leak hunting - it is very likely NOT the correct fix.
What I need is a paper/blog series/whatever that talks about CPAN, Rubygems, NPM, PIP, and Maven. A lot of my foundation comes from having only really interacted (as a publisher) with Rubygems.
That should be 

```
ok_('has_builds' in info['hits'][0])
```
Because this is so specific to Socorro and, in fact, signatures, move it into the `SocorroCommon` class. It doesn't need to be a private function actually.
shouldn't the cookie location be an attribute?
"ecosystem" should not be hyphenated.
should it default to the pytz list of values?
argh looks like I can't require 'spree/core/testing_support/fixtures' in 1-0 stable #1157
Done in fe6e93e
Alexander, what he means is you condensed what's between the brackets to one line - it's easier to tell what's changed and what's added in future if it's spread over multiple lines.
Abort if the `wget` fails here or on SDK2.
Please change it to `$isRetweet`.
why use defaults from the params namespace if the values do not change based on OS? Why not just specify the defaults in the class declaration?
Doesn't make sense for this kind of module, remove.
This is a nice choice, but I suggest allowing the user to edit the non-existent page through an option. Lots of people run port 80 on 0.0.0.0, so this will false pretty often.
Making PrependMigrate (#939) a standard option on Windows payloads and defaulting it to true in ```Exploit::EXE``` would solve that problem.
If I change the image to be non-square, I get errors. I think some of the width/height values are switched in the segmentation algorithms.
Because column 9 will always be included and the corresponding coefficient must be 1 (constraint). That's just the basic principle of total least squares using the SVD.
I know what the basic concept behind your thought was but:

This wouldn't be "consistent" / clean in the case of the similarity and affine transformation (more than 8 columns in the A matrix - projective and affine would work perfectly fine).

In my opinion this would make it much more complex and thus difficult to read and evaluate later. I would not make the code shorter in exchange of readability.

Don't get me wrong, but if this is a feature you really want you would need to implement it yourself in a separate PR. I'm also a bit short of time at the moment.
I'll take a look.  I actually don't like the model matrix the way it is now.  It is needed in two locations - once returned by `update` and once in the uniforms list.

So do you think this is the right workaround?  Why not add the model matrix to the central body instead of changing the RTC algorithm to look like it is potentially doing something else?
It's probably obvious, but don't just double the width and height; use `min()` to make sure the new size accommodates the image.
Isn't this a minimum height, not a maximum?
Belongs in CesiumMath.js, right?
Mark,
I would like to set the $ErrorActionPreference to "Continue" like the default. Is there any specific reason it has to be Stop?
`Backgrounding the app...`
Should be `typeof data ===`
DEPRECATED_REMOVED does not match deprecation version in documentation.
You can also just call logger with an empty string:

    logger.log();
how about iOS/android ?
See above. You can leave the assignment here, but declaration needs to go above.
Mobile Web supports this too.
Use a blank line above and below the copyright. See the other examples.
put docstrings in triple quotes, so it will be picked up by the documentation generation tools.
Nice, I was hoping for something like that.
I'm happy to amend this commit in an effort to get this merged in quickly.  I'll paste a diff with the small change I make so it's clear.
Should be fixed as a part of #295.
@michitux the message is only displayed occasionally (when the security cache times out), but yes a placeholder might be more appropriate.

@adrianlang good point, but I assume the admin would disable this feature then? But we can easily check the appropriate auth capability. I'll have a look
Excuse me then. Indeed there is no need for else.
I think you wanted to change this to the Hibernate line

Define the version of JBoss' Java EE 6 APIs, and of the Hibernate family BOM we want to import
There's an isHandled() method on ExceptionToCatchEvent that you can check
to see if the exception was actually handled. Just because it goes into the
exception handling stuff doesn't mean there was actually anything to handle
it :) You can also add qualifiers to the constructor, which isn't shown
either. Probably both important aspects to show in a quick start.


On Tue, Nov 27, 2012 at 5:44 AM, Rafael Benevides
<notifications@github.com>wrote:

> In
> deltaspike-exception-handling/src/main/java/org/jboss/as/quickstarts/deltaspike/exceptionhandling/Controller.java:
>
> > +
> > +    // Inject CDI Event
> > +    @Inject
> > +    private Event<ExceptionToCatchEvent> catchEvent;
> > +
> > +    // The Service implementation always throws Exceptions
> > +    @Inject
> > +    private MyService myService;
> > +
> > +    public void testMyException() {
> > +        try {
> > +            // This Operation will throw an Exception
> > +            myService.doSomeOperationWithAnException();
> > +        } catch (MyException e) {
> > +            // Fires the Event with the Exception to be handled
> > +            catchEvent.fire(new ExceptionToCatchEvent(e));
>
> Jason.
>
> I didn't understand what you mean.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jboss-jdf/jboss-as-quickstart/pull/361/files#r2234526>.
>
>



-- 
Jason Porter
http://lightguard-jp.blogspot.com
http://twitter.com/lightguardjp

Software Engineer
Open Source Advocate

PGP key id: 926CCFF5
PGP key available at: keyserver.net, pgp.mit.edu
1) и чего делать с ентим AttributeError? ловить его? У меня не выпадает - всё работает вроде. 
2) DeprecationWarning не умею и не считал его нужным в данном случае - это просто 2 разных поведения. Настройку вставил типа для backward compatibility
So?  If it fails to insert, by definition it does not have a valid androidID.  And why would you need a stale androidID when tracking?
+1 Probably need a policy for all the user strings. Eventually can get @hflynn involved in fixing all my many many discrepancies.
dont need $app->primary_group
I have made the code change - however - I disagree with the necessity of this in this instance 

----- Original Message -----

From: "nrporter" <notifications@github.com> 
To: "plymouthstate/plymouth-webapp" <plymouth-webapp@noreply.github.com> 
Cc: "bscoleman-psu" <bscoleman@plymouth.edu> 
Sent: Wednesday, October 3, 2012 10:56:04 AM 
Subject: Re: [plymouth-webapp] changes for adding new BM Report App (#425) 


In legacy/hub/HUBSQL.class.php: > +    {
> +		  $sql = "SELECT * FROM `bm_rounds` rounds WHERE rounds.bmr_id=? ORDER BY time ASC";
> +      $rs = PSU::db('hub')->Execute($sql, array($data['id']));
> +			$rounds = $rs->GetRows();
> +			$data['rounds'] = $rounds;
> +      return $data;
> +    }
> +
> +	return $null;
> +	}
> +
> +/**
> + * add a Round Entry for Building Manager Report to the MySQL hub database
> + * 
> + */ 
> +	function addBMRRound($now, $data) 
If you are adding something to the database, it is best practice to return some sort of success indicator. It costs nothing in processing terms, and adds the ability to grow functionality later on. Please return the result. 
— 
Reply to this email directly or view it on GitHub .
Ah, yes. That needs to be fixed. Thinking about it more, `$_activeTokens` should be an instance member, as it would be wrong to share these tokens across instances, with potentially different Service Bus services.
See the frontend toolkit.

On 16 Oct 2012, at 20:18, James Abley <notifications@github.com> wrote:

In app/assets/stylesheets/tariff.css.scss:

> @@ -261,7 +261,13 @@ mark {
>  }
>
>  #wrapper.tariff header.page-header hgroup h1{
> -  background-position: 15px 15px;
> +  @media(min-width: 1020px) {

It feels like we could do with a way of sharing all of our standard
breakpoint sizes rather than gradually replicating across all apps.

—
Reply to this email directly or view it on
GitHub<https://github.com/alphagov/trade-tariff-frontend/pull/21/files#r1858622>.
"deprecated" not "depricated"

Also needs a space between those errors, I get:
     WARN: Depricated CLI arguments are being used.Wrong number of arguments specified.