labofsahil/github-event-dataset-2012 huggingface dataset:
3 0 stable

This is fix of the problem in this discussion: http://groups.google.com/group/rails-sqlserver-adapter/browse_thread/thread/e04613db0882f491

I had to do something different than in https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/155

That discussion fixes the problem that an sql-query might have a , (comma) in it. My problem was that an sql query might be longer than "expression direction". The code as it was took only the first word of expression and made the second word the direction, even though something like this: "CASE WHEN ended_on IS NULL THEN 1 ELSE 0 END DESC" is valid sql, where the expression is the entire sentence, and the direction is nil. 

I think this problem still remains in the branches for 3.1. and 3.2.

- Niels
It would be much more natural to return collections directly in conditions below instead of applying a "single return pattern"
Oh sorry. Just saw that you think urlencode() is too strict. Why?
If we made it non-static, yes.
No, what I meant was that there's probably some older code which don't always follow these design guidelines.
In general I always try to pass instances through references when they are guaranteed to not be null and shouldn't be owned by the called instance.
For these changes I can agree that there's no reason to keep mRoot as a field except for in OgreSetup, as it's part of Ogre and Ogre expect it to be used through the getSingleton methods. But I do want it to be held as a field in OgreSetup, just for clarity's sake.
anthy 經常 send_text("")，（沒仔細研究）
造成op仍然為初始值 NULL，op[opN]=0 本身是用來截走過大的mem，所以便會SEGFAULT。( opN是實際字符長度 )
已於直接 send_text("") 下測試沒有其他問題，
看code ***似乎*** out => text會進行strcat(poo, text) // NULL + SEGFAULT的動作，但實測似乎沒有出現......
或許該 *out = "" 之類 一下 ?
還懇請有志之士仔細review 相關 code，不過避免在NULL上操作的動作應該保持。

**anthy也許該修一修?**
Nice improvement, I wasn't thinking about tags when I wrote this.
其实按规范，这个serial要自己记录下来，防止重复。因此要不你就用一个递进的数放在ini里？不过最多只能产生65536个证书……
You're look at intermediary files here, which Make tries to be smart about.

.cf http://www.gnu.org/software/make/manual/make.html#Chained-Rules

"You can prevent automatic deletion of an intermediate file by marking it as a secondary file. To do this, list it as a prerequisite of the special target `.SECONDARY`. When a file is secondary, `make` will not create the file merely because it does not already exist, but `make` does not automatically delete the file. Marking a file as secondary also marks it as intermediate."
Are i and j here sqrt(-1) or are they symbols that lie somewhere along the imaginary line. My understanding is that this sympy command means the latter in which case I don't think these tests show correct behavior. 

Couldn't j be -sqrt(-1) just as easily as sqrt(-1)? In that case we shouldn't make statements about j*oo being + or - I*oo. 

