labofsahil/github-event-dataset-2012 huggingface dataset:
`rmagick` has a history of being a PITA and introduces the need for folks to have ImageMagick libs, I think. What are your thoughts on checking for the existence of RMagick (`defined?(RMagick)` or something) as opposed to always pulling it in in our Gemfile? That way if the user pulls it in (and you can show them how in the README), they get the map .. but others don't inflate their bundle with `rmagick`
This is the wrong expected value, should be 7373125414476351000
Bacana, GITHUB é uma mão na roda mesmo, ele abriu uma versão do projeto pra vc e seus commits são aplicados a ele ;)
lokcnes we hadnt attacked the bosses directly with melle so you cannot say that :O
Hi @risha

In your message you are missing the \ before the last ' character - but maybe that is just in your message.

The datepicker might not allow that format - have you tried {dateFormat:\'dd-mm-yy\'} ?
`unless` instead of `if not`
changelog from wrong repo commit.
Can you try again from master now?
Alright, that all seems to make good sense.  Thanks Alexey.
Use `libexec` as the private folder, not `include`.
You said "Maybe I should float elements or something"...did you try that?
Good catch ... thanks :-)
Im learning ruby so aplogies in advance if this is a stupid question but why do you assign STDIN and STDOUT to instance variables? Is this purely to make testing user input easier or is there another benefit?
@jamesturk - just so you know I signed your name :)
Put angle brackets around "port" as well
looks good. but why exactly? so we don't have to require('fileUtils')?
No code changes, just run "rake db:migrate". Particularly useful for searching test_cases (eg. for problem judging), since there are already so many.
I agree that we need to be able to run the dev server to test library code. My question was really more like, "Does the configuration need to be shared between the tests and the app?" In other words, couldn't we completely decouple the app from the library by making the configuration a parameter to the bits that run the dev server, and have separate configuration for the library and integration tests. The integration tests could reuse the configuration from the application, since that's really a test at the level of the application anyway.

One thing that struck me is that, in looking at the configuration in one.application, it includes things like "goog.require(one.sample.model)". That seems problematic for testing the library code, in that it introduces a runtime dependency on stuff in the application. So again there, it seems like it might be better to have separate configs, and a way to specify them when starting the dev server, rather than an implicit dependency.

But again, I'm more than willing to believe that I haven't thought this all the way through. If you still think this approach is best, then I don't have a problem with merging it in.
Ah that's very interesting. I see at least 5 or 6 other instances of "--" in this file, though. :P
Pois, eu só percebi isso quando vi que a versão que tinha sacado do software livre era maior que a que saquei daqui... Bem, fica o patch para quando actualizarem aqui!
от того что я прочитаю лог конфы у меня падать не перестанет.
@apneadiving this commit 6c230401b1326e64bb617c7525da090098cb0208 changes your guide somewhat because Refinery::Helpers is no longer used. See http://groups.google.com/group/refinery-cms/browse_thread/thread/ec9a6793af201a86 by @mobyjames
Why not make the object called "op" and have the name/type as another property?
This show the log file in the DSM interface. Not perfect, but can save time.
can we have a verbose option which produces the output like now, and per default a silence option which only shows:

   CHECKDEPS package1... ok
   CHECKDEPS package2... not found

and at the end maybe a summary like:

