labofsahil/github-event-dataset-2012 huggingface dataset:
They're same for now, but will likely change as we go along.

-viral



On 26-Feb-2012, at 11:36 PM, Stefan Karpinski wrote:

> Is it just me or are the LDFLAGS values all the same?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/d3d30e854bc622e3cc8b2f299b2b017b0e7d7b30#commitcomment-1012705
@richarddong @huxiuhan ok~
I don't really understand the 'case' in the pattern. 'case x && *x:' is no valid code, imho.
Why? If this is for your own personal changes, you can add this line to your **$GIT_DIR/info/exclude** file (that one doesn't get uploaded to the central git repository) instead of committing this file.  See `man gitignore` for details.
Points from IRC chat:

* There is benefit in the name->id cache since it saves a database lookup (negligible) but, more importantly, prevents scattering Strings in the code
* Having inspectors on state would be awesome, ala `delivery.queued?`. You can do this with delegators (so that `#queued?` delegates to `#state.queued?` which is basically just a [StringInquirer](http://apidock.com/rails/ActiveSupport/StringInquirer))
这里的这些嵌套的if，能不能单独搞成一个Page类的方法，专门用来判断某一个用户是否有权限做什么事情
The overhead is not all in `write`:

```
julia> m=memio()

julia> @time for i=1:100000; write(m,'a'); end
elapsed time: 0.005269050598144531 seconds

julia> @time for i=1:100000; with_output_stream(m, write, 'a'); end
elapsed time: 0.01262807846069336 seconds
```

`with_output_stream` more than doubles the time without doing anything fundamentally different. This is simply a design/performance point that hasn't been addressed yet.

I'm sorry. Before, the justification was that we might as well implement the full printf spec. I disagree with that, but ok, it is definitely defensible and I can go along with it. It has value. But now you are just making up extra complexity as you go along. Emitting code to check for nan and inf for every occurrence of %d is not sane, and I might as well just go make a large donation to the PyPy project. Nobody needs %d to handle NaN.

At the end of the day, if it performs well with the nan checks I guess I don't care, but we're not there yet and we have to wonder how long it will take. Seriously, all people want is the ability to type `"%.3f"` and print some columns of numbers. This is getting way out of hand.
OK,got it...
在 2012-01-11 19:39:09，lvzongting <reply@reply.github.com> 写道：
>我的那个项目已经移交到 github.com/xdlinux  欢迎加入这个组织。
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/Mrdoooger/xdeat/commit/bb222157da4982dcde506e0346778bdf5367efed#commitcomment-856699
So each connection is going to spin up 6 sockets? which means 6 port numbers too right?
after compilation in GCC, code of x/100.0f fully equivalent code of x*0.01f, then note not have sence...
@Faq I feel the same when people write in russian.
@raynar https://github.com/mangosR2/mangos/commit/dd97052b7b8ee0f1f5f5a038347904362eb5dc32 must fix most of this issues.
is that category hack there ok? With it you cant cast bladestorm after whirlwind, but you can cast whirlwind while bladestorming. WIthout the hack, you can cast bladestorm right after whirlwind.
Tried process.kill(), process.terminate(), and process.send_signal(), none seem to be terribly effective.

os.kill() works correctly, but requires Python 2.7 for Windows support.
Ditto, justification.
Planning to move all this awsome stuff to mangos/mangos master?
Sure I'll comment and test properly. Thanks!
Just got the dynamic form to add employees to a project working, try not to break it.
If any appointment has more than one phone number, the "single column" for both is going to be the width of two phone numbers anyway.  If you have only one phone number with your current implementation, there will be a big blank space to the right of that phone number. With two columns, you'll have roughly the same amount of blank space, except it'll be on the left some times depending on which number is missing.
I see

```gherkin
  Scenario: Saving a Search when not logged in
```


The use case here is there is no authentication provider so users can't sign in and searches can't be saved.

Maybe it makes sense to add variations of

```gherkin
  Scenario: Have No Searches
```

and

```gherkin
  Scenario: Have Searches
```

I think the new scenarios would need to run with `has_user_authentication_provider? == false`.  Would that be very complicated?  I'm new to ruby and have no idea.
will address the various comments and recommit momentarily
Something in your repo that's ignored by this one?
yes, this was hardcoded until I got things working building properly. I had to guess the values these fields should hold as macos does not have a parseable launch script like linux does. This will be changed before being merged upstream.
indeed seems like there's something called `gizmo` http://unity3d.com/support/documentation/ScriptReference/Gizmos.html http://www.rotheblog.com/2005/07/design-topics/what-is-a-3ds-max-gizmo-how-do-i-turn-on-or-turn-off-a-gizmo-in-3ds-max/

What about `CameraTool`/`AxisTool`? `CameraHelper` feels a little like `CameraControls` to me..
... you may have hit the delete button a little hard there. private messages and the contacts page seem a bit broken, layout-wise.
most likely not
"författare" should be changed to "skribent" since "författare" is the term used to describe book authors.
do we need the method_exists check? I would assume that instanceMethods only returns if it exists.
bootstrap.responsive should have a license header on it
We claim to support 1.8.7 .. this is why <http://travis-ci.org/#!/highgroove/zonebie/jobs/825862> is failing.
If job does not have any builds self._data[buildtype]["number"]  raise "TypeError: 'NoneType'  object is unsubscriptable" 
Return "None" instead.
Эта миграция уже в продакшене. Так ее изменение никакого эффетка иметь не будет. Это бессмысленно
I definitely should do that, yes. But when it's a hot fix related to a previous commit, I want to push it as soon as I can in order to avoid being spammed by forum messages, pull requests and new issues in the tracker.
Why? It is just internal implementation. Kind of private...
Please remove outermost parens.
I kinda did for single values, but I'm removing that in the next commit..
Oh yeah :) Effectivement, thanks :)
Do you mean the function name ``onDestroy``?
# adapt or die, yes
# but only where there's some win

What's the win here other than a minor shrinking of lines of code on an execution path that is not related to performance?

If there were some significant win, then yeah, sure. But since there are plugins out there on which I don't have a commit bit...meh

Also: Have you been with the ruby too long?
Okay, that is a cool addition.
Isnt this Taking Olloths Commit No where?
There is no need for checking if projects are null
as we spoke about on IRC those 'fixes' are workarrounds till beans gets arround fixing the dump
> If you have a filename with a slash in it, e.g. `Image_Big/Small/Medium.pdf` then its actual name is stored on the computer as `Image_Big:Small:Medum.pdf`.

Interesting. I had no idea. I thought it would refuse `:` in the filename under any circumstances.

Anyway, I created a test file named `Slash/Name.pdf`. It still works as you describe after my change.

name: `Slash/Name.pdf`
details: `~/Slash:Name.pdf`
Argh, my bad.  Sorry.  :(

I missed out the `git add src/base.lua` step between `zile src/base.lua` and `git commit`, to make sure your edits were staged prior to committing.
Quick search through docs was really missing. Thanks John.
I tested it as well and found 2 issues :

- Adding a `default` Connection for the sqlite database conflicts with app's default connection if exists. Should be named to `li3_docs`
- Clicking search results is broken (redirects to the keys typed and not the harvest symbols). Hitting keyboard enter works like a charm.
That would be a new pull request, please post it explaining the benefits.
Yay! Thanks, Jer!
i have seen it ;( i will look at this
Sorry I think this is another [DOCS]?
I added it to prevent the risk of faultiness (as in someone tunes a regexp and added by random utf8 chars (which should be wrong already) but still fires that back, thanks for the reminder, tho. In this case the compilation process is no matter :)
need to clean up view file once i;ve finished playing around
Just a question for the syntax. 
Why do you prefer to use spaces than tabs?
Why the trailing comments ? //
Why have you flipped the default implementation ?
Tried both `imap` and `inoremap`. Same result: silenced AutoPairsReturn, but also killed its magic.
Es sind keine Lokalen variablen das Strimmt, aber es ist "the next best thing". Sie befinden sich in der Closure der Funktion:

http://www.lua.org/pil/6.1.html

http://www.m4x0r.com/blog/2011/08/lua-local-performance/

Man könnte es nochmal aus der Closure in eine Local variable kopieren
What about the deprecation cleanup for R5, many plugins will break anyway, so why keep this?
Heroku required a rake of the precompiled assets in order to work, if this step is required again then these were the extra steps needed.

bundle exec rake assets:precompile
git add public/assets
git commit -m "vendor compiled assets"
git push heroku
All indexing work is already done via the standard backend already since 4.0 to make it possible to have the MassIndexer work at the same time as normal operations. The threadsize for indexwriter was already ignored, this should have been removed before but we didn't notice that.

> ... and deciding of the repartition?

I'm not sure I understood which _repartition_ ?
yes I mean in an impl package. I'd say don't clutter the code with deprecation in this case.
Sorry I missed this earlier. Caching this doesn't seem warranted, as it shouldn't be called too often. But the API for getting the string should definitely be `to_hex`.
This module is always imported as "RT":

import py.clojure.lang.rt as RT

and then used via:

RT.list()

So a search-replace for RT.list should do the trick. 

or 

RT.map()
Sande, changes done from my end.
https://github.com/plone/Products.TinyMCE/commit/ade51dde1e6e72cedc1e9225755a732fd2435335
Well, it actually fixes #622, i.e., it is possible now to replicate the persisted file via browser.

I'm building an ECL example to add to the regression tests and if that doesn't work, I'll open a new issue and investigate, otherwise just add the test.
Thanks Rob.

The problem was indeed related to TCP endpoints configured on my application. I had to put code for Port Monitoring and resolve the port issue.

Girish
Me while writing the serialization code
![Science Dog](http://i.imgur.com/GksBP.jpg)
The second one is not a LoS bug, but holes in the height calculation, about the first one, idk what could be causing it, maybe its just like that, anyhow, you wont be able to stay in the pipes after the battle has begun
Yes i see, happends when Player::RemoveFromWorld() is called (when switching continent or logout)
There is a check GetViewpoint() that expects NULL value returned.

Apply this until I found proper solution: https://gist.github.com/1556363
Not to be any kind of dissrespectful in any way, but the first page of the issue tracker is only crashlogs, and after the seasonal quests been implemented server is highly unstable :) I really wish i could help, but i am no expert in c++. But i think that some time should be spent clearing up the first page
I know that in this folder that optipng only hit doc files; however, I ran optipng on the entire platform directory and did this commit/push to just eat the change in the build directory.
this needs more tests. There are also utf16 boms, e.g. https://github.com/mozilla/zamboni/commit/d476408b6eb11a1888952a5a4c75822eb8ef0fba#L0R676
We need to agree on a tab size, I suggest 4.
Bump - please respond
我 adobe 系列 ps, illustrator.....沒一個會用的。

暈.....我的本意是讀書兼翻譯，但現在發現都讀不了書啦 - -|||

對啦...我發現你的readdoc有漂亮的 note, warning 框。

是你自己修改 css 做出來的嗎？

--
Juanito



On 23 January 2012 15:07, huangz1990 <
reply@reply.github.com
> wrote:

> 我也是美工苦手，弄图片不行阿。
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/JuanitoFatas/acl-chinese/commit/c6da431b47da5fdd21bab13c809c729c28d4f0f4#commitcomment-891303
>
Hello.
Oleg, yes it passed.

I've just made it as as **passed** to avoid entire build crash.
These two lines must be  
```assertTrue```
CS: `TRUE`, `FALSE` and `NULL` in uppercase.
ok entonces. haces el cambio?
Can we get more test cases that test tuples containing expressions and other symbols?
@denis-sh: Thanks. Will think more about it. It indeed may be good reason not to use ->.

Still grammar needs update. ;)
yeah, FYI, I'm not "stoffing" the internal Exercise code :p
Kudos for simplifying this line
why? its autogenerated when you migrate
kewl. thanks. pretty minor contribution, but I'll try to keep em coming.
Generally parent methods do something, while nonrel overrides them to do nothing. I've removed the "parent part" of those comments in some later commit (that's not really needed to understand how nonrel works) and added somewhat more interesting things to a few -- all the others could probably make it without a comment.

Reading the resulting file should be a lot more comfortable than venturing through the commits -- they really ought to be squashed, I'm just missing the time to do so :-)
duplicate :p
As for MT4, this is a dead product to me, I seldom use it and always build on MT5 nowadays. My plan is to rewrite this completely to 1) use MT5's listing framework (not in MT4) and 2) drastically reduce the number of tags in this plugin. So the next iteration will actually be a completely different plugin, MT5 only — if I have the time and energy to work on it of course.
Sure, that's probably easiest then. We can re-add the issues reported.

