labofsahil/github-event-dataset-2012 huggingface dataset:
Er, wtf, github, trying that again:


    <ivysettings>
      <resolvers>
        <ibiblio name="public" m2compatible="true"/>
      </resolvers>
    </ivysettings>
マージされてしまいましたが、トップレベルに ```class Core``` はちょっと大胆すぎるので ```module Automatic``` にして、その下に Core を置いた方が良いと思います。
Gem package 'hatenabm' must be improved.

/opt/ruby/1.9.3/lib/ruby/gems/1.9.1/gems/hatenabm-0.1.2/lib/hatenabm.rb:146:in `toutf8': uninitialized constant HatenaBM::Kconv (NameError)

However, the type of the argument would be helpful for refactoring this.
should be 'monkeyman' here
Certainly. It would likely be better to separate them into their own file for each kind of application (like I did with the systemObjects); I'll add that here.
This syntax is wrong. Needs to be fixed.
Also we should probably handle the case for reverse proxy. But be aware of the security implications.
Reverted back to the older commit so that New Voters are not removed, need that for displaying the whole information of the Ward.
May I ask what the point of these changes is?

Most of them make no sense.

Also: removing the license information violates the license (not cool, dude) ...
ooops, correct it for me :(

On Sat, Feb 25, 2012 at 3:17 PM, Daifu <
reply@reply.github.com
> wrote:

> item.hop_num = 1 is assignment not the comparison.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/khaing211/g-simulator/commit/8ff3b5f8c0b054f9399364c96844bc66d39f2228#commitcomment-1011619
>
This type of code should exist in helpers. Check out app/helpers
Added Splats
Please remove this file as well since it is automatically generated.
PDB: I see what you did there.  I think we should have a second constructor--one that takes the timestamp as an optional argument
What do you think of this patch to handle E overflows?

    --- a/dda.c
    +++ b/dda.c
    @@ -243,7 +243,12 @@ void dda_create(DDA *dda, TARGET *target) {
            dda->x_direction = (target->X >= startpoint.X)?1:0;
            dda->y_direction = (target->Y >= startpoint.Y)?1:0;
            dda->z_direction = (target->Z >= startpoint.Z)?1:0;
    -       dda->e_direction = (target->E >= startpoint.E)?1:0;
    +       if (e_delta_um < 1000000) {
    +               dda->e_direction = (target->E >= startpoint.E)?1:0;
    +       } else
    +               dda->e_direction = (target->E >= startpoint.E)?0:1;
    +               e_delta_um += 0xFFFFFFFF;
    +       }
     
            if (DEBUG_DDA && (debug_flags & DEBUG_DDA))
                    sersendf_P(PSTR("%ld,%ld,%ld,%ld] ["), target->X - startpoint.X,
See [http://php.net/manual/en/domdocument.loadhtmlfile.php](loadHTMLFile()). It allows you to parse "legacy" (as in "invalid") HTML.
Блеад, что эт? Оо
how many strings are left? I am able to translate the last bits and pieces this week
真好这个工具   哈哈哈
### minor change
I just *deleted* some **whitespace new lines** in the end of the file.

![reaction](http://www.blogcdn.com/www.urlesque.com/media/2010/11/reaction.jpg)
Was the kinect_skeleton.zip (which has the stick figure Max patch) removed?  When I try to download it, nothing appears on my hard drive.  Am I doing something wrong?
I squashed I all together, and added a fixup (the call to super in the
constructor).

-- 
Regards,

Andrew Ardill
а зачем класс то?
my goal is was to simply plot `lena` at the same scale so I guess I don't really
need `imresize`. I'll change this to `ndimage.zoom` and see what happens.
I'm not sure here, since the point was to make a comparison vs PIL imresize. @poilvert, what do you think ?
haha sorry for the delay, haven't had much time for it lately. still not happy with it but hey at least it works for now
event_param's are empty?
haha.. seriously man, love your attitude Aokromes.. /bow! he always help!
So this won't work until Vault is fully implemented?
nice job :D

may you can create another condition which contains QUEST_NOT_REWARDED [ !GetQuestRewardStatus(questid) ]

this would be good for some gossips which shouldn't be shown after rewarding  a quest - i think its not possible with the current contitions

with best regards :)
-(7613,11,0,0, 'achievement_spore_loser');
+(7613,11,0,0, 'achievement_spore_loser'),
+(7613,12,1,0,'');
I don't understand the goal here. Are you trying to get the HTTP request headers sent to the sink? Maybe it would make more sense to just include the final request headers in the `Response`.
This depends on widget initialization, which makes it hard to test (it would be great if we could test interactions in unit tests similar to the ones we have now). This can be separated by managing the event handlers in this class and setting up a forwarding from asWidget once it's initialized. That way, we could have an additional method in GraphSvgDisplay that can be used to fire events from tests (e.g. fireTestEvent(Event e)).
Could this all be skipped with newer apt cookbooks? See https://github.com/cramerdev/cookbooks/blob/master/newrelic/recipes/debian.rb (haven't had time for a pull req.)
Sure, but refinery isn't loaded here either: https://github.com/resolve/refinerycms/blob/master/refinerycms.gemspec#L2-3
Hmm, now I think about it that's a silly idea too. You want to use lifted `&&` and lifted `||` on `Region`s. I guess this was all inspired by your original `Monoid Color` instance. I looked at it and thought, "hey, `clear` is obviously `mzero` and `cOver` is obviously `mappend`" and then thought about analogues for `Region`s. 

Perhaps what we should do is just add `preClear` and `preOver` to the `HasColor` class?
np 

* fixed in commit 0932e7e6146c9b768267310bb2393801ad7a56ca
It works fine,just gotta script frostfire orb :P
ya plein de truc bizarre sur les rubriques
c'est pass sur deux lignes !

sinon le reste a l'air de fonctionner

jvais me coucher...


Le 11 fvrier 2012 21:20, Puddi Puddi ! <
reply@reply.github.com
> a crit :

> aprs mise  jour a fonctionne, par contre, dans le cas o il n'y a
> aucune rubrique de slectionne (quand on en a pas cr) a fait une erreur
> ... c'est normal ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/OlivierB/Polyjoule_siteWeb/commit/96ef74087c4df2c4a51b64c940eafadbef915a39#commitcomment-956961
>
I know you just grabbed the code from core to accomplish this.

However, I think it's way more elegant to just do what I did here: https://github.com/auzigog/django-sendgrid/commit/6c543e51e8fbe1b86d0279f0da1a7e0102b8047a

There's no reason not to inherit from EmailMultiAlternatives. It doesn't hurt or complicate any existing functionality. Emails are still sent the same way if you choose not to attach any extra "parts." It's more elegant to just extend EmailMultiAlternatives and you don't ever have to worry about your copy-pasted code staying in sync with Django core.

What do you think?
Ho visto dagli altri pull request che hanno fatto che ci pensa lui a d aggiungerli. Inoltre deve rifarlo per forza visto che nel frattempo ha ricevuto altre traduzioni...
Sorry I was not awake. In fact github was not up-to-date (now is). If you can rebase your chances, I will do the merge.
Hey Tom, noticed you also an updated wifi driver for the zte blade on CM gerrit when you submitted these changes. Currently I develop for the commtiva z71 witch uses a similar (if not the same) wifi chip but the current drivers have some serious problems. This new driver you uploaded, is the source available somewhere?

Thanks in advance, keep up the good work on Blade (brother has one, very happy with it)
@ d) I thought I get the bundle classloader by asking an already loaded class in the same bundle for the classloader
Fixed upstream: bos/pronk@9d1bb7c8ed84e453a9151d1071dac71231c55423
Coding style: spaces around arithmetic operators.
Actually, thinking on it, view() should be separate. Nevermind then.
This position and z-index seem pointless, as you add them again to .zocial > span in the next ruleset.
I think I understand what you're saying now...

My mistake was thinking that the behavior you just described, "this technique allows for a single theme body yet still having different platform themes as your base", would also happen the way I was suggesting.

You are sure it won't work the way I suggested though? You tested it or are just assuming because it what makes sense? Cause I agree with you, it makes sense, but in my head, it also makes sense the way I described it. Maybe not as much sense, but it does a little lol...

For instance, let's assume the library has 2 definitions for ```Theme.ActionBar``` on ```values/themes.xml``` and ```values-v11/themes.xml```. On my app, I create a custom theme inheriting from ```Theme.ActionBar```. If I run the app on a API below 11, I'll see my custom theme, no arguing there. But if I run it on Honeycomb (v11), I won't, unless I "remember to make it to the other", correct?

Sorry if I'm being annoying, but are you really sure this is how it will play out? Don't take this the wrong way, it's not like I don't believe you but I'll have to test this for myself...

I mean, when Android goes to look for my custom theme, it will only find one available on ```values/themes.xml```, no matter the Android version. And the parent will be ```Theme.ActionBar```, which will redirect to the library code. On the library code, ```Theme.ActionBar``` will have 2 definitions, one in ```values/themes.xml``` and the other in ```values-v11/themes.xml```. **Isn't Android going to pick the one corresponding the Android version running?** Just because there was no ```v11``` folder on my app, doesn't mean it shouldn't look for one in the library if needed...
It would be helpful if you could explain how you did it.
This broke the site :(
I just checked which version of Python stable Debian branch has set by
default, and it's already 2.6. So I agree that it is safe to assume that we
don't need to support <2.6.

On Sun, Jan 15, 2012 at 18:34, Menachem D. Mostowicz <
reply@reply.github.com
> wrote:

> You do realize that supporting python 2.4 through 3.1 is going to cause us
> a lot of headaches?
> We could either support 2.6 - future or 2.*, but supporting < 2.6 and >=
> 3.0 is madness.
> (At least, that's the response I got in IRC when I asked how best to
> support 2.4 - 3.1)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/InvisibleEngineer/XKCD-Downloader/commit/4a682d012184913adb604e2f3f4bf0503358a8e5#commitcomment-868650
>
Dela :D
On Tue, Feb 28, 2012 at 22:28, Kai Hendry
<reply@reply.github.com>
wrote:
> I have it so CTRL+L is disabled in webcnoaddressbar since you can use
> it to override the purpose of missing URL bar, so folks can't navigate
> just anywhere. :)

But why allow the user to use C+l? in a Kiosk I expect the useful
links to be on the webpage, no more.

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
Походу покоцал лишнее, или это Калина покоцал? Хз короче, разберусь завтра
{ on previous line
@cjmaclellan 

notice relative path so we can run the same without changing the code
Can we fix anything in this situation? If not -> abort with a meaningful output!
There are Windows specific changes made the XML module in the commit linked to above. /cc @allenrabinovich
Yes, sorry, just going through exact same thought process!  Also are you able to help me understand why Travis failed?  https://travis-ci.org/#!/yui/yui3/jobs/2712349  I thought the build process would add these from the src to the build but it looks like they haven't?  What have I missed?  

Thank you!
Should "ImgButton" be "ThumbnailButton"
`positions_merged[position.ticket.pk]['amount'] += 1`
Why not ```getSearchBase()``` instead of ```BindLdapAuthenticationHandler.this.getSearchBase()``` ?
Glad we got everything cleared up and found the most elegant solution. I've added a page to the Wiki with a Memcached CacheMap example for ClearPass here:
https://wiki.jasig.org/display/CASUM/ClearPass+and+Multiple+Server+Configurations

I also put in a note on EhCache configurations.
Does not help, netty already caught the exception. Or should I rethrow fatals?
I think you’ve lost the TestKit along the way?
I think it's the caller who should decide _how_ something gets executed. I'd remove the Future {} and the Await.result.
why extractor + allocation instead of type check and return value?
good catch, I'll make a local val of type Seq (or is there a better immutable type to use?)
`removeChild` may well change the type of ChildrenContainer (that’s the whole point of it).
you think about that the values are VectorClock and that is expensive? or is it the size operator you think about?

I have an idea that the seen table doesn't have to contain the full VectorClock values, but only a hash of them (less to transfer, faster to compare for equality, and that is the only thing that the seen table is used for (I think))
FWIW, this implementation of InputStream.read is incorrect. Read should return the unsigned equivalent (0-255), or -1 in the case no byte is available. See http://stackoverflow.com/questions/4332264/wrapping-a-bytebuffer-with-an-inputstream/6603018#6603018
Very good explanation, but I feel the wording is backwards. I would rephrase it like this.

```
Every time an actor does not terminate by its own decision, there is a chance
that some messages which it sends to itself are lost. There is one which
happens quite easily in complex shutdown scenarios that is usually benign:
```
I'd change that to: CAUTION: Do not use if you do not exit the JVM as you shut down your ActorSystem, as it will leak threads.
I find this sentence about `unbecome` not being called confusing. When is is called? Don't you have to call it yourself?
PR-675-merge: here you will remove `with AccrualFailureDetectorStrategy`
It is used by the 'ping' command to just check that a node is up and fine. Could be improved to send more info than "pong". But it will do for now I think.
ok, I'll fix it.  It may not be at the top of the list yet.
The yielded sites _are_ equal to the contained ones; the `mesh` is explicitly designed to discard depth information. Also, a `Site` in the context of all of these calculators must be considered to be on the Earth's surface (depth=0.0).

__iter__ is just a convenient way to reconstitute individual Sites from the collection.
I think your point is correct.  It could be a separate instance method, or we just make fetch do this.  We may want to distinguish a few cases.  I may add some things and then open this request again later.
Эта строчка вроде нужна, так как поле текстовое.
`@global $skyphp_storage_path;`
Czemu na Network? Po co classmethod?
Ta metoda powinna być na roli.
don't want to merge incomplete code like this
Elaborated in comment.
If you're going to allow people to specify multiple SMTP servers, you should also properly export/import them.
^^=
This part isn't Rails-dependent, so I'd prefer to keep this here.
Obvious but i added it for completeness.
I'd like to see a brief description in this comment, in addition to the exception flag. Please explain the appropriate use of this function. It doesn't have to be long, but it does have to explain what it means to match one transaction with another, and should give an ```@param``` mention of the other parameter. Clearly, this is the other transaction you're trying to match this one with. BUT, what does it mean to match one transaction with another. 

Also, I see you've explained when it will throw an exception, however :  Why should this exception be thrown when someone tries to match two transactions of the same type? What do you mean by type? Why would matching two similar transactions result in a 'mismatch' ?
Can you document how start/end are represented in this event?  Is there a flag indicating which one is changing?
please add support for ``Twig_LoaderInterface`` loaders
It is style issue. The two forms are semantically equivalent – the method will still be invoked when using dot syntax. See the "Dot Syntax" section of [The Objective-C Programming Language](https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/ObjectiveC/Chapters/ocObjectsClasses.html#//apple_ref/doc/uid/TP30001163-CH11-SW17)
Oh, good catch. I'll use "constrains" instead.
Won't this break sunos+dtrace builds?
No, that's a great catch, thanks!

Fixed on df33211e14d94cb9dee6b717ae93ad845699ce3a
Yeah. I removed them because for now we don't need them at all. `PooledByteBufAllocator` does not run its own thread.
This change is modifying the return value of `update_column`. We should probably add a test to ensure it returns a boolean.
Sorry I missed the detail about this new table in your comments.
Silly mistake. It's correct on master, which uses a much improved code through transform_keys :)
Qualify with `ActiveSupport::` :)
Remove this extra blank line
Why you added this?
That's what my comment says (and implies that I'll do it.)
Good point. Will fix immediately.
Sure, but the PR is against `master`. Would be nice to have something like a deprecation warning, but I'm not sure how to express that.
angmom, linmom -> angularmomentum, linearmomentum
What does "change the system" mean? Could you write example code?
Manifold should be immutable (It is basic). 

You could add an `add_patch` method which produced a new Manifold object. 

Alternatively perhaps these objects should not be Basic.
It's not "necessary" anywhere.  It's just a code quality issue.  See PEP 8.
Why this restriction? It may not make a lot of sense, but we usually only raise exceptions when nonsensical expressions cause actual errors.

Also, there are too many newlines.
Right. Will work on it more.
@davidhrbac can you please check cs translations according to number choice formats?
It might be worthwhile to mention the other setting as well.  If you think that's useful, why not add a little heading below to explain a little how these values can be set and what they do.  Like for `caching_policy_chooser`.
How about this?

for value in iterable:
    break
else:
    assert False, message # or AssertionError like above
yah should just do same check as main file as it should switch to path if fs isn't defined. i dont have a 0.6 to test on. @cowboy can you confirm?
We're not doing anything special with `block`. Would this be cleaner?

``` ruby
yield 'count', count
yield 'min', min
yield 'max', max
```
Looks like this function now returns all tasks to be run, not just the baseline tasks?
$ sign missing before _last_statement.
This is unsafe, they're accessed by different threads. At the very least, you'll need to suppress compiler optimizations with `volatile` but preferably you'd guard them with a `uv_mutex_t`.
Shouldn't this be called ...RepairActions... instead of ...PairActions... ?
What are these <% delimited blocks for?
should these be in wiki views? I guess then they'd be /en-US/docs/files/* ?
indentation off I think?
Split to new lines, I really do not see the point in docs for these.
Should not return by reference according to my understanding of php.
There are sample files for nginx and lighttpd. You need to explain to people who are capable of configuring their web server what phpbb needs. Meaning, something like "make /foo url rewritten to /app.php/foo".
Please document what you mean by "ham".
Please use parenthesis to clarify logic. I'm not sure which of the following is intended:
`if ((!$attachment['in_message'] && !$config['allow_attachments']) || ($attachment['in_message'] && !$config['allow_pm_attach']))`
or
`if (!$attachment['in_message'] && (!$config['allow_attachments'] || $attachment['in_message']) && !$config['allow_pm_attach'])`
I suppose the second one doesn't make much sense, but regardless, having parenthesis to clarify the logic would greatly help.
nit, super long line
fs and sf ... seriously ...
Please use strings as key instead of objects to allow GC. Also, in case you manually set a field, please use the form

```
@ast["field:domain_name"] = 'value'
```
It will likely only fail during a very strange situation, since the `fs.fullPath` came from the filesystem.  This could happen, for example, if fullPath is a symlink to nowhere.

The practical affects of failure is that `mojito start` fails.  This end result is desirable (I think) because Mojito would not have been able to use that file during runtime if it can't load it during server-start time.  I'll add try/catch and good errors around this.  (I suspect that `readFileSync()` would have reported the failed file name, but it's best if we report it just in case.)
no reply :)
Really?  I think that might be controversial.  As an app developer I might want my local model to override my global model.

Looking at the implementation of `get()` I think the following will result in the local model being returned from both calls:
```javascript
Y.mojito.models.cuba = localModel;
addon.get('cuba');  // returns localModel
addon.registerGlobal('cuba', globalModel);
addon.get('cuba');  // returns localModel
```
in ... chera per per mikone too hover !!!!!
asab vasam nazashte
As agreed, `RotatedGeogCS` should now use containment for the ellipsoid instead of inheritance.
I'm not sure, I copied the code in Sort.infer_shape. I can change both.
Try:
     private static Map<String, String> m_partitionColumns = new HashMap<String, String>)_;
All right, I will add corresponding comments to children classes. Thanks, John.
Would "file_path_in = os.path.join(basedir[2:], filename)" be equivalent? It seems like it would be clearer and more consistent with the next line.
This looks like a bad merge/rebase: the `ClientType` argument was removed from `riak_kv_keys_fsm_sup` by #433.
Update the declaraction to reflect changes.
We should continue this discussion in Anssi's thread about the same area of code above.
Yes, it is a mess, but improving how this works is going to take a massive overhaul to how models work (see Anssi's comments above). SQLite doesn't support schema changes anyway, so anything we do to it is going to be fragile-ish, but at least the code in there works and doesn't leak state to everything else (it did for a while, the tests caught all that).

If we can find a good solution to the models problem, though, this goes away.
There is the possible clash with the previosly defined options, isn't it? What will happen if we have 'settings' in the REQUIRED_FIELDS?
Shouldn't the database feature for transactional DDL be consulted about this?

In current databases, where transactional DDL is not supported, DDL generates an implicit commit, which would bring Django's account of the transaction state out of sync with the database. Things may break when somebody uses the API in a way which includes both DDL and DML.

Further, I think it could be reasonable for a database engine which doesn't support transactional DDL, to flag DDL commands under a transaction as errors (although I'm not aware of any engine behaving this way). With current code, the backend for such an engine would then need to override start(); this is not terrible in itself, but start() does other stuff as well.

So, even if you think it unnecessary to check for can_rollback_ddl here, I suggest that the lines controlling the database transaction and its Django management be taken out into their own, separately overridable methods.
Thanks, I'll change that.
I will disable it as soon as I have fixed the last q/a test
Do I need to replace all the Eigen vector, matrix to columvector and 
Matrix?

On 2012-08-16 17:53, Feng, Xixuan (Aaron) wrote:
> In src/modules/crf/linear_crf.cpp:
>
>> +};
>> +
>> +class LBFGS {
>> + //shared variables in the lbfgs
>> +public:
>> + //shared variable in lbfgs
>> + double stp1, ftol, stp, sq, yr, beta;
>> + int iflag, iter, nfun, point, ispt, iypt, maxfev, info, bound,
> npt, cp, nfev, inmc, iycn, iscn;
>> + //shared varibles in mcscrch
>> + int infoc;
>> + double dg, dgm, dginit, dgtest, dgx, dgxm, dgy, dgym, finit,
> ftest1, fm, fx, fxm, fy, fym, p5, p66, stx, sty, stmin, stmax, width,
> width1, xtrapf;
>> + bool brackt, stage1, finish;
>> +
>> + Eigen::VectorXd w;
>> + Eigen::VectorXd x;
>> + Eigen::VectorXd diag;
>
> Since you have been "using namespace dbal::eigen_integration",
> "ColumnVector" instead of "Eigen::VectorXd" is preferred. Also
> "Matrix" instead of "Eigen::MatrixXd".
>
>  --
>  Reply to this email directly or view it on GitHub [1].
>
> Links:
> ------
> [1] https://github.com/madlib/madlib/pull/199/files#r1398135
You mean, only use translate3d with no fallback? That'll be okay, for the target devices… right?
@Nezz - This look right to you?
Right. Do you think anything should happen? Or should it just silently ignore the bad formatting?
We can ditch the idea stuff here, too.
For instance `python.rb` does:

```rb
    # Don't use optimizations other than "-Os" here, because Python's distutils
    # remembers (hint: `python-config --cflags`) and reuses them for C
    # extensions which can break software (such as scipy 0.11 fails when
    # "-msse4" is present.)
    ENV.minimal_optimization

    # We need to enable warnings because the configure.in uses -Werror to detect
    # "whether gcc supports ParseTuple" (https://github.com/mxcl/homebrew/issues/12194)
    ENV.enable_warnings`
```