"package1" not found (http://www.domain1.com/path1/package1.tar.bz2)
"package2" not found (http://www.domain2.com/path2/package2.tar.bz2)
...
Well, if there is a mime type already, let's use it and remove this comment.
Great! I pull and merge asap hope to finish by the end of the week, we are very busy.
noob du c++ c est corrigé reste a testé
@nickstenning looks good. BTW here's the DAO / Domain Object layer around ES in another project: https://github.com/okfn/bibserver/blob/master/bibserver/dao.py (you've already done a great job of wrapping ES (via pyes) but could still be useful especially re doing querying in a nice way ...).

Aside: and for good measure here's the DAO code for ES in node.js: https://github.com/okfn/hypernotes/blob/master/dao.js
and the git ls-files returns again! =)
有効なバージョンは http://dbflute.sandbox.seasar.org/ を参照してください。
This needs Javadoc
i confirm it. Forget this pull request then. We should find an other way to get the needed file name.
Ah, oui !! C'est le `plainTextConvert`  qui enlève tout ce qui est entre `[[` ? Super !  :)
Wouldn't it be better to keep the comment in the file for future reference and add the config line after it?
Good point. Yeah, I don't like the preg_match() call their either, relies on a very specific GET signature) - I'll see what I can do.
(Found this looking through the forks of AFNetworking—don't mean to creep or anything)

You might want to consider using `AFHTTPClient`, especially if you do a lot of requests to the same base URL. See the Gowalla API client in the example project.
Based on the commit, we should add the following updates going forward:
- Track the last record we processed so that we can include it in the next cycle of a task. 
- Improve user search to allow the user to define the operation on each attribute.
I have opened the following jira tickets to support this:

http://jira.openiam.org/browse/IDMAPPS-385
http://jira.openiam.org/browse/IDMAPPS-386
Why `H <: U`? Why not have,
```implicit def hlistFilter[H, L <: HList, H, Out <: HList]
     (implicit aux : FilterAux[L, H, Out]) = new FilterAux[H :: L, H, H :: Out]
```
It's marginally simpler, passes the tests and is more consistent with eg. `select[U]`.
Saying that deceleration always continues in an "upward direction" is incorrect. This can be removed.
This particular issue has already been resolved in git master as of a week or three ago. There was a bug in GIT support which resulted in multiple files being returned when status was requested for a single file.
```ruby
unless other.instance_of? Byte
  raise ArgumentError, "Expected Byte, got #{other.class}"
end
```

This would help debug the test failure that you're having. If it helps you, chances are it'd help the students too. This kind of check would be useful across `ks_primitives.rb`, so consider wrapping in a method like `ensure_byte_instance(arg)` and calling `ensure_byte_instance other` here.
here you are creating jQuery object twice when you create it from the  string

this.$el =  $(element);
this.el = $(element)[0];

should do as it will always first wrap normal dom in jQuery, convert string to jquery wrapped DOM etc and then just assign the DOM node to this.el + we dont need to do the isString check which is done in jQuery anyways
Dann änder aber auch nicht die users index und die appointments index !!!!
0.upto(7)

When you know both ends of a counting loop, this is the way to iterate.
Integrated this check on the level — using a numerical check agains the established tree_levels variable (which we can assume is the size of the levels array) instead of type checking, which is likely a bit slower.
Range check?
Fails with:

```text
Login response: {HTTP::Response=HASH(0x2557e58)->content}
Changing domain of cookies...

.cookies does not seem to contain cookies at /usr/share/perl5/HTTP/Cookies.pm line 432.
Login successful.
Displaying shouts is not implemented yet.
```
Coding at 5am, don't you have work in the morning?
Some debugging code left in by mistake
I wonder if it would make sense to have an abstract Layout Computation class for algorithms that just do a single iteration (and thus don't need an executor)?
Sorry, I didn't meant NPEs but class loading issues (looks like a conflict between classes embedded in IJ and Groovy).
So you're saying that we can support multiple unique properties?

On Thu, Jan 12, 2012 at 6:18 PM, Jonathan Matheus <
reply@reply.github.com
> wrote:

> We can now that we're using a UOW. We previously opted not to rely on
> Raven change tracking, but recently changed our minds. I'll change it.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-861627
>
Tested with Sindragosa Frost bomb

Long missile dist : Spelleffect arrives 2-3 sec after missile hits targets. (Visual lag or Effect too fast ?)
Short missile dist : seems good, but possible improvement
received unexpected opcode CMSG_CANCEL_TRADE
@LiMCrosS, added those in 5c62fd1d6a05f9afb1d3b5cbd2901710d1223b4a, thank you.

About PHASEMASK, you don't need value2 to negate the condition since there's a new field named NegativeCondition that does exactly that.
The README.md should contain details about the codebase itself - the mention of forks and base projects should be dropped.

Similarly, the changes to the code discussed are covered by the commits and pull requests already.
@jhinkey Here are the UTF16 values:

С = U+0421
у = U+0443
п = U+043F
е = U+0435
р = U+0440
If possible, try to locate the path to this file in the filesystem.
If found, we can pass it to Less so that `@import` directives work well.
not running fine with migrations  in development mode:
 when doing a rake db:rollback
yves$ rake db:rollback
==  CreateAreaServicesTable: reverting ========================================
-- drop_table(:area_services)
   -> 0.0021s
==  CreateAreaServicesTable: reverted (0.0022s) ===============================

rake aborted!
An error has occurred, all later migrations canceled:
wrong number of arguments (1 for 3)
TRACE--------
/Users/yves/.rvm/gems/ruby-1.9.3-p0@rails32/bundler/gems/acts_as_archive-0ec158dd1ca8/lib/acts_as_archive.rb:206:in `delete_with_archive'
/Users/yves/.rvm/gems/ruby-1.9.3-p0@rails32/gems/activerecord-3.2.0/lib/active_record/connection_adapters/abstract/query_cache.rb:14:in `delete'
/Users/yves/.rvm/gems/ruby-1.9.3-p0@rails32/gems/activerecord-3.2.0/lib/active_record/migration.rb:740:in `record_version_state_after_migrating'
/Users/yves/.rvm/gems/ruby-1.9.3-p0@rails32/gems/activerecord-3.2.0/lib/active_record/migration.rb:705:in `block (2 levels) in migrate'
======

I tried to disable it in the development environment
ActsAsArchive.disabled = true
but now way ...
--> getArcItem(number).getArc()
can you add "write", "take note" ?
Why giving the method a parameter if it isn't used in its body? ;)
Should explicitly test whether the layer exists instead of simply blindly catching any exceptions.  Also, maybe a 404 would be helpful for clients of this api when the layer does not exist.
This commit solves gh-7 for now.

Test server life cycle is now handled from Calabash.
Yo dawg, I herd u liek changelogs in yo changelog
Oooh, don't do this. Do this instead: https://github.com/danielbachhuber/Edit-Flow/pull/14#commitcomment-841293
No, I just wanted to know what to use to test the video player part.  the camera definitely didnt work.
Judging from the other bits in this commit, I can assume that "Nearby pages" means "Pages about stuff that is nearby (geographically)", right?
joe flacco
callisto:ahn_liquid_call jsgoecke$ ahn -
Starting Adhearsion server at /Users/jsgoecke/Dropbox/Development/ahn_liquid_call
/Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/adhearsion-2.0.0.alpha2/lib/adhearsion/configuration.rb:103:in `method_missing': Invalid plugin adhearsion (Adhearsion::ConfigurationError)
	from /Users/jsgoecke/Dropbox/Development/ahn_liquid_call/config/adhearsion.rb:38:in `block in <top (required)>'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/adhearsion-2.0.0.alpha2/lib/adhearsion.rb:48:in `config'
	from /Users/jsgoecke/Dropbox/Development/ahn_liquid_call/config/adhearsion.rb:17:in `<top (required)>'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/adhearsion-2.0.0.alpha2/lib/adhearsion/initializer.rb:150:in `require'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/adhearsion-2.0.0.alpha2/lib/adhearsion/initializer.rb:150:in `load_config'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/adhearsion-2.0.0.alpha2/lib/adhearsion/initializer.rb:42:in `start'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/adhearsion-2.0.0.alpha2/lib/adhearsion/initializer.rb:13:in `start'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/adhearsion-2.0.0.alpha2/lib/adhearsion/cli_commands.rb:97:in `start_app'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/adhearsion-2.0.0.alpha2/lib/adhearsion/cli_commands.rb:44:in `start'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/thor-0.14.6/lib/thor/task.rb:22:in `run'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/thor-0.14.6/lib/thor/invocation.rb:118:in `invoke_task'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/thor-0.14.6/lib/thor.rb:263:in `dispatch'
	from /Users/jsgoecke/.rvm/gems/ruby-1.9.3-p0/gems/thor-0.14.6/lib/thor/base.rb:389:in `start'
	from script/ahn:9:in `<main>'
callisto:ahn_liquid_call jsgoecke$
This commit goes the other direction, actually. It just makes it so you can do run(app) or run([app1, app2, app3]). I don't care much about backwards compatible API changes just yet, so if I need to add extra arguments I'll just do so when the need arises.
Possibly slightly more obvious what we're doing if this returns `True`...
I think this should either not be static, or we should actually enforce a singleton waiter pattern.

The use of isIdle in the tests is patchy, too; I think we should add the check to setUp and tearDown instead of inconsistently adding it.
At a minimum we need to get an upgrade doc going with gotchas like this 
in it.  This one could have been handled with a test for the old table 
and a rename as part of the new migration.  It could be removed 
eventually, but for the short run it would be worth it.  I'm guessing a 
lot of upgrade issues can't be handled so easily, but something needs to 
be done to help migrate live edge sites to the new release.

Devise 2.0 had multiple incompatible changes and they were handled with 
deprecation warnings.  Something as fundamental as renaming tables would 
certainly merit a deprecation warning rather than silent fail.


On 2/21/2012 4:12 PM, Philip Arndt wrote:
> Honestly I'm not sure currently how you would upgrade, and I'm unhappy
> about this :(
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/resolve/refinerycms-inquiries/commit/a971e7db0050d0c63a51dea3f298e163953f0c2d#commitcomment-993821
>
I only added the ability to disable it to make it easier to see if it it had any effect....
おおっ、ついに合格ですか！？ありがとうございます！！
Should be at least 16384 bytes here
Well I could but it is going to be removed within the next few months so it is kinda nice to just have them stacked
Hmm.. Interdasting.
And where did you see span with title in span[@Class="filesize"] on 4chan? On which board?
I really wonder if this will have any effect, because I added this to my sense kernel a month or 2 ago.. and some reported working, some reported not working... and that was with modified camera.apk from sensation
FrameworkBundle is also used for the services it defines for the asset packages. So you should have both
linker should work with 2G splits when compiled with TARGET_HAS_2G_VMSPLIT.

kcm files are for alternate key events, which doesn't apply here. The only thing you need is a kl file with the mappings for the events you do generate; that's why I'm asking you to test it, since I don't have a swift.
But mark up will start showing faster, isn't it?
and more hours plus public use with 10 keys
You can also remove swiftmailer and the annotation drivers in the autoload.php

also you should used a MapClassLoader instead of the ApcOne ... which seems not be used ...
It's declared, not assigned. Am I missing the line you're referring to?
Please put the above comment in the CODE, such like:
FIXME: The "change" event will refresh property view so "click" event of datalist is not triggered. We have to look  up the ":hover" class here to decide which item is clicked
getConfig should be `getRandomConfig`. Shouldn't have to read the docs to know it's randomized.
I don't understand. What "additional table data"?
`loggedOut.html`? I am not sure if this is used at all. Is it?
The error handler configured to manipulate the response by default uses status code 311.
Given that the name in the code must match the kernel name, do we actually need to hash self._name?  I guess it can't hurt.
but we have agreed on using types on public api, makes it harder to break the api
for example, the type of Duration.apply is FiniteDuration, but we might want Duration
if it is for an optimization we can leave it out, otherwise we should stick to our decision to use types on public api
Missing "must be (true)"
klassname_for was only referenced in a couple of specs.  Got rid of that and I think this is good now.
Code from `classes\view.php`. Minor changes.
On the client side it's valuable to not use a leading slash, since this is a relative path. I know that hasn't necessarily been the convention, but I think there was general agreement that we should move this direction for the sake of accuracy.
Update your template, it's 2012.
https://github.com/ethn/wagn/pull/21/files#r2228480
Sorry, I had not read the whole file... I'm not at maximum developper ability right now :)
As long as you don't change TimeoutException :)
I guess the solution to the issue I discovered is to also compare the original item id with the item being checked's id, not just with its set-id.
Is it possible to access the system deadzone/calibration settings through Direct Input? I think it would be better when the user doesn't explicitely want to override with an advanced setting.
spaces(1);
content.append(str).append("\n");
Is this resource a method when this else block is reached ?
I think that the associations in Card::Reference are what needs this.  It may only need it because we nest it in Card, but do we care to find out for sure if this works?
Do the NixOS tests that depend on test-driver still work with this change?
I know, but this way it seemed grander :( Some day..
what if we have multiple servers running on the same code?
A completion block parametrized form of this method would be awesome!
Guess so, but I'll check when I upload a fix for 1.6.2 to PyPi.
minor optimization:

```
for host in args.config.all_hosts():
    if host in args.hosts:
        print host
```
Probably also worth noting this in ``Qiime/ChangeLog`` as well, since you'll be adding the above note.
Числа
We have commented code here. If it is not needed please remove
This need a refactoring.

```ruby
association_name = association.to_s
record_class = record.class

merged_options[:root] = association_name

if record_class.to_s.underscore == association_name.tableize.singularize
  merged_options[:type] = record_class.name
end

records.to_xml(merged_options)
```
Replaced regular expression with string literal, as suggested by @rafaelfranca
If not how come I can do `<%= request.inspect %>` in an ERb template? It's the same context isn't it? I can also call `request` from a helper.
You're essentially calling stream.to_s two lines in a row.  Why not leave the original line that reads `stream = stream.to_s`?
Perhaps the test names could still have the `strict_mode_disabled` in them? Seems like it's what's being under test, not? How about:

    test_mysql_strict_mode_disabled_dont_override_global_sql_mode

Or something like that?
Is not more clear to just do

```eval "::#{old} = new(old, current)"```

Or am I missing something?
Should we also ignore `call_reject_if`? What if I am updating the record to a state that should be rejected?
It's a chain of method calls, regardless which object receives the first message. Indenting makes that clear.
fix translation "Antetítulo" (this is a localization for Venezuela)
I think we shouldn't throw that exception by default. At least in Akka 2.0 there is no exception being thrown by default.
Could you add some comments on the role of `fallback_pf` in the implementation?
What about literals such as $null, $true and $false?
I don't remember the reason for these. Is the list complete? What about `##` and `==`?
though that would have to account for ambiguities etc
No, tansform. It's Californian.
That's some code I forgot to delete. I'll remove that tomorrow. good catch
I don't think that will cover the numpy and script.sparse cases fully.  We need to make sure that adjoins/dagger works with those types of arrays

Sent from my iPad

On Mar 28, 2012, at 5:37 AM, Julien Rioux<reply@reply.github.com> wrote:

>> @@ -75,73 +71,13 @@ class Dagger(Expr):
>>     [1] http://en.wikipedia.org/wiki/Hermitian_transpose
>>     """
>> 
>> -    def __new__(cls, arg, **old_assumptions):
>> -        # Return the dagger of a sympy Matrix immediately.
>> -        if isinstance(arg, (Matrix, numpy_ndarray, scipy_sparse_matrix)):
> 
> It's below, we just call arg.conjugate().transpose() whenever possible.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1158/files#r614692
as below, should this be a FOR loop? (Personally, I don't care but they should probably both be the same.)
expontent->exponent
I don't understand why this line is necessary at all.  Isn't this the default if the case is not hit?
I would think so. Taking the dual of a matrix that just happens to be symmetric seems totally legitimate.
y was introduced but not used?
Explicit space after `elseif`.
This doesn't seem right, why are you looping over the boundary elements and calling EQUATIONS_MATRICES_ELEMENT_CALCULATE, but above you loop over all the internal + boundary nodes calling EQUATIONS_MATRICES_NODAL_CALCULATE.

You do something similar in EQUATIONS_SET_JACOBIAN_EVALUATE_STATIC_NODAL and EQUATIONS_SET_RESIDUAL_EVALUATE_STATIC_NODAL.
Isn't this an rcov artifact? I was actually getting < 100% due to the line after this one, which was 100% under simplecov, and is also 100% now I have condensed into a single line. @levent
That's ugly. We said that we would keep an interface in Silex with the boot method.
Yeah I was kind of irritated about the m4 directory.  I forgot about set.
add short desc such as:
# remove paramtere:'bg_command=<val>' from ARGV to prevent endless run in the backgroung executions which will crush the machine.
# add quotes to all parameters (sinc ruby interpreter removes them in the first execution)
I would move this to 'helpers.py' and create a jinja function. 

Also note that you shouldn't use hardcoded urls. Instead of http://reps.mozilla.org/u/ you could do something like 

```
return '<a href="%s">%s</a>' % (reverse('profiles_view_profile', self.display_name), urljoin(SITE_URL, reverse('profiles_view_profile', self.display_name))

As I say in the other comment, you can transfer this logic in the template.

```
You'll want to 'tokenize' the query first, that is split it into words.
Done (this comment remains because this line technically didn't change).
Should/is this event uninstalled when the upload succeeds?
Yes you should not be using public properties. The same actually applies to all the other public properties there which should be removed as part of the refactoring.
With the columns that are added instead?!
Nitpicking: "We dont use camelCase but under_scored"
Add a description please. Someone unfamiliar with the implementation should be able to understand what the class does by reading its description.
Two padding, why?
Should be only ```padding: 5px 10px```
That is an important thing for the admin to know, in my opinion. So I'd say call add_log() to add a log entry to let the admin know that permissions may have changed or the directory may not exist. You never know; it could happen.
Yeah, intentional  - I don't know the actual rtt times so I just assert on the number which should be 3.
please also comment it's a value between 0 and 100
I think `Y.Lang.isObject()` returns true for arrays too.
That makes a lot of sense @jgoz thanks for the tip :)
You're defaulting to 0 here, right? If so, why? If no identifier is specified, you shouldn't try to guess it, but throw a 404 page instead?
Not 100% sure, since I've not run it in a debugger. But when `s = "/Date(-59011459200000-0000)/"` then `tzCharIndex` will be `6`. Now if you look at `(tzCharIndex > 0) ? tzCharIndex - 6 : s.Length - 8)` then `(tzCharIndex > 0)` will be `true` which in turn will mean that `tzCharIndex - 6` will be used. So the substring operation would operate from position 6 and with a length of zero.

That's where the bug is and the logic that needs to be reworked to be compatible.
add a sub-ticket under the "refactoring/cleanup plan" ticket
unless it's very important that this is temporary, recommend changing name to be more informative
Looks good, never mind.
We should be raising a warning if either of these two attributes is not present.
Urgh. I've been reading this for 2 minutes and I'm still not sure I can see what is going on...
What happens if you add ```check_graphic```?
Please remove. If we find a need we can easily add in a genuine implementation.
I saw a PR for this from @lamblin, are you sure it is not fixed? Otherwise, do you remember what you fixed related to it?
Again, should there be error handling, here?
dottedmag: With insert_text, the text is appended at the end of existing text , including the existing whitespace - so the behavior is as expected. In order to replace the text, including existing whitespace, you would use set_text instead.

sigurdteigan: What do you have against potatoes?!  Maybe next time I'll be inspired by Dr. Seuss!! (one fish two fish red fish blue fish....) =P
Not if you look at the function signature:

```c++
void loadStringstreamFromFile(std::stringstream &stream, std::string file);
```
these __future__ imports may need a "note" box?
Not sure how this got here, but I'm trying to keep the compiled JS out of the codebase. This should not be checked in. Did the makefile do this? It might have been naughty and put some JS outside of the .gitignore paths...
Luigi,

I have used python for 6 years now and never saw or used a factory pattern in a *python* code base.
A prominent fellow countryman of yours is of the following opinion:

    Creational patterns [are] not very common in python because "factory" is essentially  built in!

See www.aleax.it/gdd_pydp.pdf for some more wisdom on this subject.

IMHO factories are a clear indication of code "smell" -- python is a *dynamic* language after all.
For me this page contains ideas, but not requirements. "You can't manage what you don't measure.", i.e. I prefer to see 80% coverage and so 20% uncovered than 100% coverage with 99% excluded by developers.
This is where it is called: https://github.com/jquery/jquery/blob/master/test/data/testinit.js#L91
just call the method from above, don't do the the two 1/x again(duplicated code)
I can set the search path at the beginning of function and restore it at the end. Please see the following code.
```sql
    EXECUTE 'SHOW search_path' INTO searchpath_text;
    EXECUTE 'SET search_path = ' || searchpath_text || ', MADLIB_SCHEMA';
    ......
    EXECUTE 'SET search_path = ' || searchpath_text;
```

* As we need to concat the string,  we should consider the special chars. I try to add schema "a'b" to the search path, however, it failed. Any suggestions?

```sql
testdb=# set search_path="a'b", public;
ERROR:  syntax error at or near "b', '"  (seg0 aojwang:40000 pid=12297)

testdb=# set search_path="a,b", public;
SET

```
For args[2], the expected number of observations, this error msg is correct. You are thinking about args[3], the degree of freedom.
This should be marked as an example and formatted as code. E.g. like this:

```
#: Example::
#:
#:     SPOTIFY_PROXY_HOST = u'protocol;//host:port'
```

Note the double colon after "Example" and the extra indentation of four spaces of the setting compared to the "Example" label.
Ah, in that case I would add a constant ``TRACK_AVAILABLE = 1`` and do ``if track.availability() == TRACK_AVAILABLE:`` until pyspotify provides these constants itself. Makes it a lot more clear what on earth one means :-)
Use triple backticks at the beginning and end to avoid the formatting ;)

OK, so I take it the directories get the paths assigned from the configuration step. In that case you can get the paths like so:

```
    NOTES
    A) yaws ships with a default configuration that
    expects the following directories to exist:

      #{prefix}var/log/yaws
      #{lib}/yaws/examples/ebin
      #{lib}/yaws/examples/include
```

This works since you have access to Formula's instance methods, with `prefix` and `lib` containing the formula's specific locations of those respectively. It also means you won't have to upgrade these every version bump. ;)

(Though if the stuff yaws is looking for is stuff that should be surviving past version upgrades, it may be better to use `inreplace` or similar to fix up the paths in yaws.conf before installing so it points at the standard prefix locations instead of inside the Cellar.)
If we're using double-quotes (`"`) to enclose strings everywhere, then this one should use it as well :)
Add a comment here explaining that this is for commander 4.0.3
Hmm, finde ich schwierig einzuschätzen. Allgemein glaube ich, dass diese Funktion am unwichtigsten von den vier ignore*/filter*-Funktionen ist und am wenigsten gebraucht wird (im Core und in unseren Addons brauchen wir sie nirgends). 
Ich finde aber das meine Variante bessere ins Gesamtkonzept und besser zu ignoreDirs() passt. Also sobald ein Ordner gefunden wird, der nicht den geforderten ignore/filter-Regeln entspricht, wird an der Stelle abgebrochen, und die Kinder nicht weiter untersucht.
Add some JavaDoc explaining what this is and what the contract should be.
outpipe is a pipe owned by the socket.
You should not call any method on this pipe or you have race condition.
When you call the temrinate method on your pipe, a request is sent to the socket. Once the socket receives the requeest, it will terminate the pipe itself.
I think this was intentional - it looks like @Hoikas  undented the entire preprocessor section.

@zrax, do you have a preferred way of handling indentation for this sort of inline preprocessor voodoo?
Changed the formatting.

Also added tests for _digest_result and converted to use StringIO instead.
`.empty?.should be true` could be `.should be_empty` - same kind of readability improvement can be done for some of the other tests too.
Oh good. Thank you for the explanation. There's something surprising to me...

Why is retrieveQueues returning a value that is not null when OptInRequired happens? In CachedMap:168 I would have expected `items` to be null, which would throw a NullPointerException on CachedMap:169. That's why I thought there would still be an exception throw before `fill()` completes. Any idea?
We could probably dry this up having a default `Batman.DOM.setInnerHTML` that does the work with the child nodes and `willRemoveNode`, then delegates the actual DOM operation to the platform adapter.
I feel like I've seen ways to copy text from a web page, but all my googling results in a Flash-based solution.
The most common word is "file". There is a similar question about folder/directory
In general, it seems like the two event handlers here are similar enough that they need to be factored out into a separate function, with a boolean parameter that switches between whatever behavior is different between the two.
This name is slightly confusing--it sounds like it's a kind of file list. Maybe `fileListPromise` would be clearer?
as added precaution I think it would be good to set shouldShowHintsOnKeyUp to false in the close() function as well.
yes, I forgot it
mark_safe?
This was moved from the original template before I async'd. You're probably
right that that would have been better back then, but now that it's async,
using messages wouldn't work, so I've kept it this way. The server
generates the build error html and then the async UI just dumps that in the
build errors div.


2012/12/12 Biyeun <notifications@github.com>

> In
> corehq/apps/app_manager/templates/app_manager/partials/build_errors.html:
>
> > @@ -0,0 +1,87 @@
> > +{% load xforms_extras %}{% if build_errors %}
> > +    <div class="alert alert-warning alert-block">
> > +        <p>Build Failed!</p>
> > +        <ul id="build-errors">
> > +            {% for error in build_errors %}
> > +                <li>
> > +                    <i class="icon icon-exclamation-sign"></i>
> > +                    <span>
> > +                        {% if error.type == "invalid xml" %}
>
> is handling the error.type in the template like this really better than
> using messages?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/core-hq/pull/373/files#r2391353>.
>
>
I think you want either `if (num !== undefined)` or `if (num)` here. (Fuzzy equals is hard to remember, but for example `null == undefined` is `true`.)
Yeah, wasn't sure exactly the best way forward on the domain specific apis. one minor fix was to set the name= to include the domain. but yeah, namespacing and ensuring uniqueness is an issue. but didn't see another alternative that wouldn't require updaiting the global URLs, aside from say, adding a new settings block that maps these domain specific url's as well. 

I'd be fine with not having the __subclasses__ call not be used because it's too magical.
Can you clarify what's wrong?
Probably there should be one section for some general things, for home page, and then each for subpages.
Why the case change and added parameter here? It doesn't look like it's being used here.
Looking at things a bit more I also don't think we need the 'header-block' class on these. There is only one mini-col on the pages that we're using it so can just use that.
in `document.fullscreenElement && document.fullscreenElement !== null`, if  'document.fullscreenElement' is true, that implies 'document.fullscreenElement !== null'.

can we use just 'document.fullscreenElement' here?
Short text here that describes image (alternative text)
Sure, it could be something like:

```
deps = [ self.source_path]
if os.path.exists(self.metadata_path):
    deps.append(self.metadata_path)
```
Its really not clear to me why you're adding the ```'stateconf_sate_func'``` key to __opts__ when it is not used anywhere outside of this code. It could easily be a local var without the extra effort. At first I thought there would be a mechanism for setting any module's state function to that key as the comment suggests, but that does not exist afaict. Can you clarify what this accomplishes and update the comments to match it. I think its overly complicated at the moment for what it actually does.
unless we have concept of 'started down'
add another example with least directives before this example.

For example:

       #默认配置: cookie=route mode=insert fallback=on
       upstream foo {
           server 192.168.0.1;
           server 192.168.0.2;
           session_sticky;
       }
I think this will mean that methods like update() will fail by silently blocking forever.  Not sure if that's a good idea.
You're so hard on this ;-)
Should we add a minitest for the name/label/id change on the controller?

Should we add a CLI test for the name/label/id change in the python?
No problem.  I raised the question since the bugzilla and commit only mention removal of the footer links (and not copyright).

I don't personally have an issue with the copyright being removed from the footer, unless it is something that is required as a means of protecting the UI (which I don't believe was the intent when originally added).  The code itself does have proper copyright/licensing information to ensure that GPLv2 is maintained, which is probably more important. :)
We should probably refactor all these `"%s_%s_%s%s" %` sort of things to use the [new string formatting style](http://docs.python.org/library/string.html#format-examples)
this class needs to be placed in another file. Currently, the parsing of the file will throw a fatal error when AMQP is not available instead of skipping the test (see the tests for the other handlers relying on dependencies)
Is this a bug on telephony API permission management, or it's a feature?
You have it in the module docstring, but I think it would be helpful to have an `Example` section here showing how to use this class, since it's not as obvious as just calling a function.

(I like this functionality, by the way).
I agree, this should be

```
dlong = (long2 - long1) * cos((lat1 + lat2) * 0.5)
```
done (I specified "great-circle distance")
I'm not sure exactly how to deal with this, but I think there needs to be a path to allow for a `units` specification that fails to parse here.  Maybe catch the parsing exception here and just set `self._units = units`.  Then the other code needs to allow for the case that `self._units` is a `str` and not a `Unit`.

I just started playing with the new units module and see that it raises an exception for a unit specification that doesn't parse.  That's good, but in the context of users defining a table, there should be a way to put in whatever they want for units.  This will be especially important for tables that get read via non-standardized formats like ASCII tables.  We don't want a failure to read a table because the units don't parse.

Other than this, it all looks good, thanks!
No, I don't believe there is.  I'll update this with a little more detail.
Yes, there was an issue with the sentence. I've reworded the paragraph to make it clearer.
so this is already implemented in numpy iterable. is it sensible to reimplemnt this here? Sorry maybe this discussion has already happened and I missed it.
Yes, I don't think they need to be fully fleshed out, detailed docstrings.  But this looks fine!
If you look back at our API, you'll notice something crucial: you need a distance.

That is, a spherical coordinate is two angles and a *distance*.  Once you have that, you are free to represent any point an 3D space this way, which gives you cartesian coordinate conversion for free. This will then make the time-derivative side of things much easier to implement - I know this from work I did in `astropysics.coords`.  

@wkerzendorf - I am *strongly* opposed to even thinking about `SpaceTimeCoordinate` at this time.  Do not let the perfect be the enemy of the good - we should write an API that represents spherical coordinates in standard celestial systems, and the rest should wait for a (likely *much*) later version of astropy.
garbage_collect/0 should be fine here.
There can be a mechanism for that, I just mean it shouldn't be handled this heavy-handed. Again, we can worry about how to delete videos later, we don't need to define that when we're trying to flesh out the User model.
Fixed, thanks for noticing.
I didn't know that, will update tomorrow as I'm not at home now.
I'm not sure if I understand you, nUnit is non-zero, when updateDisplayUnit() is called via ths displayUnitChanged() SIGNAL and 0 when called to init. Do you want me to use updateDisplayUnit(0) and remove the default-init of 0?
Also please try and add at least one line description for the params of all new functions you have added.
This loop and conditional are empty.
done
>
> In core/src/script/CGXP/locale/de.js:
>
> > @@ -201,6 +206,16 @@ GeoExt.Lang.add("de", {
> >          noLayerSelectedMessage: 'Keine Ebene ausgewählt'
> >      },
> >
> > +    "cgxp.plugins.GetFeature.prototype": {
> > +        actionTooltip: 'Informationen in der Karte abfragen',
> > +        menuText: 'Abfragen der Karte',
> > +        unqueriedLayerTitle: 'Diese Ebene kann nicht abgefragt werden.',
> > +        unqueriedLayerText: "Diese Ebene unterstützt die Abfrage mittels " +
> > +                "Rechteck nicht (WFS GetFeature).",
>
> This one should also be changed I think (accordingly to what was 
> proposed for fr). For example "WFS GetFeature" is something the user 
> doesn't need to know about. We should probably have @eleu 
> <https://github.com/eleu> translate the french messages to german.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/camptocamp/cgxp/pull/308/files#r2445858>.
>


-- 
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)
but this would create lots of directories with a single file in them, no ?
perhaps `help_text_dismissed?`
Good idea moving this here from find_all_for_entity_check in Flapjack::Data::Contacts :-) (duh)
Vyssie sa porovnava != '0', co je cislica...
302 and 301 are not "types" but they are HTTP codes. A more proper name would be $redirectCode.
@TwP This needs some tweaking.

The issue is how to weigh tokens we haven't seen yet. We don't want the probability to be 0, otherwise its going to reduce the entire probability of matching the language to 0.
There's already `load_pkg_description`
We should try really hard to not add these methods to Nimbus. The keyboard ones specifically use constants that don't work on the iPad anymore with pull-apart keyboards and popovers.
But that still breaks if message has a newline. Oh, Just looked at the input format. This is easy.

```python
output = dict([line.split(': '), 1) for line in message.split('\n')[:2]])
output.update(' '.join(message.split('\n')[2:])
return output
```
Agree that is a better approach.
This line is weird.
placeholder is a valid html5 attribute
Why do you specifically need to handle this case? You changed _page_name in process_view already (and if it's an issue you could change _view_module as well), shouldn't the default code handle the rest?
We might not be able to fix this, but: One thing I am concerned about is the race condition here. If there are a number of people hitting this code, they'll all spin up HTTP requests in parallel, until one of them stores to cache, and all subsequent users will be cache hits. If we do this on the front page, this will perhaps be quite the number of users.
This is very expensive. 
To POST something it will need to connect to the database twice! Also, the `self.get()` does a bunch of other luxury operations that we don't care for right now. 

There has got to be a smarter way to doing the UPSERT so that we find out if it was already there or not and thus written. I know it's not standard ANSI SQL but something that PostgreSQL supports.
The naming convention for mailers is to call it `xMailer` where `x` is the name of the resource or controller. Here, `ProspyMailer` would be correct. It's always best to stick with naming conventions.
Can be refactored as a "elseif" so the redirect above could be removed (the user will always be redirected to "index" anyway at the end of the action
Tak ten setTimeout není moje funkce, leda že bych si udělal alias s přehozenejma parametrama.

Jinak jako ne, tohle z toho nemá vyjít, playbackMirror není funkce. Grrr CoffeeScript! Dám na začátek řádku 82 čárku, líp jsme to tady nevymysleli (když píšeš v CoffeeScriptu objekt a máš ho ve složenicích, tak musíš buď dávat čárky na začátky řádků anebo srovnávat ty nový property names s otevírající složenicí).
Is this actually Swedish, or have you removed the `#translate` tag and not translated it?
Can you remove the #translate tags once you have translated a line please?
It would make for shorter code for sure.
These conditions are meant to find params that are not optional. And as you said, all variables are required in the hostname, so they are not useful here. Simply remove it.
Oh, I see what you see. I miss that. Nice catch.
Technically correct would be Csprng (cryptographically secure pseudo random number generator), but that looked a bit too long to me.

Why not leave the name as it was, "Secure Random", it's not as long, but still descriptive? At least, I like that name more.
IMO this is not necessary.
Fixed as requested.

On Tue, Sep 11, 2012 at 7:42 PM, brian wickman <notifications@github.com>wrote:

> In src/python/twitter/pants/targets/internal.py:
>
> > @@ -62,7 +62,7 @@ def sort_targets(cls, internal_targets):
> >      def invert(target):
> >        if target not in visited:
> >          visited.add(target)
> > -        if target.internal_dependencies:
> > +        if hasattr(target, 'internal_dependencies') and target.internal_dependencies:
>
> if getattr(target, 'internal_dependencies', None):
> ...
>
> achieves the same thing
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/twitter/commons/pull/80/files#r1582996>.
>
>
ty very much 4 the tips
To avoid using value as throwaway variable:
for key in request.COOKIES.keys()
parentheses not required on `(highlight)`
Leaving the Endrsments for now as there is some side discussion on that (space limitations). My suggestion is to merge Skills with Endorsements section.
shoud be `.DS_Store`
Why do you put the templating part here ? You could use:
<pre><div><b>Rudder ID : </b><span id="id to replace" / ></div></pre>
And actually only replace what is only available server-side, letting one who is looking in the template understand the result look&feel.
You have a stray `print` statement.
It happens (news items being shared between libraries), and I am one of those having asked for it. I hope we are making this possible in ding2.
It's just a prototype, I did this when I was offline and I didn't know how look twitter/facebook/github profile's urls
Lo que comentamos. ahora. Dejar el locale  de la fecha de la internacionalizacion
Hm. Not sure I want to expose owners here - among other things, these will often be emails, don't need to give scrapers another target.
please remove commented code (and a few lines below) and useless comment like `// Return`
So even if this is the "main" view, it should have its own, separate element. Body is the general container so for instance if you ever decide to have more than one views, you cannot have them since there can only be a single body element on the page. Also if you ever to remove that, you cannot do that either since body cannot be removed.
this is part of #433
It would be better to poll it checking for server-state=running like in ManagedDeployableContainer or DomainLifeCycleUtil
You don't need to add every test to pom.xml, only tests that need to be run in full profile mode, and even then in most cases there should already be an existing package already listed that you can place the test in.
You don't actually need \ in this case. The presence of the parentheses is sufficient.
Shouldn't pdebuild be doing the patching? O.o
if ($address->getId()) is enough, just 10 lines of code later the similar construction is used.
nit: remove the console.log debugging statements
nit the code with `{}` as mentioned above.
`this.time.textContent = d.toLocaleFormat('%R');

this line replaces L40-L43
OMFG,I really need some vacation. :-(
If I would return there, 'open-system-dialog' and 'close-system-dialog' switch cases would never be achieved. Anyway, I am moving the event handling to the (now) 'TrustedDialog', as you suggested, so there is no more switch cases related to 'TrustedDialog'. I am including the security check for the 'origin' var on my next commit, even if it is not needed for this case.
I'm not in favour of such change. Storing UID in the category is a workaround because there is no a 'source' attribute currently supported in the Contacts API v1. It is expected that it will be in v2. We can detect easily that the category is a number and not display it for instance.  Last but not least prefixing it can have an impact as we would need a regression through all the functionality
Now, these value selectors works covering the full view-port in the device (UX decision), so it has to be over the status bar
Probably my bad but how would I get access to the DOMRequest.result inside request.onsuccess? Would be a good practice to something like: request.onsuccess = this.notifySuccess.bind(this, request) and then have access to the DOMRequest as the second argument to the function? Thanks!
I feel like the whole part below (= updating the status messages) should be simpler than that. I’m looking what I can move to the webL10n library right now.
@fcampo +1