FYI, @601am has a private fork as well, branched off the most recent Endevver one. All of the changes are in a single Perl module though, so we can easily handle it later if we contribute those back. I'm not sure what the plan is on that.
Can you explain why that is? (I mean to me, not in the comment!)
not declared

V4L2_CID_CAMERA_RETURN_FOCUS
Ah yeah I forgot this change was only in the master branch, just brought it over to amd.
:)
And the dog didn't drive you up the walls?
Thanks, I just fixed it in 7d57f100384344f127fa02da9dddddee073b900e
Actually, this should be a visible warning, but well....
and it is done in the newest commit
One line on a commit
It's in the "All_World_Updates.sql" now...
Commit message should be "Changed $dom2 to $dom.  Other $dom variable has different scope."
Yes it should. I'll take care of that. Thanks for reviewing this!
You don't have conv either. Just test it yourself...
it might be better (for changability) to write this in a unit test and extract the src (as explained somewhere before).
Readme looks great, nice work!
For those interested, a sanitized version of our postflight script is located at: https://gist.github.com/1537820
file=filename.... tse 8)
Is "that" necessary?
For a moment I thought you were going to return an iterator that keeps updating with the latest log lines until EOF. The docstring is not very helpful here :-)

It should be simple to re-use that hypothetical ring buffer class for this.
Might I suggest a property with an intelligent setter
Native speaker here... IMO "will" is better, if the intent is really to indicate "always". Another suggestion would be to use the terminology given in RFC 2119 (http://www.ietf.org/rfc/rfc2119.txt) which in this case would mean using "shall".
Would you prefer immutable.Seq(...) over Vector(...) here too?
And correct me if I'm wrong, but I thought does is the singular form, and here you are referencing a singular - for-comprehensions, so it should be doesn't, not don't.

As I don't understand what data-flow composition in akka really is, I won't argue the point anymore, but as a native english speaker, "flow-expressions compose" doesn't sound right at all. (See the examples here http://dictionary.reference.com/browse/compose).
If you wanted to use the form "compose" instead of composition, I would have thought it would be "The flow method acts as the delimiter of dataflow expressions ([...]) and to compose flow-expressions."
This should move in after the if-check
I might miss something, but where is the encoding (utf-8) specified here? What serializer is used here? Isn't it JavaSerializer (java.io.Serializable)?
must call user_errors.
This shouldn't throw an exception. Lazy responses make usage much more enjoyable :) Returning a view (or viewmodel) is valid and should be converted to a response.
I do not like it. This has to be something to do :-1:
So, if I understand you correctly, you suggest that we corrupt the signal with something like: 

signal = signal + np.random.randn(*shape) + np.random.randn(*shape) * 1j 

And then take the absolute value of that as our final signal?
s/State/Start/
yeah, a little cleaner I think?
this is c/p from mainline, but sure, agreed.
Shouldn't this option be required ?
This needs to be renamed.
I guess you mean not have a 2nd `When` after the `Then`, as the `Given` is part of the output I'm checking against... right?
This change was made to accomodate the longer `objective_c` line that is forthcoming.
this.data = this.body =  request.post;
All header files are now gone in binding.gyp and winpty.gyp
We don’t use this now, right?  It seems to has to be removed.
Is this a mistake? I don't see any `path` variable around.
This can be collapsed into a `CGRectMake`.
I copied the TRUE style from line 48 of the same file.

https://github.com/shurcooL/twui/blob/f3efe03a2dd69bc2293dfd4331683290431d7b86/ExampleProject/ConcordeExample/ExampleView.m#L48

I looked, and maintainContentOffsetAfterReload takes the same (BOOL) parameter as  alwaysBounceVertical, hence I don't see why they should have different parameter values.

Do you want me to change them both to YES then, or just alwaysBounceVertical to YES (and if so, why)?
When I ran this (with real Java, not Doppio) it compiled as:

    System.out.println(("string" + a) == b);

Instead of what you likely want, which is:

    System.out.println("string" + (a == b));
Needs a comment (here or in the .cc file) explaining that `key` is copied but `value` changes ownership.
This function needs lots and lots of tests.
have not much experience with this kind of interop. what exactly do you think must be converted/handled by the C code.
i added the two methods already in a local test branch and it works when i call test_printhook from C#. but i was not getting any messages from PD though. this was my test code:

the C# type t_libpd_printhook is defined as:

[UnmanagedFunctionPointer(CallingConvention.Cdecl)]
public delegate void t_libpd_printhook([In] [MarshalAsAttribute(UnmanagedType.LPStr)] string recv);

in C i have this and i am getting the "Hallo" back:

/*set hooks*/
void libpd_set_printhook(const t_libpd_printhook hook)
{
	libpd_printhook = hook;
}

void libpd_test_printhook(void)
{
	libpd_printhook("Hello");
}

thanks for your comment and help!
いいんじゃないのー？
``HashWithIndifferentAccess`` is defining [#dup](https://github.com/rails/rails/blob/master/activesupport/lib/active_support/hash_with_indifferent_access.rb#L182-186) and it doesn't call to ``super`` or ``initialize_dup`` never, so if we override ``initialize_dup`` or ``initialize_copy`` here it won't be called never
Yes it is and the answer [is here](https://github.com/rails/rails/pull/7927/files#L3R145)
No other methods in MigrationProxy are documented. I'd suggest making the class nodoc, as it's an internal implementation detail. Or, since the HTML docs include the source of the implementation, the 1-line implementation is pretty much self-documenting.
Do you have a citation for this? I just went straight from the docs.
yes, add the names too
You need to test the collection helper here.
It still looks not indented in github, something weird?
Double space after `gem` :)
How is this handled now?
Need `()` here.
I'd rather we don't break up lines by dots.  It's annoying to figure out who the recipient of the `map` like this.

Why are we checking `respond_to?` on `to_sql` here?  The previous code did not do that.
Typo: `externl`.
`${impSym.name} == ${impSym1.name}`

(When do we get that tracer macro?)
We should also enable linking in doc comments - look for the TODO in `MemberLookup.scala`.
```
/** [[java.lang.String A string]] is your best friend. */
def buz(d: String) {}
```
Realistically, minimal fix is the only way.
I guess this is why you made that method a public method instead. Weren't you able to write a decorator for the `Payment::Processing` module?
That will include overriding all methods of Basic right?
Is this common knowledge? If not, could it be reversed? "Higher rank implies a greater partition" seems backwards.
And everywhere else.
Resave the notebook using the latest version of IPython (this should be 3).
I will explain it in the comments - basically it is replacing each pair of vectors with their commutator, hence from n-vector fields we go to n-1 vector fields.
The space is fine, but you changed 0 to "x". That's the change that I am asking for --- was that a bug in tests?
bump question on subs and "why not immutable_matrix"
Jason, I don't think that Methods are usually listed in the class docstring, although rhs should be taken out of here.
Removed.


On Fri, Nov 2, 2012 at 4:18 AM, Julien Rioux <notifications@github.com>wrote:

> In sympy/printing/tests/test_latex.py:
>
> > @@ -800,3 +800,11 @@ def test_Adjoint():
> >      # Either of these would be fine
> >      assert latex(Adjoint(X+Y)) in \
> >              [r'\left(X + Y\right)^\dag', r'X^\dag + Y^\dag']
> > +
> > +def test_Hadamard():
> > +    from sympy.matrices import MatrixSymbol, HadamardProduct
> > +    X = MatrixSymbol('X', 2, 2)
> > +    Y = MatrixSymbol('Y', 2, 2)
> > +    print latex(HadamardProduct(X, Y*Y))
>
> Leftover from debugging?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sympy/sympy/pull/1608/files#r2014272>.
>
>
Fixed it. So do you think this is good to be merged?
no need to use excessive new lines (i.e. just one of the two following)
https://code.google.com/p/objectify-appengine/wiki/MavenRepository?wl=en
I was going to deploy it, but didn't, but now it makes sense.
Not sure why the comment is talking about interface meshes?
Oops forgot to mention that its because condition number 7 (the mixed one) is now deleted.
Yes, but please notice /multiline flag. :)
В итого полечаем  2 роута:

`/events` - выводит только список всех ивентов, ответственный контроллер - `EventsController`
`/user/events` - выводит список ивентов, которые создал текущий пользователь, контроллер User::EventsController
`/user/:id/events` - выводит список ивентов, созданных юзером `:id`, как это лучше всего сделать для тебя будет сложновато понять, но попробую расписать (сам я такое пока не делал, поэтому нужно будет проверить). Один экшн в контроллере может обслуживать несколько роутов. В данном случае, я предлагаю сделать следующее: контроллер `User::EventsController` мы сделаем так, чтобы он смог 'обслуживать' следующие роуты:

             user_events GET    /users/:user_id/events(.:format)          user/events#index
                         POST   /users/:user_id/events(.:format)          user/events#create
          new_user_event GET    /users/:user_id/events/new(.:format)      user/events#new
         edit_user_event GET    /users/:user_id/events/:id/edit(.:format) user/events#edit
              user_event GET    /users/:user_id/events/:id(.:format)      user/events#show
                         PUT    /users/:user_id/events/:id(.:format)      user/events#update
                         DELETE /users/:user_id/events/:id(.:format)      user/events#destroy
               my_events GET    /user/events(.:format)                    user/events#index
                         POST   /user/events(.:format)                    user/events#create
            new_my_event GET    /user/events/new(.:format)                user/events#new
           edit_my_event GET    /user/events/:id/edit(.:format)           user/events#edit
                my_event GET    /user/events/:id(.:format)                user/events#show
                         PUT    /user/events/:id(.:format)                user/events#update
                         DELETE /user/events/:id(.:format)                user/events#destroy
Ah, I thought I'd managed to avoid that creeping in. Oops.
I think it is clearer to materialize the `System.currentTimeMillis() - startTime)` into a variable and then use this on each of the subsequent lines.
A line of documentation about this trait would be nice as well.
example should have "given a collection with the following documents..." and the example corresponding result set.
Yep. The full tab copy isn't going to show up until July 1, so I will work on it between now and then.
"the guest user's data"
Yes that's fixed in my branch. Nils has to pull my commits :-)
On Dec 18, 2012 4:22 PM, "Joas Schilling" <notifications@github.com> wrote:

> In phpBB/develop/create_schema_files.php:
>
> > @@ -1273,6 +1273,18 @@ function get_schema_struct()
> >  		),
> >  	);
> >
> > +	$schema_data['phpbb_moderator_cache'] = array(
>
> table names are mixed up here
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/phpbb/phpbb3/pull/1067/files#r2455948>.
>
>
Class missing docblock
At the very least, possibly: min(max(v, 1), ...)
Removed the lines now
I guess it's not a huge issue, but I do prefer without leading articles. PHP in the previous line should be uppercase, anyway.
Why are you using finder to locate templates? Finder doesn't account for template inheritance. Path provider should be used here
These statements are part of existing code. But yeah, could do that. Also needs a comment then.
This was actually just a quick hack - I wanted to ensure `res:finish` gets called after all the writes. `end` and last `hop` events (every hop which is parsed from the last line) get emitted in the same tick which means that `res:finish` could potentially get called before all the `write` calls.

I will do it properly and modify traceroute module and make sure it always emits `end` and `error` event in the next tick.
Space after comma, please.
Is this X11 Display?
Cribbing off the sector plot form to make sure it was formatted right. Without it the text doesn't match up with the galaxy select form.
so why those aren't just arguments to SurfaceDisk and there is a need for .equip?
They should just be declared in a private way inside function where they are to be used. We should definitely not use '_' (underscore). Look here: http://javascript.crockford.com/code.html#names
@esc24: That is a typo in the docstring, leftover from the previous implementation... I thought it was cleaner to take only ``numpy.ndarray``s  and simply pass the data attribute of the cube (much like ``iris.util.rolling_window``).

If this were to accept cubes also then we would need to check for the type of the input. What would be the best way to do this?
What is this? I don't understand...
@rhattersley - I can't come up with a name this isn't long winded and doesn't make my lines too long. I've added a note to the docs highlighting the use of nearest neighbour, but if you want it in the name you'll need to propose something you're happy with.
Can you add a link to the scipy sparse doc? I think you can use this link:

http://docs.scipy.org/doc/scipy/reference/sparse.html
The same long doc is duplicated 3 times. Can you try to keep the doc in a single place? It makes doc maintenance much easier.
The main long explanation could be in the help of theano-nose, while other doc could use a short one-liner summary.
I've added a nose-seeable test that runs this file using os.popen. I tried using subprocess.Popen but couldn't get it to work. I've left the code in as a comment in case anyone has suggestions.
Unrelated remark: why not TimeTrackable?
.all() is spurious
i only edited the 2 lines i dont know why git has changed the whole file.
If we're doing best practices here, this should be a `lazy_reverse('authors_list')` or something along those lines.
Great. I login a couple times a day usually. I just signed in.
If `move` capture did a copy then a move, test with `v` would fail.
What's with the grouping parens around `jQuery.each`?
I think there was a jQuery bug report somewhere for this right? Something about calling on iframes?
Do you need to check for `(string)` here? What else could be inside of `eval`?
*Facepalm*
That's a much better solution. However, some tests need te be updated. Will submit a new pull request shortly.
How about:
```
if dsize <> 0:
  plpy.error("found %d records where %s was not of expected length (%d)" % (dsize, attrColumn, numAttr))
```
Convention is to use \to in the signature, and \mapsto for the definition.
Yes, it's currently detected implicitly. Since polymorphic/anonymous types cannot be resolved, the type conversion will fail. E.g., the C++ AL does not know how to convert from double to typeoid ANYELEMENT, so it will complain: "Invalid type conversion. Backend expects type ID 2283 " ('anyelement') but supplied type ID is 701 ('float8')."
We tried to test this, but registering as a QQ developer seems to require having a Chinese mobile number (and maybe a Chinese ID # too). (Just getting an account doesn't; it's the developer signup.) Nobody at Meteor HQ has a Chinese mobile number, and we don't want to merge code to core that we can't test (both now and in the future as changes are made). So we're not sure what to do next. Any suggestions?

(In any case, we are happy to continue providing feedback on the package, since even if we can't get it in core yet it still can be used from Atmosphere.)
Methods could be overridden. Fixed now.
I'm currently installing Snow Leopard on an empty HDD. I will try and install Gegl in on this OS.


On May 4, 2012, at 8:45 PM, Charlie Sharpsteen<reply@reply.github.com> wrote:

>> @@ -1,13 +1,28 @@
>> require 'formula'
>> 
>> +class NeedsLion < Requirement
>> +  def satisfied?
>> +    MacOS.lion?
>> +  end
>> +  def message
>> +    "Gegl requires Mac OS X 10.7 or newer: It uses OpenCL 1.1 which is not implemented in Mac OS X before Lion"
>> +  end
>> +  def fatal?
>> +    true
>> +  end
>> +end
> 
> Hmm... well it is working fine for me on two machines: my main desktop and a VM running a pristine copy of SL. Got GIMP 2.8 up and running using #8835---although I am not a heavy user so I didn't do much more than load up a few images and paint around on them and apply some filters.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mxcl/homebrew/pull/12063/files#r776089
Executing a binary also ensures that the binary actually functions. Anyway, we don't want tests like this. Good library tests compile a small test program; see e.g. glib or msgpack.
Sweet, it worked, thanks!
Sans accent, c'est pas top ... Faudra trouver une solution pour modifier l'encodage ou un truc comme ca ...
Cython docs - [Decoding bytes to text](http://docs.cython.org/src/tutorial/strings.html#decoding-bytes-to-text)

The initially presented way of passing and receiving C strings is sufficient if your code only deals with binary data in the strings. When we deal with encoded text, however, **it is best practice to decode the C byte strings to Python Unicode strings on reception, and to encode Python Unicode strings to C byte strings on the way out**.
I think the best way is using HashMap instead of List
I've added escaping for &<>"'/ as per https://www.owasp.org/index.php/XSS_(Cross_Site_Scripting)_Prevention_Cheat_Sheet#RULE_.231_-_HTML_Escape_Before_Inserting_Untrusted_Data_into_HTML_Element_Content
This will likely break a lot of code. If a Description has children, we should consider it a suite, regardless of how it is named. So at the very least, we should do this:

    return getChildren().isEmpty() && getMethodName() != null;

...but I worry that that might break some people, too.
`if (` not `if(`
yeah, i originally did that before noticing that they don't include the version in the comments like they used to.  that used to provide a quick and easy way to know what phonegap version mulberry is currently on -- this was actually how i knew we were on 1.3.0 currently.  i suppose we could insert the version in a comment ourselves.  don't feel strongly either way ...
Randy mentioned that this one requires a modifier key (Ctrl), and it seems nontrivial to add modifier-key support to simulateKeyEvent()... so it's fine by me to leave this as-is.
Just occurred to me--we should disable the Live Highlight command when live development isn't active.
Good catch.  I think you're right that it's safe for now: EditorManager doesn't reference StatusBar unit APP_READY, and StatusBar doesn't reference EditorManager until someone calls show()/hide().

But ideally yeah, I'd like to fix #1869 so that EditorManager no longer has any references to StatusBar.
why are these required?
had it as single quotes for consistency, updated to double.
Please remove this eclipse flag :-)
yule ? I think this one needs to be renamed, and the hints/labels associted with it... Unless there is an explanation for this naming, of course.
Can you be more precise? What means that it is not displayed properly? So the initial value of the form should be what your browser sends. What is the behavior you would want to see? What is the behavior you are seeing?

But maybe in our case it should work differently. As we are already creating the temporary user, things are a bit different: we should create that user based with language field based on the browser setting. If the user register, we just fill the registration form based on the existing user values. For example, this means also that if the user changes the language through the footer language links, we should store this change into user object.

What do you think? So language field is filled with the proper value where we create a temporary value. And in registration form we prefill form with the values from the current user object. If user changes the language we store that anyway (in temporary or non-temporary user object).
Added TODO for now. This cannot be done with "just" mongoengine constraints right?
Lint is complaining about this
Change to deleted without incident rather than added without incident.
If this doesn't matter just ignore it
"New York City" sometimes and "new york" here, which will miss geocode cache.
was this intentional? I understand why `warn()` would become `info()` as in below, but here it seems like we should keep the `TODO`?
DaFuq iz dis? o_O
漏了 seek 文件偏移量
Hmm da man fast alle zeichen auch mit %irgendwas url codieren kann sehe ich wenig sinn in strip_tags(). Ich schlaf mal drueber ;-)
HC4 contains two connMgr implementations: "basic" and "pool". In short, "pool" one is meant to be used in multithreaded fashion, that's the only connMgr usable with them (if we do not count the deprecated oldie connMgr). Hence, for multi threaded use case, the only connMgr is pooling, but it does not mean that httpClient using it will pool (it depends not only on config, but also on remote response, response subsystem is making -- it might set Connection:close per request, etc).

So, IMO, the fact that "pool" connMgr is used does not mean nothing yet in HC4, it does not mean you WILL pool. But true, it could be noted in Javadoc that pooling is explicitly disabled, but that connMgr is used to make client instance thread safe.
ch->command == NGX_CMD_IPSTAT 就不用执行ngx_channel_next_handler？
not sure why you don't combine these actions thought a flatMap. Weird things can happen.

Also, you need to block at some point in your test. I believe we need a typeclass for that.
So... I just looked this up. It's kind of gross we have an assert_contains that's totally different than assertContains. Ick. That should get cleaned up.

Seems like assert_contains isn't really worth putting into a method.
Could/should katello-reset-dbs work in headpin mode? (I have no idea if it ever worked w/ headpin or not.)
At some point, yes, but there was the question of someone else setting up users and so that was a different can of worms and different RFE.  Plus I think @mccun934 is sick of me working on this :)
This does - https://www.google.cz/search?q=katello&hl=cs&lr=
Why are you doing this instead of a has_many :through?
What was the conflict with timers? This is a pretty poor implementation of a timeout.
Ah I see, I was thinking of specific in-project tests. Ignore me.
Need a space following `if`. `gjslint` you code!
Theoretically the directory that built distributions go into can be different from 'dist', and there are a few isolated cases where it is different.
This is fine for now, but if we want to upstream this it would be better to add an option called `dist-dir` (see `distutils.command.bdist`--it can be copied from here), and in `finalize_options()` add: 

```
self.set_undefined_options('bdist', ('dist_dir', 'dist_dir'))
```

This way if someone has configured a different default dist dir it will get used.
And yes, don't use `return` in Ruby.
This is on by default, should probably say so in the description
Hmm but in other places in the source code we always simply use `#include <QMenu>`. So I guess it would be more consistent.
If real_strprintf is "safe" and you are right, it's quite often called ... I'm fine with removing the init. In general my intention was to reduce the risk of buffer-overflows / buffer-corruptions causing weird things to happen. It would be nice, if we were able to rewrite funtcions to use std::string instead of char * or char arrays.
Is there a way to write this that doen't involve triple-y nested conditions?
from: http://php.net/manual/en/language.references.pass.php

There is no reference sign on a function call - only on function definitions.
I think that it the user don't know what's a server WMS is he isn't able to use this interface, than I think that's better to be technically correct.

But I just have a look on QGIS, ant it's named "Ajouter une couche WMS" than no problem for me to rollback.
api.parse_ts already exists
متن ویکی پدیا در رابطه با گنوم
An is check intuitively makes more sense to me, but I haven't looked for
possible side effects.
Yes, makes sense. Then the user directly sees that matplotlib could not be imported in the file IPython/core/pylabtools.py.

Changed locally. I have just some problems in the other file with logging...
The difference is that TraitError results in a clearer "invalid config" fail, rather than the regular error which looks like a crash.  I can add a log.error(exc_info=True) above, if that would be preferable.
This feels wrong to me -- if caller has requested a specific location, seems we should _reject_ inputs without a location, not accept them.
(I have simply preserved the behaviour as it was.)
It does not matter for these case. I don't check that they edit they edit themself I only check is user admin or not. Good idea with customer matcher I'll do that
No special reason. I'm fine with changing to var.
There is no need to define a new type just for Solaris compat.

```c
struct dirent *de_buf, *de;
de_buf = git__malloc(/* platform specific */);
```

The platform specific code is left as an exercise to the reader. *hint*: since the only corner case are systems which define `dirent` with a 0-array, you may want to use `NAME_MAX` instead of `GIT_PATH_MAX`. It should be smaller and just as accurate.
@nulltoken, @tanoku, @arrbee, how would you feel about something like this?

```c
typedef struct git_checkout_opts {
	git_indexer_stats stats;
	int existing_file_action;
	int apply_filters;
	int dir_mode;
	int file_open_mode;
} git_checkout_opts;

#define GIT_CHECKOUT_DEFAULT_OPTS {  \
	{0},                              \
	GIT_CHECKOUT_OVERWRITE_EXISTING,  \
	true,                             \
	GIT_DIR_MODE,                     \
	O_CREAT|O_TRUNC|O_WRONLY          \
}
```

This means the API surface would look more like this:

```c
int git_checkout_index(git_repository *repo, git_checkout_opts *opts);
int git_checkout_head(git_repository *repo, git_checkout_opts *opts);
int git_checkout_ref(git_reference *ref, git_checkout_opts *opts);
// ...
```

... which seems more logical to me.
无关的代码可以删除
example https://github.com/mozilla/playdoh-docs/blob/master/_build/.gitignore
This is a bit inefficient because you're doing a separate query for every suite. I guess it can be fixed with prefetch_related in 1.4; without that you could do a manual version of prefetch-related by doing a single query for all RunSuites related to the for_run, and then just looping through them in memory to match them up with the right suite.
ahh, right.  dang, just an oversight.  thanks!  and thanks on the > 0 thing.
Nit: If you don't need `parts` you can do: `path, __ = self.lookupNamePath(crash_id)` and pyflake won't complain.
ah but they are.  these functions are called from _regsiter using the dispatch table.  that call is wrapped in the transaction class.  that means that all database interaction with the chain of calls are also within a transaction.
The message is returned as are other similar messages. If those are L10n'd, then yes. Please advise for how to correct this and I will follow up.
The naming is weird for a class which will be the frontend of the Cache component
There is no need for this function anywhere else
Well, property access is faster than a method call, but I won't argue much here. Calling a process is probably a magnitude slower that it is not significant.
why removing all empty lines ?
please don't use inline style for that, use class/css instead
yes, and require-dev for the testsuite (and a requirement in framework-bundle)
it should be using a type name, with the type being registered.
you should not remove it. ``DataEvent`` extends from the base Event class so the EventDispatcher component is needed
Not really. Look at the internals of Options. Options overloads existing options by default instead of replacing them. So in order to replace them, the options need to be removed first.
You need to use `count()` not an alias.

Also you should add normal: `use Symfony\Component\DomCrawler\Field\DynamicFormField` and call here: `new DynamicFormField($path)`
does it work for the ODMs too ?
Maybe extract this logic into a static method, so it can be re-used by other implementations which want to be BC.
Absolutely, however I think this methos should stay to force using a specific adapter.
I think it is more important to prevent false usage than to require the casual use of `array_values()`.
URL じゃなくて アクション（すなわちメソッド）の試験なのを意識するなら symbol の方がよりよさそうですよね。
(And this is just refactored code, I've extracted this to separate method from previous version of `tableView:cellForRowAtIndexPath:`.)
Just curiosity: does the `>` do some kind of inheritance?
age faghat backgrounde lazem nist dige
man fek mikardam chon social links marboot be footere va foter to tamame page ha yeksane pas jozve layout hastan
间隔可以留个表格
字体——不要
I was thinking something more generic could apply, but since this is supposed to indicate a 'kernel', I thought that the device would be more appropriate. This would allow future devices, if they used a similar protocol,  to specify their name to indicate which particular protocol is in use.
@mockee 哦，刚看到`viewDidLoad `已经有`[self configureView];`了。靠 setter 刷新展示其实也ok，好像这里不写应该也会正常显示数据吧
@Sunight  看个人习惯吧，说得通就好，我一般喜欢用 <功能描述>+<控件名称>，例如nameTextField之类的。话说`m`是什么意思？
it's passed from the caller (new app.GroupListView({id: XXX}))
It is good to say what is wrong/unsupported but I think it is better to error rather than log + ignore?
we can remove these
convert to console.warn() or just delete the handler.
I’m not sure I understand what a “subheader” is. I’d probably just consider that whatever comes after `<h1>` is a secondary header, e.g. with a `h1 + *` selector.
Why are you removing the mac address field from the wireless settings? Every other commit seems okay except this. Explain.
Related: #3245

@vingtetun you have also failed to update the comment of this function in this pull req.
Nit: two-space indentation everywhere, please. It’s part of the Mozilla guidelines and some text editors rely on the indentation for markup folding…
In any way I restore the edit mode here

https://github.com/mozilla-b2g/gaia/pull/5021
Same as the previous comment, maybe CSS can replace this function, something like <code>-moz-calc</code>?
But if it's complex to modify in this pull request, we can do it in a follow up.
betewen -> between
Hi Rudy,
Per @timdream's suggestion before (face to face), I will pick you commit into this PR, so we could solve this issue within one PR but with two commits.
I'm doing it right now, will let you know once I'm done. 
Thanks!
Do this inside the try
You need to update the functional test. In particular, make sure `GetResultsWithMaxItemsAndExcludeFailed` still passes.
Yeah, I thought I should do that and then didn't; will do.
Should include "I know about ['bluesteel', 'contests']"
Better way: JURI::root(true).'/'
Spout.getEngine() instanceof Server would be better
The reason I went with the  method, rather than using `this.getClass().getResource( "equipment.item")` is because `getResource()` doesn't do wildcards, and only returns one file.  I figured opening the jar file for this package, then doing a regex search on the filenames for `equipment.item` files, offered the only viable way to open all the item files.

That said, I've since written an alternate approach.  Rather than storing the item files as individual files in the jar file, I zip them up first, and store `Equipment_v4.1.0.zip` inside this package's jar file.  Then, I can use `this.getClass().getResource("Equipment_v4.1.0.zip") to open the zip file, and read all the `equipment.item` files from within.
What i wanted to say: Before, the operations were executed immediately, now i build an array of executions, which will be executed delayed - after each operation/package is checked for availability. Will remove it.. Thx
That said, for memory usage and speed, this is probably a good idea yes.
btw, shouldn't you return true here ?
True. Will do a rename...
that doesn't make it more readable.
each your request contains the bug.
Likewise belongs in common?
You're right. Having a crash with buffer set to 0 and buffer_len set to a non null value make me think this was incorrect. It has to be a memory corruption.
Yep! fixed.
Error checking for fwrite is missing.
This notice does not seem to have been replaced; probably requires a new method that can be overridden by `PluginsLink`?
Good catch!  Fixed.
this change in padding would make btn-micro bigger than btn-mini and pointless. Just delete it and use Bootstrap's native btn-mini if you want to increase the button size
Thanks, I understand now. Thanks for your help. I have added the needed line.
surprises? :p
I think it would be better to explain freeze reasons. or refer to another comment that explain them.
should be class here too
Nit: I found name to be a bit confusing, but don't have a better suggestion.
Does record need to be fetched both here and on line 61?
Some of these methods are just passthroughs, but some do work.  For consistency, I kept them all.
Nit: Just missed last time. It should be aOptions
I wouldn't call the member property 'test' because it can cause confusion with the global test instance.
Please shorten to Cc and Ci and align the second row with `C` from `Cc`.
Possibility of having some topological breakage here.
Just pengasas should be enough.
curlies are off
@vrthra I tried running this on compress/zip and it took 3 seconds to complete. Is that an issue when installing a bunch of packages? I don't know. Could you check with Ben and/or Stefan about this approach? Do other puppet package providers go through the trouble of only reporting on latest packages that are applicable to the given platform and have all dependencies resolved?
we can put the `.container` class to the `#content` class via `@extend` on the css/sass side.
I fixed this in my pull request to your branch.
Good catch. Verbose always wins.