Both of which are not required with superenv. The first because CFLAGS is not set. The second because -w is only applied during make operations.
Will drop in next commit.
Accidental extra indent?
What is #281 replacing it with?
As you can see the request contains information needed to identity the session that will be obtained from the store. This is needed here because you need the session when you execute the intent, so I don't see what the problem is. This request is tightly coupled with the store because this is the way we want RAIN to work: retrieve only a component's session.
Would be better off as an integer **routeType** constant (file names change, but constants live forever)
shouldn't this section be called "Spider contracts"?.

Mainly for explicitly sake, and also consistency - for exampel the "XPath Selectors" section is not called "Extracting data".
Using the name (Strings) has to be slower than using the ids, no?
Yes it should. A line after the class declaration looks good. Without it the fields all get squashed.
No--it's to set the map/list icons. (The toggle bit is just to make sure it keeps the class it's supposed to have.)
Oh, I guess the promise returned by `$.get()` has a `url` property?
It seems more useful to select the entire line including newline, so that if you hit delete the line is completely removed (instead of leaving behind an empty line).  To do that you'd want to select to col 0 of the _next_ line.
I was thinking in markdown :-) Fixed.
This is ok (same conversation as another thread).
I looks like everything is serialized correctly here. I don't think we need to make any further changes.
This is the root ``name`` in package.json
Ideally this should be based on the editor mode, since the file extension -> language mapping will be under the user's control.  For now, could we spin off a cleanup bug or add this as a bullet in  the file-extension user story?
I'll add it in, thanks.
Did you test this change? This significantly changes the behavior.
No need to recheck - already know db_handle->db != NULL
also of note it'd be better to use Django's builtin simplejson in cases like this