Also, unless the symbols i,j really are defined as sqrt(-1) it would be better to choose different names as each of these is often used to represent sqrt(-1)
Original comment about relative to the template root layout lookup is not actual now. In 2.3 it was right [2.3/layout.rb#L229](https://github.com/rails/rails/blob/2-3-stable/actionpack/lib/action_controller/layout.rb#L229). But now (from 3.0.0) the template is always looked in app/views/layouts folder - [layouts.rb#L372](https://github.com/rails/rails/blob/master/actionpack/lib/abstract_controller/layouts.rb#L372)
Wat is this?



On 1/3/12, Divam
<reply@reply.github.com>
wrote:
> Please revert this commit
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dfordivam/orc/commit/b83427735f875abbc5acee91b870c828e302320c#commitcomment-832812
>

-- 
Sent from my mobile device
yep, nice & neat.
''' process.binding('stdio') '' is now removed as of Node v0.6. It was a private API.
jQuery seems to use toString since it's internal function is generic for any type, including Array (treating Opera array instanceof problem).
Not necessary on zepto.
Then you'd have a `std::vector<int>` with 1, 2 and 4 in it.
i think its maybe better. [myfork commit fb26e26f6e](https://github.com/1devs/zepto/commit/fb26e26f6ed444a447f99ed76238fa501567378f)
because we can get response text in return value and other text methods work fine ;)
what are you think ?
I have placed a table below for now (as a quick fix) similar to the log report, for the current user sent today, going to commit shortly
Is that a *todo* or a just a comment ? I guess since you copy/paste from the PythonQt related block that I wrote I will have to clarify that :p
У меня именно так:

show variables like '%time_zone%';
+------------------+--------+
| Variable_name    | Value  |
+------------------+--------+
| system_time_zone | FET    |
| time_zone        | SYSTEM |
+------------------+--------+

Но всё равно утром ежедневно фризы. Что ещё можно посмотреть кроме mysql?
The service was not ready to work with rest api.
We must improve this.
But I did not understand, what you want to remove? Just this line of ConnectionError?
done: https://github.com/mozilla/zamboni/commit/4d40404
Thanks for the pull request :)
@dimiandre: "oh my god all spaces converted in tab o.o revert this plz -.-" +10
Bad bad, you only changed the asp.net implementation.
Nice.

Glad to see you added the stuff missing from the options-map! You beat
me to it! ;)

/M

On Thu, Feb 16, 2012 at 12:04 PM, Adrian Cole
<reply@reply.github.com>
wrote:
> good catch!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jclouds/jclouds/commit/6c8ff2a9fd06e52b8e60fdd5344da57e6214e727#commitcomment-973360
Jep, saw that too. Getting to grips with GitHub and was a bit eager pressing that button…
А не те же єто грабли, что и с EnterVehicle?..
oh man, comments on my commits always scare me, as i always think theres sth wrong..
Guess those lines you ask, are there for reapplying aoe auras. For example if someone leaves a Consecrated ground, then comes back, he will again be getting dmg. Without these lines, he wouldnt be getting any dmg, once he left the ground, even after coming back. Or not sure what you mean by "why no continue" :p
Эта штука 32-х разрядная. у меня с ней в x64 не компилится. Хоть бы взяли последнюю ноябрьскую версию она уже поддерживает и х64.
Actually we've talked about Rogue's poisons that tick *on enemy* and break *rogue's* stealth as if Rogue have taken damage.
The intent of put(r, e) is to provide a universal generic interface for putting stuff into stuff using a unified syntax. As such, adding policies and options works against that charter. We're better off defining independent functions such as putArray etc.
So if an email is passed in, we return an email. I guess we are planning on expanding this, otherwise it feels like a bit of an echo chamber right now.
Please use implicit line continuation here
@clofresh thanks for catching this. So much for late night coding...
AFAIK preferred macro is `_WIN32`

http://msdn.microsoft.com/en-us/library/b0084kay(v=vs.80).aspx

I don't see anywhere where `WIN32` is defined or recommended. MinGW defines `_WIN32` and for example, in Ruby, there is no single usage of `WIN32` but all `_WIN32`
I think this commit breaks setting up custom logs. I am using Log4j and set up my logger instances in config/environments/*.rb. This worked just fine on 1.0.10, but upgrading to 1.1.13 broke my logging. By the end of Application.initialize! my loggers had been reset back. I am guessing it was this commit that did it. 

I suggest it should be reworded to use `config.logger ||= ...` so as not to overwrite others customizations.
Thanks.. i will have a look on it.

Vivek
kimi kandırıyorsun koçum?
The change is OK but I'm not sure what was the reason to have this dependency there.
test-jar dependency provided ApacheLicenceTest and WellFormedXmlTest. If the build passes then Wicket Security actually didn't need test-jar dependency at all.

@papegaaij : ping ^^
@wilsaj 
1. You are right about the docstrings, thanks.
2. For me, the column thing is occurred in version 0.3 from "easy_install" and the current dev version. While there is column with int type default 0
print **default_value**              -->    ColumnDefault(0)
print type(**default_value**  )    -->    <class 'sqlalchemy.schema.ColumnDefault'>
This is what I am getting rendered "diff

0,Welcome to ,1,vcwebedi,0,t,-1,he,1,'s,0, ,-1,N,1,document,0,ation,-1,al,1,! ===================================== ..,0, ,-1,Libr,1,w,0,ar,-1,y,1,ning:: A,0, ,1,w,0,o,-1,f,1,rk,0, ,-1,Medic,0,in,-1,e,0, ,-1,In,1,progre,0,s,1,s! Th,0,i,-1,g,1,s Sp,0,h,-1,t,1,inx_,0, ,-1,S,0,e,-1,gm,1,xt,0,en,-1,tat,1,s,0,ion a,1,llows a,0,n,-1,d,0, ,-1,Registr,0,a,1,u,0,t,-1,i,1,h,0,o,-1,n,1,r,0, ,-1,T,0,o,1,r c,0,o,-1,lk,1,ntr,0,i,1,bu,0,t,1,or,0, ,-1,(ITK). ITK,1,to,0, ,1,ed,0,i,-1,s,1,t,0, ,-1,a,1,Sphi,0,n,-1, ,1,x d,0,o,-1,p,1,cum,0,en,-1,-s,1,tati,0,o,-1,urc,1,n in a w,0,e,-1,,,1,b,0, ,-1,c,1,b,0,ro,-1,s,1,w,0,s,-1,-platfo,1,e,0,r,-1,m,1, and,0, s,-1,yste,1,ub,0,m,1,it,0, th,-1,a,1,e edi,0,t,1,s,0, ,-1,p,1,th,0,ro,1,ugh a ,0,v,-1,id,0,e,1,r,0,s,1,ion,0, ,-1,devel,1,c,0,o,-1,pe,1,nt,0,r,1,ol ,0,s,-1, wi,1,ys,0,t,-1,h,1,em or,0, a,-1,n,0, ,-1,extensi,0,ve,-1, ,1,r,0,s,-1,u,0,i,-1,te ,0,o,-1,f,1,n,0, ,-1,s,1,c,0,o,-1,f,1,n,0,t,-1,wa,0,r,-1,e to,0,ol,-1,s,0, ,-1,fo,0,r,-1, ,1,ev,0,i,-1,mag,0,e,1,w,0, ,-1,anal,1,s,0,ys,-1,is,1,tem,0,. ,-1,Devel,1, C,0,o,-1,p,1,nt,0,e,-1,d,1,nts: ..,0, t,-1,hr,0,o,-1,ugh ex,1,c,0,tre,1,e:: :,0,m,1,axd,0,e,1,pth:,0, ,-1,pr,1,2 c,0,o,1,nfi,0,g,1,u,0,ra,-1,mm,1,t,0,i,1,o,0,n,-1,g,1, ,0, ,1,push_,0,method,1,s/push_meth,0,o,1,ds web_edit Re,0,l,1,ated pr,0,o,-1,gi,1,j,0,e,1,ct,0,s,-1,,,1, ================ -,0, ,-1,ITK,1,`Numpy's,0, ,-1,em,0,p,-1,l,1,yd,0,o,1,cweb s,0,ys,-1, l,1,t,0,e,-1,a,1,m`_ In,0,di,-1,ng-,1,c,0,e,1,s an,0,d,-1,ge,0, ,1,t,0,a,1,b,0,l,-1,gorithm,1,e,0,s,-1, for,1, ================== *,0, ,1,:,0,re,1,f:`,0,g,-1,ist,0,e,-1,r,1,n,0,in,-1,g,1,dex` *,0, ,-1,a,1,:ref:`modi,0,nd,1,ex` *,0, ,1,:ref:`,0,se,-1,gme,1,arch` .. _Sphi,0,n,1,x: h,0,t,1,tp://sph,0,in,1,x.pocoo.or,0,g,1,/ ..,0, ,1,_Nu,0,m,-1,ul,1,py's pydocweb sys,0,t,-1,i,1,em: http://,0,d,-1,imen,1,ocs.,0,s,1,c,0,i,1,py.,0,o,1,rg/,0,n,-1,al da,1,umpy/Fron,0,t,1,%20P,0,a,-1,.,1,ge/

diff_cleanupSemantic"
I'm not using .nameannounce command at all. But must be completed with all security cases.
a few more:

* bandwagon.views
* localizers.views
* sharing.views
* tags.views
Closed this because I wanted to edit it. See #184
server - client synchronization ))
I had an issue months ago with PEX causing noticable lag whenever a user switched worlds. Pretty happy with bPerms now.
Unless Facebook has something patented about “liking” things, I have a mostly -1 feeling about this change...
Pretty close, just needs a little bit of polish
That’s fast :)
@ozten - when checking out this branch, I am getting an error "Cannot find module '../jwcrypto/underscore'" from views.js.  

In views.js, I had to change line 12:

````
-und = require('../jwcrypto/underscore'),
+und = require('underscore'),
````
can you merge this with line 217?
SBLEU
Nice! Will test next week, since we seem to have a race with fast interconnects and Direct-Send configs anyways.

Cheers, Stefan.
@PheonixKotori: I think what we've been doing (just sending errors as a response to the POST) is fine. It also doesn't need to go to all clients, just the one that caused the error.
This mapping is nice. Would it be much effort to support it also in the other direction if no labelAnchorPointX/Y symbolizer properties are set?
One general comment about this implementation: is it thread safe?

It's not a big concern for now because appengine sends request sequentially. But please think about it.
The connection to onionoo should be made over Tor, we don't want to be giving away too obvious identifications of ooni-probe usage.
k, I'll push
@guilhermeblanco there a way to make this BC until at least 2.3?
This is a comment on my last commit.  Code-review style.
it should? i thought it was advantageous to not define a mode so then the plugin would participate in all the modes.
SPELL_NEW_SUMMON_TEST is undefined confirm
Right, if you use `addEntity`, etc., then it's fine imo to use `entity`.
What about `getBy`, `insertBy` and `getByValue` (all from `Store.hs`)?
Sorry to be a nuisance, but while I was away I thought of two other points:

1. IDNs allow any unicode character, not necessary just letters, AFAIK; e.g. `☃.net` Although I don't know if that counts as a "letter" or not, but I think for the purpose, what you've specified is enough

2. More importantly, you haven't accounted for IDN ccTLDs, e.g. `www.test.рф`! :)
The wizards reacts bad to these missing values, displaying random values for the missing colors in 'inline' mode.  After the save all missing values are replaced with colors.
我说怎么clone慢的要死，原来天启拉了一个分支乃又拉了一个
I believe a workaround is already in place for the lack of placeholder support. A script is loaded that will show the proper placeholder text and not the labels. I'll revert this later.
So that inline JS don't fire uselessly and cause side-effects.
I need it for RailsAdmin, but I think it can be pretty useful for anyone.

It's always useful to set a class on a wrapper as a hook, for 3rd party people.
it might be easier to just accept the pull request ;)
good suggestion, but actually, I'm going to link it to GitHub :)
So, can we remove the ```StringEnumConverter``` from all the serializable enums?
But the fact that the default module can't define imports seems a bit restrictive, I think we should change that. Surely it's possible to make a special module file for default modules. Otherwise the only thing they're good for is for toy programs.
Hi, thanks for the contribution!  I can look at this more later, but for the moment I get:


(django-pydev-venv)mac 09:48:18 (master) >  ./setup.py develop
from: can't read /var/mail/os.path
from: can't read /var/mail/distutils.core
from: can't read /var/mail/pyclojure
./setup.py: line 5: syntax error near unexpected token `newline'
./setup.py: line 5: `setup('
(django-pydev-venv)mac 09:48:23 (master) >  ./setup.py install
from: can't read /var/mail/os.path
from: can't read /var/mail/distutils.core
from: can't read /var/mail/pyclojure
./setup.py: line 5: syntax error near unexpected token `newline'
./setup.py: line 5: `setup('
(django-pydev-venv)mac 09:48:49 (master) >  

after merging in your changes.  Any ideas?

Thanks again,
John
The security council is still debating.  May get a veto from Russia.  

Kelly Knee
Water Resources Engineer/
Project Manager - RPS-ASA
55 Village Square Drive 
South Kingstown, RI 02879-8248
USA
Tel: +1 (401) 789-6224
Fax: +1 (401) 789-1932
Email: Kelly.Knee@rpsgroup.com
www: asascience.com | rpsgroup.com
A member of the RPS Group plc


-----Original Message-----
From: Charlton Galvarino [mailto:reply@reply.github.com] 
Sent: Thursday, February 16, 2012 11:32 AM
To: Kelly Knee
Subject: Re: [GLATOS] Add "-" to the CRSAC deployments so they are not in China (1826aa1)

Did you get UN approval?

--- 
Reply to this email directly or view it on GitHub:
https://github.com/asascience-open/GLATOS/commit/1826aa17b2a790a9abb136804a607772cb1a3804#commitcomment-974275
Yes please, or even better, refactor the two functions into one (although this might be a challenge).

There is no good reason for them to be split, although the reason they are (which could be worked around) is that one of the sentences needs to have no hyperlinks, whereas the other does.

So if you can address that in a generic way, we should be able to combine the functions into one.
You need to include either [softdepend: [Vault]] or [depend: [Vault]]
Until we have this working within a generator, you must execute it from your main app with: 
rake spree_warehouse:db:seed_roles
It seems like jasmine is ok with it, but jasmine-headless not :(.
+(7612,12,0,0,''),
architecture, how does it work, what is filtered, when is it triggered. Mention it covers both DQL and Persisters aswell as lazy load and collections.
I thought this fixed the bots' not being able to login. However, I was wrong. They still timeout.
im not really sure what i did there ..

could not debug it but somehow these 2 lines blocked every seasonal quest for like half of all players (even if they never did them before) :| commenting this just got these quests back, no crashes atm
Yes.  I'm relying on the assumption that chunkHeaderSize is a multiple of 4 - not necessarily true.
@icambron: It will [have the line number prefixed](https://github.com/jashkenas/coffee-script/blob/806df9bc1eea4d7d10011c3b4fb848fc1815e0e4/src/lexer.coffee#L534). Isn't that sufficient?
I'm not completely sure whether that issue is resolved now.  I've just recently had a couple of successful builds without needing to rerun, but #8 was somewhat sporadic. I also changed the binutils command sequence a bit below which might have something to do with that failure.

I'll close the issue when I either get more feedback that it isn't occurring or I haven't run into it after some number of builds.
시험 삼아서 올려보기~
ok Cool, so in that case, this is correct, I am going to open a pull request for that, so it will be added to the Github services. (unless you object of course)
I'll pull this note in for now @jukka but ideally it will be something we can address in the configuration xml.
These are the changes:
+ Changed the Manage Integrations: Shows tne existing integrations, you can select one to edit, view, run, delete (Under construction operations)
+ Changed the last wizard arrow (Some issues with an upper line)
+ Changed the last operation to Save Integration

Working: Select Operation Logic, code for insert, update, upsert
This is awesome! When does that come out?

On Mon, Jan 30, 2012 at 5:26 AM, Ross Lawley
<reply@reply.github.com>
wrote:
> This will be easier with mongoengine 0.6:  https://github.com/hmarr/mongoengine/commit/0301135f96be83ee613664e875409a9d4b590137
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pydanny/django-mongonaut/commit/52a1d7a5df8aec4af45fc4d4e502c17d049ad9ee#commitcomment-913722



-- 
'Knowledge is Power'
Daniel Greenfeld
http://pydanny.blogspot.com
https://github.com/coyotte508/pokemon-online/commit/a8a78bfc2cda0ad3c719ac79513279c9d154ecce#L10R6
I don't think the format page should be as a subpage of the command page. Should we change this?
I've removed metadata.json from my repo (that's the only file responsible for errors in puppet).
Also it's now a parameterized class.

Feel free to try it out. If you need anything, please let me know.
A couple major issues with this pull request

1. What you submitted is *not* a pull request. It's some comments that could be turned into issues. Feel free to submit them in the django-mongonaut issue tracker.
2. I can't accept a pull request with comments in a language I don't speak.:P

Regards, Danny
Organized this a little differently, the anchor tag is all of the content inside the `<td>` element but does not wrap around. 
* wrapping `<a>` around `<td>` makes it unclickable.
In the remaining you have `Leave database and model work for much later (if ever).`

But we have Aura.Sql ?
Haven't used this comment feature before. Perhaps I'll use this as an opportunity to express a bit of dislike for csproj files. Though, xib files at this point in time seem worse.
We also are going to be upgrading the JSON for the places from the 1.0 form to the 2.0 form, right now we make them go to the About options page and manually transform the data, we could possibly integrate these two steps. I will file an issue related to this.
I could have sworn I fixed this... oops. Thanks for the correction!
Wait, we have camera on ICS now?

On 16 February 2012 19:22, Lalit Maganti <
reply@reply.github.com
> wrote:

> I know - but the Skate doesn't work with the CM7 libcamera source. It
> crashes the camera app on gingerbread and crashes the phone itself on ICS.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TomGiordano/kernel_zte_blade/commit/c02c7dfddf7f05ff8e19c0012248507e51d1edee#commitcomment-974912
>
Sorry, this should be without semaphores.
since I can't use getActivity().getSupportActionBar() under Sherlock*Fragment, would you add like

```java public ActionBar getSupportActionBar() { return mActivity. getSupportActionBar(); }```
Hi Ulli,

i couldn't reproduce your problem. When i make a purchase, the purchase screen of android market will be shown, when i press the button "Accept&Buy" and choose the "FAKE Creditcard", i will go back to my app and 1-2 seconds later, i see "purchaseState = PURCHASED" in log. Did you try with a real item or just tke fake item?

An
Shouldn't an error here be more than logged?  How do we even alert the user that something bad has happened?
Yes, this hasn't been done yet.
accept以外の時でも accepted_answer が 変更されたら、エラーでもいいのでは？っと思ったのですが。

accept以外で、accepted_answer以外は変更可能である必要はあると思いますが。
It is already enabled :(
Ahh, I didn't know that this compiler bug can be circumvented by static imports. Then again, this must be how we worked around the problem in Search as well.
todo: instead of using gnome-terminal, look into using a cronjob and zenity
This is the only line changed/added from the default ueventd file
num tindioqsefalo
Well, adding r/mylittlepony to the script doesn't play nice with the MLPlounge...Shoud it be in the supported list?
i tested and all channels work %100, but some dvb-s2 transponders 30000 3/4, you should change transponder  to 29998 3/4,
The dispute is quite clear up to this point. It is on the role of the acceptance status, namely, whether it should be more visually catchy (=red) or more informative (=green). I prefer the red color because it offers the most prompt notification about the acceptance, and to some extent magnifies the "sense of achievement" of students. And your choice of green "accepted" status leaves the students' extra attention to unexpected (i.e. compilation) failures. If I should say, making such a choice is more of personal flavor than philosophical / cultural correctness.
radio...! could it be?
This function reads nicely.
Thanks for making this public and especially for sharing precompiled .txt files.
I tested this release on my Analog Watch. It sometimes reported 0.0V battery but that seemed harmless.

One big issue though. All notifications that don't fit the display (scrolling messages) make the display freeze. (Which sucks battery quick)
We had this kind of error before when scrolling was new. Did this perhaps revert a old fix?
My bad, I imagined whole mechanism slightly different and got lost. I thought of additional data being got by additional query and this way we would in fact pass 2 rows to the constructor. The second one (i.e. row from get_object_entity_as_row) is what I called "additional table data". Problem is that here we may this data already merged in get_data output and this way it's hard to treat it separately. Ok, forget it. I'm just concern about the test below. What if someone removes/renames "title" column? I don't see how to do it better though. Maybe we need more explicit way to get expected attributes list with distinction of source tables? This way we could make diff of entities table columns from available ones and see if the rest is consistent with what we expect (to decide if we need to fetch data ourselves or may use what we got).

Second thing I'm referring to is idea to extend "secondary table" concept for types to subtypes as well, i.e. as common metadata replacement for files (sth like content type hold in table like objects_entity_file) and pages (sth like subtitle/abstract hold in table like objects_entity_page). But it's rather thing for future. Very good for complex search contidions though.
One small note: in a comment block would you mind explaining the change you have made so it's easier for adopters to understand?
"Access" mis-spelling again
case ((_, sensibleName), (_, otherSensibleName)) =>
  val newSensibleName = sensibleNAme ++ otherSensibleName
Isn't there a MurmurHash in Scala libs? In the nightlies there is a scala.util.hashing.MurmurHash3. Is ours customized somehow?
How much slower is the test now on average?
absolutley
Weird if the test passes ;-)
private is not visible for Java afaik
private[akka] is another thing
otherwise there will be a lot of those comments
this never worked did it?
Frankly I dunno. I would like to defer improvement of that code to further PRs.
strictly speaking the context user is often c.user or c.author which can be an ip address but for now maybe just add a fixme and we can lookat doing a better job when the 2939 branch merges as that has helpers for this sort of thing
Same issue as with the RPM extension common. Unless I'm reading this wrong, it's going to conflict with Puppet Common which also owns this file.
Replaced with:

:doc:`Consumer <../consumer-client/introduction>`
The duplicate changelog entries for 0.1.9-1 probably isn't good but don't think it's a blocker.  rpmlint won't like it ;)
when are any of these values `NaN`? [this rule](https://github.com/siq/gloss/blob/master/src/widgets/powergrid/powergrid.css#L11) should prevent that.
I think this gives us all the associations we could need and uses fetch on cards.
или ты про строку 207?
java does autoboxing
Images are created solely for the purpose of displaying raster data. Here, you are _modifying_ a data-inherent raster data property 'log10Scaled'. This is a very bad idea.
Erubis is being used by the setup command to parse various skeleton.rb files. So: erubis is a runtime dependency for Harbor itself. Without it 'harbor setup' fails.
Generally speaking, we start the indentation level off at zero in implementation files (even inside namespaces) - please de-indent this function.
It depends whether you want to make life as easy as possible for people 
writing rebar configs. Should they have to always type the ..._test or 
..._test_ part even though it is obvious, and should they have to 
remember if it's a plain test or a generator?

    /Richard
Can we do this search at sql object creation time rather than on first miss? 

Something like "oh, here is a method that is not annotated, and not a 'known' like toString, let's do a search"
This is not compatible PHP 5.2, as `static` keyword was first introduced in PHP 5.3...
why creating a new function calling the PHP built-in function with the same arguments (and wrong ones as the second argument of ``json_decode`` is a boolean, and it accepts more) ?
Did this comment get truncated?
Just args[0], no need to cast.
Can you turn this into a simple sequence of `exports.EFOO = 'EFOO';` statements?
Indention looks wrong on github. You probably used spaced, while the rest of the script uses tabs.
No reason. I've moved it as suggested - thanks.
This is now a very large, complex method. Possible to break it up into smaller, more digestible pieces?
Yeah, you're totally right, I was not sure if render_to_body handles also some other cases that need to also have rendered_format cleared
Nope :) I forgot to remove it when I refactored this part
Possible to implement using `update_all` ?

```ruby
def touch(name = nil)
  name ||= :updated_at
  now = klass.default_timezone == :utc ? Time.now.utc : Time.now
  update_all name => now
end
```
We just upgraded from 2.2 to 3.2.
Not saying I'm an expert or anything but maybe...

```ruby
formatted_start = '"#{start.to_formatted_s(:db)}"' if formatted_start.acts_like? :string
formatted_stop = '"#{start.to_formatted_s(:db)}"' if formatted_stop.acts_like? :string

"BETWEEN #{formatted_start} AND #{formatted_stop}"
```

What do you think?
Can you indent the examples? :smile:
This seems like an awfully macro-specific method to a) have such a generic name and b) be a member of `Types`.

A general point: Prefer final classes and class members. It's much easier to read if I know there can't be an override hidden in a subclass.
well, okay, but when should one use `zip` then if `zipped` is always preferable?
That was the typical  drawback of default subtypes.
I'm not sure what you mean.  I'll I did was replace `l` with `lamsol`, since dsolve() returns an Eq(), but this expects the solution of lambda.
On 17.04.2012 11:14, Christopher Smith wrote:
>> +        That is, compute the ideal generated by product `xy` for `x` an element
>> +        of ``self`` and `y \in J`.
>> +
>> +>>>  from sympy.abc import x, y
>> +>>>  from sympy import QQ
>> +>>>  QQ[x, y].ideal(x).product(QQ[x, y].ideal(y))
>> +<x*y>
>> +        """
>> +        self._check_ideal(J)
>> +        return self._product(J)
>> +
>> +    def reduce_element(self, x):
>> +        """
>> +        Reduce the element ``x`` of our ring modulo the ideal ``self``.
>> +
>> +        Here "reduce" has no specific meaning, it could return a unique normal
>
> replace first comma with colon?
>

done
This regex now represents a valid Python variable whereas the former version did not.
Oh right, thanks, I'm just used to the math too much.
On Fri, Mar 23, 2012 at 9:38 AM, Chris Smith <smichr@gmail.com> wrote:

> > -    return _keep_coeff(coeff, term*f.as_expr())
>> > +    return _keep_coeff(coeff, term*f.as_expr(), clear=clear)
>>
>> your doctest has a clear=False, so a positive case should be clear=True,
>> unless you plan on having multiple levels of true...
>>
>
> Sorry...I'm not following. I need to pass along to _keep_coeff whatever
> the user passed for clear.
>

If they don't pass anything then clear defaults to True and I wouldn't need
to pass along anything to _keep_coeff (since the default for it is True),
but if the user calls terms_gcd with clear=False I have to pass that along
to _keep_coeff.

I made the other changes and am re-running tests.
What about leaving a blank line before the closing quotes for the docstring? https://github.com/sympy/sympy/wiki/Writing-documentation says this is recommended, though nobody raised an objection. Is there a new sympy policy on this issue? 
On a separate note, the References sections of docstrings should also be labeled as such, according to https://github.com/sympy/sympy/wiki/Writing-documentation ; I'll fix that too in the amended week1 branch.
Added Git and hg support
On Wed, May 23, 2012 at 9:19 AM, Robert Martone
<reply@reply.github.com>
wrote:
>> +            $document.off( 'mouseenter', mouseenterEvent );
>> +            $document.off( 'mousemove.lu.tip' );
>> +            Tip2.hide();
>> +          }
>> +
>> +        } );
>> +
>> +        Tip2.show();
>> +      }
>> +
>> +      //Event Listeners
>> +      Tip2.on( 'mouseenter', mouseenterEvent);
>> +
>> +      Tip2.on( 'focus', function( event ){
>> +        event.stopPropagation();
>> +          Tip2.on( 'blur.lu.tip', function( event ){
>
> We should decide on a convention for this.
>

For what -- the event namespacing?


-- 
jimg at yojimg dot net
Woah, how'd that sneak in there! I'll fix that...
needed to support other possible directories for bash
This indentation is wrong. Arguments should be lined up.
Agregámele el punto que te faltó y sale :P
This is super lame.
Sorry, I missed style. Fixed.
don't know why I put that there
OK I can change those to my{}
"As the index continues to divide the coordinate plane"

(maybe its more correct to say "As the index's representation continues to divide the coordinate plane.")

Basically, we need to avoid the passive here, because who/what is doing the dividing is unclear.
`video` also needs to be removed from [the list that appears a few lines after this](https://github.com/openjck/kuma/blob/7a30808768c4f1749cf00028eb783c655b6b7b16/apps/wiki/models.py#L73), and the attributes from that list merged into this one. Otherwise, these new attributes will be overwritten.
There are whitespaces between `[` `Settings` and `]` which doesn't match `[Signout]`

See http://awesomescreenshot.com/0bfqc7q71
Document which normalizations are performed?
Use an iterator instead, we are on PHP 5.3. E.g. `DirectoryIterator`
Use proper access modifiers
This seems like a much more user-friendly UI for all the cases when users would actually want a delta rather than absolute ending time.

Perhaps you can allow both input formats and figure out which one was used automatically.
This is almost certainly going to result in a full table scan. At the same time we have indexes on both user id and author id. Would it be better to issue two separate queries each hitting the respective index than this one query doing a table scan?
I think the best approach would be to add `$result['avatar_type'] = $driver;` and then do array_merge() as `$result` is filled with data by the avatar driver.
As `request_var('backtopost', 0)` is used second time here, maybe you should think of assigning variable for that and use variable instead. Something like `$backtopost` or `$back_to_post` could be ok.
Do a function_exists() check instead of *_once() methods.
fixed in 6339e95
`callback` should be the second argument instead to follow model pattern.
I think we want Y.JSON here.
I did an experiment:  we don't need to specify ATTRS.  I will remove.
should this file be in this pull request?
naming boolean values... often something like:

'isOutpatientDoctor' is preferred to name them. So it's clear that it's a "question", and the boolean value of "true" or "false" is the response.
```
In [2]: timeit numpy.empty((1000,1000))
1000000 loops, best of 3: 549 ns per loop

In [3]: timeit numpy.zeros((1000,1000))
1000 loops, best of 3: 659 us per loop

In [4]: a = numpy.empty((1000, 1000))

In [5]: timeit a * a
100 loops, best of 3: 2.17 ms per loop
```
It looks like the initialization is about a fourth of an elemwise operation. Not a big deal I suppose. The recv-wait should block until it fills this space though, I've never known this code to fail. If this was my project I would leave it empty, it sounds like the Theano community might really care about determinism though. Given the timings above should I make this change?
Yes. If there are no partitioned tables in the statement AND it's not DML, we go on to the next statement with no change of status (which initially means procHasCommonPartitioning == true. Thus the comment:
"// Usually, replicated-only statements in a mix with others have no effect on the MP/SP decision"
Whitespace between methods would be nice. CompactingTreeUniqueIndex.h has the same issue.
Crap. This is missing the opening <code> tags. My fault too.
Ning/Ariel Good comments - will do this.
firstLine set but not used?
"Testing only this package" vs. "To test this package"?
うまく動いたので再度コミットしました。pull request は一回しか出来ないんですね。
its needed if device is not created yet
Since the failure returns false, shall the success here return true instead of nothing?
Please split out initialization of nodes, it is slightly confusing.
notifies small typo, don't care
assertQuerysetEqual ends up being mode verbose, since it requires a transform function to get the name

self.assertQuerysetEqual(new_art.categories.all()], [], transform=lambda o:o.name)

self.assertEqual([o.name for o in new_art.categories.all()], [])

Without the transform, it fails with:

AssertionError: Lists differ: ['<Category: Entertainment>'] != ['Entertainment']
I believe this line should be changed as well.
On 08/31/2012 10:54 AM, Lars Butler wrote:
> In openquake/calculators/hazard/general.py:
> 
>> @@ -534,6 +535,22 @@ def parameterize_sites(self, site_list):
>>  
>>          return jsite_list
>>  
>> +    def initialize_pr_data(self, **kwargs):
>> +        """
>> +        Record the total/completed number of work items for the classical,
>> +        event-based and uhs calculators.
>> +
>> +        This is needed for the purpose of providing an indication of progress
>> +        to the end user."""
>> +        sites = kwargs.get("sites")
>> +        assert sites, "Invalid 'sites' parameter"
>> +        realizations = kwargs.get("realizations")
>> +        assert realizations, "Invalid 'realizations' parameter"
> 
> Same question for |realizations| as with |sites|.
realizations is an int value and it needs to be set and non-zero
good point, hadn't thought about this.

Sent from my iPhone

On Sep 19, 2012, at 5:57 PM, Matt Priour <notifications@github.com> wrote:

> In lib/GeoExt/widgets/WMSLegend.js:
> 
> >          var layerNames, layerName, i, len;
> >  
> >          layerNames = [layer.params.LAYERS].join(",").split(",");
> >  
> >          var destroyList = [];
> > -        var textCmp = this.items.get(0);
> > +        var textCmp = this.items.find(function(item){
> You could, but if hideTitle had been specified or if a derived class is used that puts the legend or layer title on the bottom, rather than the top, then you can't be certain that the layer title label element is the first in the collection. By using find, you unambiguously get the first label or label derived component.
> 
> —
> Reply to this email directly or view it on GitHub.
>
We should apply that methodology to all tests in this PR too.
I think that parse function should not be an overloading, but rather something like:

```
parseSpaceSeparated(s, Int, String, Int) : Int, String, Int
```
Yeah, I cringed at that, but if I started on grammar I'd be busy for a while.
More specifically, the jQuery( this ) .remove() needs to come before appending for the other node types that replaceWith can be called on. The fact that this doesn't break a unit test only shows that our unit tests aren't covering enough cases for replaceWith... I only know this because of the work done in https://github.com/jquery/jquery/pull/924

@dmethvin, @rwldrn, thoughts on correct behavior here?
Because we use the file resulting from the build process now? ;)
Args sorry
If `jQuery.ready();` is called in either condition, why not call once at the end? Is the `document.readyState !== "loading"` a hard requirement?
Pass the 0 for consistency, compressibility, and maximum browser support. And I tried a pull request, but couldn't select your repo. Just cherry-pick 775f634ccf2e2b1ffc767a38a436d8e51ec45c44.
Can you flesh this comment out?
Could you provide a link to the upstream issue for this in a comment? Or if it's fixed (which I think you mentioned it was?) a note to that effect and that the patch can be removed in the next version.
On 29 Aug 2012, at 4:14 AM, Jack Nagel wrote:

> No sudo

That’s difficult - backupninja must be run as root and "brew test backupninja” fails if sudo is not used.

I agree using ‘sudo' within a formula is an awful idea .. is it acceptable to have a failing test in this instance ? 

Other recommendations ? 

Ciao,
Sven

Sven Agnew: Independent Open Source Integrator 
Read my blog on Tumblr - Conversations
Contact | mailto: sven.agnew@gmail.com | mobile: +27-78-512-6996
Is this meant to be an assignment or a comparison?

If it's an assignment, I'd suggest wrapping in parentheses for clarity or moving it to another line:

```ruby
@current_user = current_user
redirect '/login' unless @current_user 
```

or

```ruby
redirect '/login' unless (@current_user = current_user)
```
> mri_surf2surf has some way to map from an (approximate) fwhm specified in mm to number of smoothing iterations.  Would that be feasible to bring to the smoothing algorithm we have to give a more intuitive parameterization of the label generation? (and the smoothing itself I suppose)

hum. Maybe but I have no clue. Basically each iteration is the size of
an edge. How many edges correspond to a given fwhm? maybe @satra
knows?
If we're going to rename it anyways, why not `pending_manual_download`?
While I agree with you, I'm more concerned with the core functionality of this pull request and don't want to burden @logan with extra work. We can move the information around after the fact.
> Когда я вижу "сливать одну ветку с другой в течение большого промежутка времении"

Там написано «сливать одну ветку с другой <b>несколько раз</b> в течение большого промежутка времени». Без выделенных жирным слов, конечно, смысл меняется. С этой точки зрения «в течение» и «на протяжении» эквивалентны, и замена одного варианта другим не спасёт от ощущения медленного сливания.

Слово «периодически», кстати, хорошо вписалось бы в предложение вместо потерянных «несколько раз»:
> Так как Git использует простое трехходовое слияние, периодически сливать одну ветку с другой (в течение|на протяжении) большого промежутка времени...
isn't that already in test_timezones.test_asobject_tz_box?

On Sun, Jun 3, 2012 at 1:17 PM, Wes McKinney <
reply@reply.github.com
> wrote:

> > @@ -540,7 +540,9 @@ def asobject(self):
> >
> >      def _get_object_index(self):
> >          boxed_values = _dt_box_array(self.asi8, self.offset, self.tz)
> > -        return Index(boxed_values, dtype=object)
> > +        idx = Index(boxed_values, dtype=object)
>
> This isn't actually what I meant in #1345, only to test that the Timestamp
> or datetime.datetime values in the resulting object arrays have the right
> `tzinfo` set
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pydata/pandas/pull/1383/files#r919922
>



-- 
Chang She
Lambda Foundry <http://www.lambdafoundry.com>
ich tests noch mit non-admin.. grundästzlihc is das von developer übernommen, insofern sollts passen
Ah. Nearly missed that tucked in there.
When you bring this change back to life, could you combine these classes all into one file? Putting everything together might make understanding the tests easier.
I kind of didn't like the shell widget. I tried to modify it somehow, tried 
for instance to implement right click menu with "paste" and "copy" but I 
couldn't get it to work the way I wanted so I replaced it with windows 
command prompt shell. And SettingsWidget::chooseFontTables() dialog became 
useless.
Sorry, this is my first attempt in programing. I'm learning as I go.

BTW are you bulgarian?

Nick



-----Original Message----- 
From: triochi
Sent: Thursday, March 22, 2012 5:40 AM
To: nijel8
Subject: Re: [QtAdb] Just committed my changes... (#6)

> -    tmp = QFontDialog::getFont(&ok, QFont(this->fontShell), this);
> -    if (ok)
> -    {
> -        this->fontShell = tmp;
> -        this->ui->editFontShell->setText(this->fontShell.family());
> -        this->ui->editFontShell->setFont(this->fontShell);
> -    }
> +//    bool ok;
> +//    QFont tmp;
> +//    tmp = QFontDialog::getFont(&ok, QFont(this->fontShell), this);
> +//    if (ok)
> +//    {
> +//        this->fontShell = tmp;
> +//        this->ui->editFontShell->setText(this->fontShell.family());
> +//        this->ui->editFontShell->setFont(this->fontShell);
> +//    }
>  }

Why you are removing font dialog?

--- 
Reply to this email directly or view it on GitHub:
https://github.com/triochi/QtAdb/pull/6/files#r589927
I think to make this more clear, you could just do
```
this._attachmentPath = '';
return this._attachmentPath;
```
here
I'm struggling a bit to get up to speed on what's going on here.  It might help to get some docs on each of these fields, for a start?
Since you are fixing up the Javadoc, perhaps this should say "Sorts the test methods in a deterministic, but not predictable, order"
Good catch, thanks. It got lost as I was trying to use `getPages`, which checks for you.
Nit: indentation.
Nevermind. I can see that there are none in this entire module.
If the goal is to merge this into mainline, we should do an actual browser vs. brackets-shell check here
I think it actually makes the most sense in KeyMap or KeyBindingManager... probably ok to leave it here for now though.
Should be just ``{Sort}``
Sorry, I should have said: `$(".codehint-menu.open")[0]` -- if you're calling raw DOM APIs you basically need to "unbox" the jQuery object.

If you wanted to make it a little cleaner for the callers, we could have simulateKeyEvent() take a jQuery object (i.e. change arg to '$element') and then do the `[0]` unboxing step internally.
Good point. I haven't tested it, and don't currently have access to a Windows machine with multiple drives. If this _is_ a problem, it would also affect extension loading in brackets.js. 

Would you be able to test this? If it is a problem, it should probably be logged as an issue since it would affect general extension loading too.
missed a space!
exceptions_to_ignore, exception_passthru, exposed_errors, exceptions_to_skip, uncaught_exceptions, Chanko.config.expose_exceptions, expose_errors, unhidden_errors, bubble.... 

Good luck :thumbsup:
should this be `StringProperty(default="enddate", choices=["startdate", "enddate"])`?
You should know that these definitions become global in Ruby, and notably pollute other tests. Either use `subject` that gets cleared every time or move the classes into `spec/support` with a better defined name.
I like to do only the assertion in the ```it``` block, but I'll change it :)
The tricky part. Does field default function take `request` as an argument or not?
Mind breaking up this line? It's getting super-duper long.
could you maybe take this out
prefix these with an underscore for consistency.
don't use direct style modification, use classList property here or on parent element
I made a similar comment somewhere else which I won't fully repeat, but, don't do this!
You won't get a traceback for the option if there's no dot in it. Only that in such case, STATE_FUNC and STATE_NAME will be identical and that the renderer will then look for states named by STATE_NAME or STATE_NAME.STATE_NAME(or STATE_NAME.STATE_FUNC). The first case is ok, the second is probably not what the user want.

Anyway, there is a real problem with the code and that is the code should be in the module's __init__() instead of at the module level so that the globals would be set to the overriden values. I just fixed this.
dynamically loaded
Tässäkin voisi näyttää lomakkeen?
Indentation is out of whack here and below
Let's add a UtilSpec with tests for these methods
Since you're doing the same for all of them you can just catch Throwable.
Do you think it would make sense to add a model function like "isModelProperty", which would basically return

    (k === 'id' || this.constructor.schema.definitions[this.constructor.modelName].properties[k])

? I believe the purpose is just to limit the looping of the property keys to those that have are actual database fields.
This is defined in katello-configure/bin/katello-configure as well. Can we have only one instance of this code?
Can you rather call it as:
  system('/usr/bin/katello-service',  'stop')
This way it is not executed via shell and is executed directly.
The issue is that not all formats have to be file-based in theory (e.g. databases) so ``overwrite`` might not always make sense.
I thought of something like that...but currently this *only* really applies to the astropy package itself--the releaser hooks are pretty generic, actually, but they're meant only be used when releasing "astropy", so there's not much room for reuse by affiliated packages (that could be changed though, if we have some reliable way of identifying if a package is an astropy affiliated package).

So in other words, rather than have affiliated packages provide their own releaser hooks (though they could if they wanted to), it would be nice in the future to make these hooks generic enough to work with any astropy-affiliated package.  I just think that for now it's not worth it, since the automated release doesn't do *that much* yet.  Once we start adding things like doc uploads and things like that, it will become more worthwhile.
I think the last value should be ``1`` not ``1.0``
I think this should be ``isinstance(x, basestring)``
Ditto here on super().
Typo: `maxiumum => maximum`
It is harder to see that the if()s belong to an else this way, while reading the code quickly, as they start on their own line just like stand-alone ifs.  Also this is style is used nowhere else in the source.
I think it'd be better to ship the patch in contrib/homebrew/makefile.osx.patch file rather than put it here.
In rspec I think the standard is to use the word "should" instead of "will".
I assume you've made a typo, putting down "globals.php" twice?

If it's manageable to use it like it is in Symfony it might be worth a go.
It's hard to prioritize what to do first. :)

Would also love to read feedback from @tijsverkoyen, @dieterve and @dubgeiser
If `show = TRUE` it's not run, just commented out. If `show = FALSE`, it's run.  (These params correspond to the Rd tags not what `run_example` does)