django.utils.simplejson
re ad 1: I think you should read discussion on that pull request we introduced [class-based views](https://docs.djangoproject.com/en/dev/ref/class-based-views/#formmixin). ;-) #89 #79

re ad 3: Yes, but the main idea of going to MongoDB was to have only one object/model/class for users and not user + profile. ;-)
Bah, I can't put this on the right line of context, but if the publish fails then you need to change re-assign self.publish as well
Instead of always calling `runCheckFn`, do you think calling `error` on an error event is more appropriate? Then, we'd only use the check function if there was no error loading the script.
I think we should be testing the actual `store_dir` that will be used in development/production environments, not the one that is used for tests, no?
I not sure whether you have pull my lastest two request or not because from
what i view on your master,  i didn't saw any changes reflect it.

https://github.com/nvcHilary/CoinCod/pull/6

For, the path changes, I will merge your files after i have confirm the
pull is already accepted. Then I will make a new pull to you for the path
changes.



On Mon, Jul 16, 2012 at 10:11 PM, Hilary Ho <
reply@reply.github.com
> wrote:

> > @@ -6,7 +6,7 @@
> >
> >  <body>
> >  </br>
> > -     <form action="http://<?php echo $_SERVER['HTTP_HOST'];
> ?>/CoinCod/Login_Form/login_func.php" enctype="multipart/form-data"
> name="myForm" id="myForm" method="post">
> > +     <form action="http://<?php echo $_SERVER['HTTP_HOST'];
> ?>/CoinCod/Login_Form/" enctype="multipart/form-data" name="myForm"
> id="myForm" method="post">
>
> @nvcVincent: See this? I thought we discuss this before?
>
> @nvcDeb: So we change this directly? but if we test it in XAMPP, it will
> show error in our localhost.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nvcDeb/CoinCod/pull/39/files#r1169333
>



-- 
Regards,
vincent
真别扭，还不如改成  remove_style
I stopped just this side of actual debianizing of the package, since we need it to keep working
right now for daily builds and the next release.
If this is all about logging, is there a reason you did not call this LoggingEventTarget?
I see. That seems reasonable.. I suspect to support that the wire protocol
should grow message ids, so you can correlate requests and responses. And
then you could theoretically do things like leave a wait running in the
background while issuing other commands.

On Friday, December 7, 2012, Conrad Irwin wrote:

> In bin/einhornsh:
>
> > @@ -68,6 +88,15 @@ EOF
> >        ehlo if interactive?
> >      end
> >
> > +    def wait_loop
> > +      begin
> > +        update = send_command('shell:wait')
> > +        puts update['message']
> > +      end while update['wait']
> > +    rescue Interrupt => e
> > +      send_command('shell:unwait')
>
> I wanted to be able to stop waiting and then run other commands. Without
> this I think subsequent commands would see messages intended for the waiter
> (as they're both multiplexed on the same socket). Now I think harder, this
> solution is a little racey, but it should be fixable.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/stripe/einhorn/pull/14/files#r2346585>.
>
>


-- 
Sent from mobile
When the master process frees the lastest cycle, notify all the worker processes with the generation of this cycle.
don't worry about it :)
I agree with you, the problem is if you look at the line where we copy the files in from the gem - the folder there includes the version of the gem and thus needs to be sharply defined :/
We'll need to propagate this change to system_groups/errata/_index.html.haml, which was a recent merge to master.
Do we want this one yet?  I guess you could update it again anyway.
Yop, we have this pattern already somewhere, +1
that sounds offensive :smile:
if {
} else {
}

and for consistency let's keep id and prop in the same order in both blocks.
I think that should still be possible.  The capturing listener usually allows the event to propagate. And even if we switch to keydown instead of keyup, apps will still be able to do their own event handlers for both keydown and up here.

Hmm...  Actually maybe we don't really need a capturing handler here.  Maybe the issue is that I need to take keyboard focus away from the current app when I display the task switcher.  I've noted this here:   https://github.com/andreasgal/gaia/issues/753#issuecomment-4559674
One more bug! Incomplete sentence here.
第一种封装说实话我没想明白，因为把所有传过来的信息并起来就意味着信息丢失，哪些是POST的，那些是GET的是需要区分的，甚至以后还要又PUT和DELETE操作的话，就更加混乱了。

我的倾向其实是尽可能保留原始数据不动，要动的另外建个数据对象（url），其实你很难完全禁制底层拿到GET或者SERVER的信息的，除非全部封装掉抹掉，但我觉得没太大必要，Logic不该用get()我觉得是个MVC关系的问题，不应该在这个层面上解决，因为我传一个$url还是传一个$_GET个体一个logic function，其实没啥区别。

我坚持的是不要改get()能获取到的数据，这个改了会非常奇怪。
Cross-node call here that defaults to 5 second timeout. Is 5 second timeout reasonable? When this call fails or times out, is throwing an exception and crashing acceptable? Or better to catch and return some error?
'WMS Import'
Brilliant. Good insight about making it look like a standard CRUD action. I'll give that a go.
Are we going to add these to all the files?
From my phone.

It is a wrong choice to make it part of the cell. A magic loaded as an
extension might want to add its own highlighting.  And you don't want to
look through all cells to monkey patch them. Hence the 'config' object
which is easy to alter.
Le 10 août 2012 03:37, "Brian E. Granger" <notifications@github.com> a
écrit :

> In IPython/frontend/html/notebook/static/js/config.js:
>
> > @@ -0,0 +1,33 @@
> > +//----------------------------------------------------------------------------
>
> I am not sure I consider this to be config information. It is more a
> static list that is associated with the auto-highlighting code. The people
> who should modify this list are not users but authors of cell magics.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ipython/ipython/pull/2216/files#r1348755>.
>
>
I see you are processing the pyout message type, but what about the other message types associated with output (stream, display_data).  Shouldn't we also forward those in some way?  I suppose we could take those other message types and put them into the `_repr_*` method output as well.  The resulting `ExecuteReply` object would end up flattening these different message types, but they would still be there.
Fixed. I made it a ``size_t`` because I feel it's more consistent, but I can change it to ``unsigned int`` if you prefer.
:fire: :finnadie:

Not sure there's anything else that needs to be here, especially since `free()` is "virtual" anyway.
Damn it - I knew that one had gone and picked the wrong one!
Nit: in this and the below, the `[]` isn't needed. This statement is equivalent, because of generator expressions, and to me seems more readable.

    dict((o.slug, True) for o in OPERATING_SYSTEMS)
I like short. Also, I like killing the possibility of two randoms coming back equal.
There is no reply form. Is that intentional for now? If so, comment this out.
We could. Although I see that it exists in master too. Wonder if there is a difference anyhow. Hmm, maybe keep it as it is.
Is it good to loosen the type requirements?
Yes. this is true. I can put that in.

On Thu, Aug 30, 2012 at 2:29 PM, Taavi Burns <notifications@github.com>wrote:

> In pyconca/templates/event.mako:
>
> > @@ -55,6 +55,15 @@
> >                  <img src="${request.static_url("pyconca:static/psf-logo-317x71-alpha.png")}"
> >                       alt="">
> >                </a>
> > +
> > +             <br>
> > +             <br>
>
> Heroku is a Bronze sponsor, yes? Shouldn't there be an extra heading for
> them after silver, instead of bunched in with the PSF under Venue Sponsor?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/pyconca/pyconca/pull/40/files#r1500005>.
>
>



-- 
Mahdi Yusuf
variable != constant =)
None of the other loaders does that atm. I could possible fix the docblock.
I don't think it should extend the new interface. This would be a BC break as it would force to implement the new interface on all existing generators (thus breaking things if someone implements the interface on its own). Simply add the new interface and implement them in the core implementations, and do an instanceof check in the RouterListener.
there is a missing space before `.=`.
for side effecting methods like this, convention dicatates the use of {}:

```scala
def bindParameters() { hasNewParams = false }
```

otherwise it’s difficult to discern whether you mean to do do the assignment, or if it’s a bug and you instead meant:

```scala
def bindParameters() = hasNewParams == false
```
Based on boomerang's coding convention, variables should be declared at the start of the function
这里好像中英文对不上……What about {基于LaTeX的北航毕设论文模板设计}{Design the BUAAthesis Template with \LaTeX{}}
Mit der Annotation HasLifecycleCallbacks() markierst du, dass diese Entity zu bestimmten Events bestimmte Methoden automatisch ausfuehrt. Das ist also der generelle Schalter, dass wir das fuer unsere User-Klasse wollen.

In der Methode setInsertDate() nutzen wir das dann. Dort habe ich die Annotation @ORM\PrePersist eingefuegt, dadurch wird sie immer automatisch ausgefuehrt, direkt bevor ein neues Entity persistet wird. Wir sparen uns also immer das setInsertDate() manuell zu machen (deswegen habe ich das im Fixture auch geloescht), es passiert jetzt von alleine. Das gleiche macht zum Beispiel auch mit einer Methode setLastUpdate() Sinn, denn die soll ja IMMER ausgefuehrt werden, wenn sich ein Entity aendert. (Haben wir nur bisher noch nirgends)
Does this do nothing now?
Я понял, про что ты. Но надо подумать, как сократить вложенность. Её неудобно ни читать, ни поддерживать в коде.
the smallcase "count" is confusing: If it is a SQL function it should be COUNT(), if it is a column, it uses an illegal name, and should, at least in MySQL, be wrapped in `backticks`.
It isn't possible, just as it isn't possible to attach more than one field_ding_news_title_image and one field_ding_news_lead...
Thanks for catching this.  I've updated the branch.
owner は埋め込めばいいんじゃないか
I think that we also need to bump MANAGEMENT_API_MINOR_VERSION for this patch?
Yes, if the model were to return something other than an array or false, it would throw a warning here.  Although the existing code would too.  If we want to make sure that the talks model has a consistent output, it would probably be something to add to the unit tests rather than a sanity check in the controller.
That's the interesting part! I'll have to go back to the original issue. Maybe we can address this higher up.
Hmm.. so, to sum up, you two want the resizable complex markup handle test to be a single test that is located in the draggable testsuite? Seems strange to me. Have I missinterpreted?
Also looks like an accidental change
sry, my fault. I worked on version from 15.12. fixing it right now.
I understand that  'options' parameter is optional, so why it isn't on the last position? Do we have to put cb as the last parameters? There is nothing about that in our coding standards, and we always need callback in here.
Definitely you are a one-liner :p Including it! Thanks :-)
Can you leave this code. It is needed for some demo purpose :)
I made a judgement call on "top sites - you don't have top site yet". It told the user they didn't have any of a thing they didn't know about. Ridiculous. In the spirit of less is more I stripped it out and left the remainder. I take the point, however, that what's left is a bit silly in context, given it's repeated directly above. Let me talk to Larissa and Patryk about this. We can polish this post-Feature Complete.
you can determine if a behavior is defined on the table using `hasBehavior('name')` btw
Just a quick note.  

You can avoid this 
```cpp
QVariant( (int)mFeatureIds.size() )
```
by just doing
```cpp
QgsExpression::setSpecialColumn( "$numfeatures", mFeatureIds.size() );
```
setSpecialColumn takes a QVariant which can implicitly convert from int (or pretty much anything).
Yep!  I'll remove this then.
sau geil! :-D
I will make it a one-liner ... the `return` is necessary here, otherwise it will return an array with only one value ... :wink:
No need for the first `a > b`, since `eps` should be positive.

Also, should we move these methods out somewhere they can be shared? The first one is duplicated in a couple places.
this is a little weird though, let's just do:

```js
exports.merge = function(a, b) {
  for (var key in b) a[key] = b[key];
  return a;
};
```
Sorry if I came of as condescending there - I didn't mean it that way :p
There's no real point to making this and nestedClassRegistration protected. There's nothing in the methods that is worth overriding. Once you have createCommand() protected, you can just override create() (you might want to have create a method that create() calls that also has the instance,  and allow that to be overridden (make create() final) and call your own methods to register commands differently.
I found many of the above removed debugging statements helpful when tracking down problems.  Especially when working on virt-autotest iteself, it was nice to be able to see step-by-step what was happening. Maybe some of them were redundant, but for sure they weren't all redundant?
..that can be used to post a message back to Gerrit directing users to the build status. 

(since I bet that will catch more peoples' eyes)
coffee’s `not in` is even better, anyway. I changed it in e65c7e9be4a129c5c75e41b5ca0793171a75c05c
s/starting/start/ no?
This would yeld!
The timeout will be put in the events queue, and stay there until picked.
The loop I used won't yeld.

I don't see why this is a good idea, sorry.
Oh, gotcha. Thanks!

That'll prompt a crash course in writing tests (I've not done much of it before). That's a good thing, by my estimation.
Oks for then `mark_corresponding_notifications_read`
Some work needed here to make sure we only call this for actual gallery pages. But then this whole file needs a bit of revision in that sense...
I think we should add a warning here if HAVE_NFQ_SET_QUEUE_FLAGS is not defined.
I'm not sure either. I rebased against master and that showed up. Any suggestions? New pull request?
Also make sure `impersonate` throws no undocumented exceptions.
What is label?
ugh I'm firing myself as code reviewer for accepting code with a typo
there is a missing ~~comma~~ **semicolon** at the end of this line :)
maybe console.exception probably would be better
Could you add another test to check the use case where `top` !== `parent` ?
Missing coma after once. `resolve only once, all subsequent`.
@seanmonstar - I am unable to set this variable using the command line without an exception being thrown - am I being a buffoon?

```
➜  ~PERSONA_ROOT git:(1931-kpi-per-platform) ✗ KPI_SAMPLE_AGENTS="{'Firefox/20': 0.5}" npm start   

> browserid@1.0.0-b1 start /Users/stomlinson/development/browserid-remotes
> node ./scripts/run_locally.js


/Users/stomlinson/development/browserid-remotes/node_modules/convict/lib/convict.js:171
        throw errBuf;
              ^
kpi_backend_sample_rate_per_agent: Instance is not a required type: ["object"]
npm ERR! browserid@1.0.0-b1 start: `node ./scripts/run_locally.js`
npm ERR! `sh "-c" "node ./scripts/run_locally.js"` failed with 1
npm ERR! 
npm ERR! Failed at the browserid@1.0.0-b1 start script.
npm ERR! This is most likely a problem with the browserid package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./scripts/run_locally.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls browserid
npm ERR! There is likely additional logging output above.

npm ERR! System Darwin 12.2.0
npm ERR! command "/Users/stomlinson/development/nvm/v0.8.12/bin/node" "/Users/stomlinson/development/nvm/v0.8.12/bin/npm" "start"
npm ERR! cwd /Users/stomlinson/development/browserid-remotes
npm ERR! node -v v0.8.12
npm ERR! npm -v 1.1.63
npm ERR! code ELIFECYCLE
npm ERR! 
npm ERR! Additional logging details can be found in:
npm ERR!     /Users/stomlinson/development/browserid-remotes/npm-debug.log
npm ERR! not ok code 0
➜  ~PERSONA_ROOT git:(1931-kpi-per-platform) ✗ 
```
Ah, ya.
@ozten - Do you want to take this out?
I don't yet grok how these IDs are used or what the rules are, but if they only need to be unique and mapped to the original email addresses, could you replace them with sequential integers? That'd always be safe.
I forgot about this, thanks for the suggestion.
why do we have to add in localhost?
This way of raising will lose the original backtrace. I think we want to use the `detail` as the second arg to the error so that we can preserve the cause of the Puppet::Error.
Please be super, super careful about the upgrade case -- specifically, going from an old version which looked at the previous pidfile to the new name. IIRC we did this moving from 'puppetd.pid' to 'agent.pid' in the RH init scripts in the 2.6 era and broke upgrades.


On Jul 20, 2012, at 11:42 AM, Matthaus Litteken wrote:

> 
> changing it to /var/run/puppet-agent.pid might be a good idea, if we plan on making similar changes to the master init script.
@pdxmph 

Thanks for taking the time to submit a patch to Puppet.

I don't mean to nitpick terminology, but I'm hesitant to phrase things this way because we've had, and have, problems keeping our documentation in sync with the behavior of the code.

Would you object if this description focused on the intended behavior instead of the actual behavior?  I find focusing my explanations and descriptions on intent allows the information to remain relevant for longer periods of time.

So, perhaps, "The upstart provider is intended to manage jobs in the `upstart` service management framework on Ubuntu systems.  For `upstart` documentation, please see <http://upstart.ubuntu.com/>."

I'm not sure we should explicitly state that other platforms are not supported.  Why is the current description a problem?
Nobody asked my opinion but I'm going to give it anyways.

This example has lots of set up that might make more sense pulled out into a before block, e.g.

```ruby
context 'with a stubbed catalog compilation' do
  let(:compiler) { stub('compiler') }
  let(:catalog) { stub('catalog')
  let(:caches) { [:known_resource_types, :env_module_directories, :gem_directories] }

  before :each do
    Puppet::Parser::Compiler.expects(:new).with(@node).returns compiler
     compiler.expects(:compile).returns catalog
     catalog.expects(:to_resource)
  end

  it 'should clear the thread local caches before compile' do
    caches.each do |cache|
      Thread.current[cache] = 'rspec'
     end

    Puppet::Parser::Compiler.compile(@node)

    caches.each do |cache|
      cache.should be_nil
    end
  end

end
```
@sciurus Probably need to skip this on Windows. You may need to do something like:

<pre>
FileUtil.chown(..) unless Puppet::Util::Platform.windows?
</pre>
is there a typo in this description?
Please don't use .map + .compact, just use .each and append to a local. The array in question could be very big and then we'd just turn around and make it smaller again. Better to just do it right the first time.
That's fine with me. Personally, I think the core extensions make the code nicer, but I understand your point.
not even used, removed.
In fact, a number of PRs were merged for 2.0.4 for precisely that reason: to ensure that the Response object composed in the MvcEvent object is the one returned by a listener. That object is the correct one to introspect at any given time, not the one in the service manager or the application instance. (I'd argue that we likely should not register the Request/Response objects in those classes.)
Missing file-level docblock
flottantes
Is there any particular reason why you don't use `self.segments` and instead pass the local variable `segments` as an argument to other methods? I think other classes depend on `Renderer.segments`, and I can't see any differences in the behavior between the two options, so it would be nice if you could explain the reason for doing it this way before I merge it.
I know. I didnt' realize these were changed as a consequence. I can have this fixed in a bit
why are you using "part"?
Might be worth grabbing @philandstuff or @nickstenning who may have some ideas on that? I agree that one worker per app in dev should be sufficient.
I blamed this line of code to me from 2008... I don't quite remember the context of this.

Now, I'd fell very comfortable fixing this if this was a bug since then. However, if this bug is new (and it's pretty obious IMHO) then there might be another root cause I'm missing.

I did some ad-hoc testing on my own, but please do keep an eye open for glitches in other recursive connections.
thus, it has nothing to do with the feature you are adding in this PR.
One `if` statement should do here.
OK, fine. :)
We (JBoss) don't really recommend the use of `@EJB` injection in Java EE 6. Hence my comment, we shouldn't be showing the old, less good, way of doing stuff.