labofsahil/github-event-dataset-2012 huggingface dataset:
Yep - it seems that the growl gem must be referenced for guard to send notifications using growl - otherwise you must refer to the terminal running growl all the time which is undignified
no, it won't. In the old version (newServers) we also add the existing ones - this is redundant, causing the misleading log message, this JIRA is supposed to fix.
先merge了我再改吧
Not sure why this keeps changing?
yeah, just checked jboss logging impl, I misunderstood it
Link people to the old map until Spoutcraft/SpoutAPI are more complete?
ok. I remember now.
getSystemPrincipal() doesn't take a URI parameter.
yup, totally agree.
I have found the cause of this issue and am fixing it now, so I will revert the revert and test on staging then deploy to production.

On 18/01/2012, at 7:39 PM, Michael Klishin wrote:

> By "do not revert" I mean "revert and deploy to production". It was possible to reproduce it on staging, too, and affected all project and builds. So we had to fix it fairly quickly.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/travis-ci/travis-ci/commit/fb300ff590abc97d527e8c28f407b72e3c89b1b5#commitcomment-879148
I got some feedback from apenwarr when I did this, but never got around to tweak it - probably should do that, but can't promise when.
ensure entries are the exact same in upgrade and database.sql script (note using TINYINT(4) below and just TINYINT here)
Since `SchemaManager::doGetDocumentIndexes()` doesn't differentiate between embedOne and embedMany, that looks like it would work; however, it's not possible if the index should also include a field from another document (embedded or otherwise). I'm working on a fix for this in #225.
I don't think you need this, if you didn't hit xml content then why still add the stylesheet?  You can probably get rid of the "stylesheetLineAdded" variable entirely.
This method actually is problematic.  Use doGetThumbnailAsStream() instead.  It does not work when you are not using FileSystemStore: 

http://issues.liferay.com/browse/LPS-25822
I don't know a thing about SVR. Should this go to the SVC as well?
На мой взгляд можно было бы хоть какое-то разнообразие сделать за счет RuntimeMemory или разных watermark. Вот нет у меня уверенности что в текущем варианте все работает как надо )
When I tried it without that nothing happened, which might explain why, but I wasn't sure why including part of the DOM makes it work.
Can this test get closer to a unit test? Right now, it relies on i.e. the exact url of the json version of the search feed. That would be perhaps more of a functional test. (Alternatively/in conjunction, perhaps also add a functional test for this.)
we're trying to focus on what we can't calculate with cm1. lots of other 
orgs today are making the broader enviro points.

On 2/13/2012 10:34 AM, Derek Kastner wrote:
> We might want to add that that's just the CO2 from using the oil and the oil is "extra dirty" due to the way it's produced.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/brighterplanet/numbers/commit/1dd8dc3c7feb32c7b20a3328d4bc5d56a25a2d71#commitcomment-961248

-- 
Andy Rossmeissl
co-founder and product design director
Brighter Planet
aim: brighterplanetar
office: (802) 458-0441 x302
freenode: #brighterplanet
twitter: @rossmeissl
http://brighterplanet.com
Yeah, that's a bug
Haha, see how our clients' con lists look?!
Hi Ryan,

What does '<table value=\"${allWidgets.collection}\">' mean? I didn't think 'value' was a valid XHTML attribute for 'table'?

Richard.
Yes, this is the S in SOLID, Single responsibility principle.
There's a lot going on here in one line.

Also, we're working on a hash, but returning an array, which I think is a bit confusing.

That is probably because you're trying to work with my existing code, but we should likely refactor to make that more readable and intuitive.
I'm the hip-hop-opotamous, my lyrics are bottomless .... yeah bottomless.
Instead of "six is same as", I think this should read, "Six is the same as" (capitalize and add "the"). The non-capitalization is also there in place_value.html, though -- anybody thinks it's a feature?
Intended.
Biohazard needs to make atleast one fix commit after the first one.
I can confirm this. LoS indeed only has to be checked at begin and end. Channeled casts will indeed continue even if LoS is broken. im 100% positive (long time retail player). And im sure there are enough vids around to confirm this.
total++ would do here...

(though not really sure what the point of allocated() is )
I just use my own one. Just a css file with Eric Meyers reset in it, a bare HTML5 document and maybe jQuery. I think things like HTML5 Boilerplate are far too OTT.
`...unless params[:sort] or params[:ratings]`
Why you push a change that is already committed, will close this and add weather plugin manually
Thanks for the explanation again. I think the offset guessing would have been solvable (especially given the failing test case). 
I may contact python-dev about the issue (since allocation finalizers on the heap makes all tests pass).
I should spent my time on preparing the next release...
This fix doesn't seem to be included in the gem that gets installed by running "gem install osmlib-base"?
I seem unable to dump xml due to this bug.
Hmm... yes
I think you should use [isInstantiable](https://github.com/vasiliygagin/cucumber-jvm/blob/e5e6153154c21f1f73d741db6fc46d2d6f436b5d/core/src/main/java/cucumber/runtime/Utils.java#L17) here.

I'm not sure I understand the check in  while condition.
more details http://bugs.jquery.com/ticket/9134
Interesting, still -> is cleaner, and much more intuitive, especially when mixing with >=. It is also used in OCaml and C++11. I also think that => is bad choice, not only because it is less clear than ->, but now it could be possible (in theory) that one will write => when one should >= and program will compile anyway without error message.

So compiler now perform lazy typing in some sense, because before that D was only doing forward type inference. Now it is more like full type inference known from Haskell or OCaml (aka Hindley–Milner algorithm).
Gives errors
Useless. Prefere to use the Tapestry t-invisible class
This can be ignored...
necesita test en RS, en mi server funcionaba antes de este arreglo así que no puedo ahí, a ver si me coges ON, me llevas un momentito y lo vemos.
> Use camelCase, not underscores, for variable, function and method names;
> Use underscores for option, argument, parameter names;

On this case, we should let $direct_complement and $indirect_complement

Same thing on each parameters/arguments
Yes it is, and no I don't have the libs to really make it work properly,
but it works with the old libs.

On 19 January 2012 18:44, kalltkaffe <
reply@reply.github.com
> wrote:

> Is the purpose of this commit to make the kernel compatible with newer and
> more ICS-friendly EGL libs? If so, where can those libs be found?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TomGiordano/kernel_zte_blade/commit/c02c7dfddf7f05ff8e19c0012248507e51d1edee#commitcomment-881019
>
Os browsers capazes de executar código escrito em JavaScript reconhecem o elemento <script>. É dentro desse elemento que se coloca todo o código.
**getMessage** may produce null
Is it not better to use bean override configuration. So push the values in rather then the service pulling them out? i.e.:

     beans {
          pluginDeployService {
             deployUsername = "blah"
          }
     }
ConnectivityManager does not have a "getBooleanExtra" method - AFAIK, this is something you call on an intent. It returned null when I tried it out.

I can use getActiveNetworkInfo, though, instead of getting the array of NetworkInfo.
since ikarosdev's didn't update his repo, there are little changes at boardconfig and I removed header file such as hwcomposer.h.
And for my build I just repo sync to cyanogenmod and build, that's all and didn't change anything packages from cyanogenmod.

Unfortunately init.rc and init files from the last cyanogenmod repo kill radio function. But I could swap init files from gingerbeard rom and off course ril radio works, but the phone become unstable.
I don't think the sentinel approach is necessary if we do the decrementing loop approach you mention in a comment above. I think the following needs to occur:

* We should update the local timestamp when we download records for the reason I explained in a couple of comments down.
* In the upload error handler:
** If we have an X-Unmodified-Since failure, this means new commands were uploaded, in addition to the ones we just processed, so we need to redownload - abort sync and naturally a redownload will happen next sync
** Else any other error means we need to upload the record no matter what so keep trying in a decrementing loop

Additional Consideration - when we've processed commands and redownload them, how will we ensure not to re-appy them? Or does it not matter if we do?

I think the sentinel approach isn't super explicit so it might be a little difficult to read. Also, it makes us have to be more careful when using  `getServerClientRecordTimestamp()`
Can make a PR ?
И это тоже в IO зачем?
Hi! I opened a pull request here https://github.com/edoburu/django-fluent-dashboard/pull/2
I had some trouble in using fluent dashboard with the application i'm working on as you can see here https://github.com/edoburu/django-fluent-dashboard/issues/1
so i patched admin-tools and fluent-dashboard. The pull request on admin-tools has already been accepted :)

I added some more icons for some models:
- contents.png for ContentType (django.contrib.contenttypes)
- calendars icons for celery tasks and crontabs
- book.png for a personal app but could be used for every group of contents
I understand the need. I'd propose having the build callback return a boolean false to inform contextMenu not to show the menu (just like the show event behaves).
oh yeah I'm totally unfamiliar with this testing process. I'm probably missing something obvious, but I wasn't sure how to even run it. It may not matter. The browser test is fine though.
I'm thinking that removing this constructor was an accident.
По-моему, правильнее будет применить .toUpperCase() не здесь, а там где возникла такая ситуация. Мое упущение - при приведении ID к верхнему регистру (дабы исключить дубликаты названий модулей, представленных в разных регистрах), и не учел еще один источник проникновения неправильных ID - из старых сохраненных OSIS линков или из закладок. В обоих случаях идет создание экземпляра OSISLink, где и нужно привести ID к верхнему регистру - так будет логичнее.
Also, fix for multiple file inputs clear on krautchan
When records are no longer to be retained by a business, personal information in those records should be made unreadable.
Embarrassingly, this isn't happening any more and things are running happily on my laptop without gtools... I don't know what the above message was about! Can't reproduce it.
This could be improved using named identifiers (like `%(tax_rate)s`).
Similar comment as above. `cas-management`, etc?
You should be able to override `size()` and yet, leave the impl body blank.
This is just weird. Could you highlight which test it is that fails and why? (There's a bug with 2.12 that it wont allow running individual tests so I'd rather we move away from that)
This property already exists in base.py afaics
It does, but not when the file is loaded. See: https://github.com/Wikia/app/blob/dev/extensions/wikia/EditPageLayout/js/editor/WikiaEditor.js#L20
not without creating closures
Right, you would have to duplicate it. No good. Leave it.
probably not. I'll link to version 2.8.0 since that is now the dependency in akka. 
BTW, Camel 2.10.0 just came out 3rd of july, which I think has breaking changes. I'll add a ticket for upgrade.
I don't really like the interaction with the error handling on this particular call, the thrown exceptions from the circuit breaker never make it back to the caller.
- Same logic as before; I'd guess that doing everything at initialization time will save a few cycles compared to initializing then populating later
I would feel more comfortable with a `join` here like `Rails.root.join("/public/404")`. I'm always nervous whenever i'm building url's and paths over whether to include or not include a slash. Join takes care of that for me.
Not currently, though it wouldn't be too bad to add
It would just require a validation of a iso8601 date string
Not a valid syntax in 2.4.
Agreed on either way. Harder for me to realize the distinction the way things are viewed in github; once I realized one was in agent.lib and the other in gofer.pulp it was more clear.
Nice, glad you caught it in these sorts of places.
You like the inject?  similar to how it is used in find_module.  The array may be a little overkill, but I was thinking that if it was :name=>[:account, :right, :content] or :codename=> ..., that might be usable with the same just opts profile in the class method as well.  The fetch_* names would be related to options like Rails does with find_by_field, etc.

But this is plenty for now.
I'm not sure plot is actually needed anymore.  If I understand, it is sort of like map, but goes down a tree of nested array.  Is that right?
I think maybe I should use just the ID.  That's the main value I need anyway, and I can rely on that much being constant.
@jmarshallnz see the note in the commit message. This is to fix compile until the split-res comes in. after that, this one gets nuked.
Can be integrated into the above if-condition.
This doesn't seem to delete the usbListenerDir and the udev rules.
# Graphing options.
A positive filter for `Procedure-Top`, negative filter of `Top`.  But even
without filters, it's important for it to have a type.


On Thu, Nov 29, 2012 at 3:13 PM, Asumu Takikawa <notifications@github.com>wrote:

> In collects/typed-racket/base-env/base-env.rkt:
>
> >  [call-with-continuation-barrier (-poly (a) (-> (-> a) a))]
> > -[continuation-prompt-available? (-> -Prompt-Tag B)]
> > -
> > -[make-continuation-prompt-tag (->opt [Sym] -Prompt-Tag)]
> > -[default-continuation-prompt-tag (-> -Prompt-Tag)]
> > -[continuation-prompt-tag? (make-pred-ty -Prompt-Tag)]
> > +[continuation-prompt-available? (-> (make-Prompt-TagTop) B)]
> > +;; TODO?: continuation? is missing
>
> What would the type of that be? Would it just have a filter for Procedure?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/plt/racket/pull/182/files#r2272534>.
>
>



-- 
sam th
samth@ccs.neu.edu
I prefer the yardoc [@option](http://rubydoc.info/docs/yard/file/docs/Tags.md#option) tag for documenting option hashes
The impact of this is that the parameter is fixed for the entire multiply operation - is this not contrary to the literature?
How can we test `Image.strip()` method?
Goes for the rest of the file as well.
A macro would be nice, like so:

```
#define TUINavigationOffscreenFrameRight \
(CGRect) { \
    .size = self.view.bounds.size, \
    .origin.y = CGRectGetMinY(self.view.bounds), \
    .origin.x = CGRectGetMaxX(self.view.bounds) \
} \
```

And the same for the left frame too! Nice and simple, and less methods. :panda_face:
This conditional is incorrect.
Really? Alright then, no qualms there.
Is it normal to have 2 times `:show` here and on line 133? :)
This is more complicated than this. After unsuccessful shutdown (i.e. when client hasn't yet sent `close notify`), server might need to wait for client to write it (that's why it's called *bidirectional*). So after `SSL_shutdown` node.js needs to emit `close notify` to client and call `SSL_shutdown()` again on any new data from client, or... terminate connection on timeout.

So I'm strongly against this change, because it needs to be implemented on higher level and account more cases. If you can just pull it out of this patch, it'll be ok for me.
We could just merge the two lines to take advantage of 120 column limit.  (Same for `AbstractNioBossPool`)
Always use { }, see CODING_STANDARDS
а что должно было измениться с линками-петлями? я не увидел отличий от qreal/master
Just a question that just came to my mind, shouldn't this be encoding now?
Can use 1.9 hash style.
@retronym: I added mention of this specific criterion in error message. Please have another look.
Lodged as [SI-6558](https://issues.scala-lang.org/browse/SI-6558).
I made the default PaymentMethod.auto_capture? call "Spree::Config[:auto_capture]", so the existing functionality would be unchanged.
You mean the code needs to be in a before block inside the context ?
of different -> of a different
Can't we simply extend matchers_identity here, instead of defining a new matchers_add and later adding matchers_identity + matchers_add?
We are in agreement (for anyone reading this) as I noted on issue 3041. Although zeros(0)[:,-1] should return an error, zeros(0,3)[:,-1] should not. I tried to smooth out the confusing set of functions that were being used to interpret indices last time; are you taking a crack at it this time, Aaron?
...colors, the following can be issued (starting with IPython 0.11):
On 13.04.2012 05:43, Christopher Smith wrote:
>>
>> -    def frac_field(self, *gens):
>> -        """Returns a fraction field, i.e. `K(X)`. """
>> -        raise NotImplementedError('nested domains not allowed')
>> +        for k, v in rep.iteritems():
>
> (unless rep is a global.)
>


No the diff is funny. You see here a bit of the from_sympy method.
I still think that hashable_content should just include the name, but that's auxiliary to this PR.
`==` should not be surrounded by whitespace
Went and removed it anyway in 70072a7 , until we really need it.
I think that's a bigger project and involves changing the interface in order to leverage the question mark syntax of PreparedStatement.
Can you de-obfuscate this?
I find it a bit confusing that `unit` defaults to None but `bounds` defaults to (-360,360).  This ends up particularly confusing if you do Angle(12,u.radian) because then you get bounds of (-360, 360) *radians*.  Perhaps there should be a special default that simply means (-360,360) degrees, regardless of what the input unit is specified as?
I think it was a one-time thing @espadrine added in gateway. I'll remove them, and we'll re-add something like this on every page if needed.
Fedor, you can turn the field in UV_STREAM_PRIVATE_PLATFORM_FIELDS into a void* and move the declaration of `uv__stream_select_s` into src/unix/stream.c.
Could you not just pass the function here (ie, change `{(e:Endpoint, p:DiffaPairRef) => participantFactory.createUpstreamParticipant(e,p)}` to just `participantFactory.createUpstreamParticipant`? You're just mapping the same parameters into place.
fix indentation, improve parallelism.
this looks incomplete.
a little nit :) if you could be so kind as to adding curly braces around your if statements that would be awesome :)
Don't mind it then. This fix is useful
No, `>=` is correct.
A memory cache in olympus was not a memory-only cache, but also handled deletion of compiled template files etc. The design is/was just bad. You can safely remove all those functions from a memory data cache.
This should be moved after the open call, and you need to retrieve the actual error message from sqlite and include it into the error message we return. There is http://www.php.net/manual/en/sqlite3.lasterrormsg.php for example.
This is generally a bad idea. If possible, avoid injecting the container and inject the dependencies directly instead.
add , after the last entries of arrays
Then just print {{form}} or {{form.as_table}}
This is unnecessary and confusing, because nothing should change at all. 
Please `git checkout project/plugins.sbt`.
This adds silently a new instance variable at runtime. (Same with `this._frameTimes`) I see three reasons why we should avoid that and instead introduce all instance-vars in the constructor or via "proto".  

1) Readability. The Reader can't see all instance-vars at a glance and also can't see the expected default value.
2) We should not encourage ourself to "pseudo-hide" code b/c it is not important or just an opt-in function. 
3) Conflicts with existing strategies of JS-Engines to optimise the execution of JS.
Why do hybrid apps need a separate dimensions.json file?
Woops, didn't change that part :(
I'll change the formatring, make it pretty, and re-upload the whole thing.
Uninformative log message.
truth to be told, the SKIDROW location isn't an official location for those games. It is a location a cracker calling himself for SKIDROW have chosen because of DRM(or so I believe)
using statement should be inside namespace declaration
Do not change the beautifications. Use jsbeautifier.org
I have a test ready for this, I will push it together with  desired changes mentioned above
If the `__eq__` method can return `NotImplemented` (as it can here), then it's better to let that pass through, rather than return `not NotImplemented` (which is just `False`). See `Coord.__ne__()` for an example.
This doesn't match the local docstring style. Nor do the other docstrings in this PR.
Seems like this code always expects ``rows`` to be constant-folded into 0, while the docstring says ``may be constant-folded``: either the docstring or the code probably needs a fix.
Talk it over with Andrew in deciding whether or not we should allow dollar signs in role names.
What if the user uses a Brazilian translation?
!==?
nah, `@config` would be undefined
This should be removed. It is not needed
Well, it's just a warning for me, so maybe it's harmless to ignore and leave it like you had it.  Why don't you add a comment and restore it to what you had.  We can figure it out later.
this line is irrelevant if you do https://github.com/jquery/jquery/pull/748/files#r884249
Too many vars now
If you have a strong opinion about either using tabs or not, please file a JIRA and add you arguments.
s/spezialization/specialization/
Isn't this breaking the back-off location pattern?
Seems like this whole block above of platform specific #ifs can go away.  There should be no platform specific code left in SpriteBatcher now as it fully depends on GraphicsDevice objects.
My stand on the helper functions just for one option is that at least three repetitions are necessary to make it worth the effort.
Was this revision chosen arbitrarily, or does it represent a specific version that the oiio devs mean to be used? The purpose of `devel` is for specific development releases.

While I like pretty tarballs, I don't know I'm so comfortable making up version numbers.
Then rather than run `make install` we can just use

```rb
bin.install 'foo', 'bar'
man1.install 'foo.1', 'bar.1'
```
Looks good, pulled. For the future note that changes like this should also be commented so that future maintainers can tell at a glance if they should remove the code or not---I added a comment during the pull.
yeah sorry, I meant something more along `this.includes = {all: [], pre: [], post: []}`
This actually doesn't work; b.overlays is itself a dictionary of Overlay classes.  As written it iterates over the dictionary keys and then tests for the presence of "overlays" in those key names.  Instead it should be something like

    for name, obj in self.overlays:
        for bar in ["pos_bar", "neg_bar"]:
            try:
                getattr(obj, bar).visible(visible)
            except AttributeError:
                pass

Or your preferred way of handling the logic :)
> Don't change getDefaultSupportedPrivilegeSet though.

Wrong copy/paste ;)

> Literally the only thing you should/could do is copy the body from getDefaultSupportedPrivilegeSet, move it to your local getSupportedPrivilegeSet, and remove set the abstract properties to false, where needed.

Sorry I'm a bit dense here :P Am I correct in:

* {DAV:}write privileges on a non-collection resource gives permission to update but not delete.
* {DAV:}write on an collection gives permission to update the collection and create/delete contained resources.

If so

* How do I give permissions to delete a collection?
* Can I give permissions to delete singles resources in a collection, or is the permission on collection basis?

Phew, I can understand why it was a bitch to implement :-P
Would a better name maybe be `qos_prefetch_count` so that it matches the names of channel variable that gets set and that it's clear that this is a Rabbit QOS related setting?
What do you mean? 
log somewhere special apart from the dispatcher log in case of test_inbound_event_routing_failing_publisher_not_defined.
Why are we not passing an error to the callback. This will generate an inconsistent state inside the middleware since it has no ideea that an error has occured and will expect a session to be passed to it.
Which did you have in mind?
```
my (@state, @time,  @date) = ((@h_state[@ok], @bs_state), (@h_time[@ok], @bs_time) , (@h_date[@ok], @bs_date));
```
?
With everything in named arrays I'm not sure how to do this nicely.  If I had nice in mind, I'd chuck all of these independent arrays and go for an array of hashes to start.
Why do you need a sleep if you're already flushing manually?
Quick, be cute - :unamused:

Sorry, too tired to code and messed up that one ;)  At least we know `equal` works ;)
I'm not totally sold on it either, but I figure it's possible we could make that function more complex in future. It does seem ridiculous that the line to call this function is longer than the function body.
To let the application server startup and listen to `process.env.PORT`
However, 0.5 second maybe fine for simple node.js applications - but not the best for a rails application or J2EE app because their startup time is considerably large
It shouldn't be all that many, especially because of the next line which purges now-in-sync tables from the session.
This has been changed.
old description was summary + more, just replaced the summary part
I agree with Bobby here *unless* you make parse return an object which can then be converted to other formats.
Why is this still here? This *is* just a bunch of spaces, right?
This might be as designed but that top padding seems VERY big
oh yeah, good idea!
it's kind of unclear why you have two different intervals here
While we're optimizing, may as well make this a static variable in the class. ;)

Is there any other property we can check on the `conn` object to determine if it's Py27+, so that we don't need this ugly statefulness + exception catching?
core modules can not be dynamically loaded
Tady nevím, proč nebylo použito, je to sice upovídanější, ale srozumitelnější a Last-Translator není hlavička ale jen součást hlavičky. Možná by mohlo být použito "..., aby mohla být použita ..." aby to více odpovídalo zdroji, ale význam se tím nijak zvlášť nezmění, takže mi to připadá jedno.
Again, why return counter?
Replaying import with uses here will make the plugin break because import will return true or false because it will try to load the file and we used it here to detect if this helper is present. To improve this in 2.0 this needs to be done somehow different now. But the way you changed it will make it break.
Actually the readme is in the project twice. I left the top-level readme.
Specs without version tags should be added as 0.0.1
That is as I understood it - that the code is what needs protecting and that the UI portion of the application doesn't need that Copyright notice especially upstream.
Good eye, I'll add this.
@xsuchy , Good suggestion.  This also got flagged by pylint.  Addressed it with commit 6b381e652f9387f61705042aaba90c179504b568
You should use "newName is not None" as it's the pep8 recommended way of comparing something to None, some classes don't properly implement == with None,  and it's faster too:

http://jaredgrubb.blogspot.com/2009/04/python-is-none-vs-none.html
Under what scenario would the user submit an empty request?
I think that empty array is a valid result. It's only that no system matches the query but it's no real error.
The last 2 sentences here aren't quite clear - the english just a little off I think, but it's also a bit on the technical side - hard to read.
We could pass a "Done" callback to make it easier to read but the reason this is the way it is is because there's one code path for all callbacks that the subscriber can handle. The logic is the same when a subscription is disposed.
Shouldn't this be returning deferred.promise()?
Remember we agreed to not introduce these style changes in the code until 3.1...
you may use `toLocaleFormat` to get the name of months from a date object (didn't investigate your use case though)

https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/Date/toLocaleFormat
Done in 94c8f79.
It might be wise to add a note either here or in a docstring for the `units` property about how to deal with the (default) `None` units.  That is, the following behavior is surprising to me at first glance:
```
>>> d = NDData(somearray)
>>> d.units == None
True
>>> d.units is None
False
>>> d.units = None #this actually isn't necessary, as it's the default, but it just makes it seem more counter-intuitive
>>> d.units is None
False
```
I understand that it's because we're getting a `Unit` object instead of `None`, but it's counter-intuitive that I can set to `None` and then not be getting None back out again.  So A note that indicates one should always do ``d.units == None`` instead of ``d.units is None`` might help things.
This part of the cython stuff does *not* have to be included in the whitelist checking stuff. It'll do what it's supposed to do if Cython isn't installed.  It's only the `setup_helpers.update_package_files` part that needs to be adjusted.
I really don't like #defines like this in an included-by-everything file like util.h.

It is reasonably likely at some point somebody will write or use a class with a dir_str() method and will spend a long time trying to figure out they're getting weird compiler errors.

Can we just use .string()/.string.c_str(), and live with "ugly" paths if you're running on Windows and not using BOOST_FILESYSTEM_VERSION >= 3 ?

PS: The change that prompted all this work is in my "not important enough to care" category...
When -par=1 this would cause no thread to get spawned for verification and matches current behaviour?
Agreed, bitmask arithmethic should be unsigned. Subsequent bit shifting can otherwise result in signed int overflows which are undefined in C(++). Though, scanning the code quickly, none of these happen at the moment. The largest shift we use with GetByte is 8.
Yes!


On Thu, Nov 8, 2012 at 5:31 PM, Brent Yorgey <notifications@github.com>wrote:

> In ItLang.hs:
>
> > @@ -39,8 +39,30 @@ type Mem = M.Map Var Nat
> >
> >  ------------------------------------------------------------
> >
> > +-- | Add two natural numbers.
> > +add :: Nat -> Nat -> Nat
> > +add (S x) y = S (add x y)
> > +add Z y     = y
> > +
> > +-- | Subtract a second natural number from a first.
> > +sub :: Nat -> Nat -> Nat
> > +sub (S x) (S y) = sub x y
> > +sub x Z         = x
> > +sub _ _         = Z
> > +
> > +-- | Multiply two natural numbers.
> > +sub :: Nat -> Nat -> Nat
>
> hmm, shouldn't that be mul?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/byorgey/it-lang/pull/7/files#r2076963>.
>
>
It is slighty, but fits in more with keeping the code more or less on the screen.

As it was you could not see most of it. It can be undone if its better on one long line.
Also please use hard tabs rather then spaces.
`NULL` should be `null` as per the code conventions.
Good spotting!
This entire function could be rewritten as:
state->allocator = MallocAllocator_new(64 * 1024);
you could simply use ``if`` instead of ``elseif`` here as the previous ``if`` leaves the execution of this loop.
This `order` command can be moved into the `:has_many => :closed_motions` association.
This should be a `constants` set
If we're going to do this I'd rather set up logged warnings than crashing the API call.
Should replace the typo in this line also.
how is this specific to a particular flapjack executive instance? It seems global across all executives, no?
VsFake should be all script-driven, or NotImplemented. It doesn't look like this is used? So just make it throw NotImplemented. Or, for :+1: x :100:, add a couple of integration tests. :wink:
Does this mean gevent doesn't support Windows, or your code? If the latter, how hard would it be to handle that?
No need to call `strip()` again here, since it was just done in the line above.
Sorry, i missed it was a trait and validation. Then it's OK.
let's assign `self.bg_processes = []` at the end of this, so that the list doesn't keep growing forever.
Yes. Thanks for saying it. I was being lazy not refactoring into a class.
Doh, somewhere along the way I set that to 940px because I was messing with padding on the container, but I didn't switch it back. Easy fix.
This should ideally be something more sensible default that we actually expect our apache to use. @rhelmer what think you?
### Don't use a two new vectors here. 

Use two vector object that are created at initialisation.  You can do this with 

     THREE.Plane.__v1 = new THREE.Vector3();
     THREE.Plane.__v2 = new THREE.Vector3();


and replace this line with 

    var normal = THREE.Plane.__v1.sub( b, a ).cross(
          THREE.Plane.__v1.sub( c, a ) );

Try to avoid the garbage collection at any cost.
Could calling the id file "generator" cause any confusion as to what is to be deleted? Maybe switch it to "id file" so it is clearer.
That's right I can do it because almost only Bindings will end up here. I can't do any instanceof here because the BindingOperation is private (I don't have the code from where I'm writing).

Sorry, I don't know the code like you do, but I don't see another simple way. Unless we bring changes deeper (rewrite-api ?).
I thought so too.  I put a few other protections in place - ork import 
ignores timestep <=0.  And I added an additional max to the 
RK4SimulationStepper when it initializes d[0] (part of the array to 
determine next step size).

The code is safe without this change.

On 10/6/2012 9:10 AM, plaa wrote:
>
> In core/src/net/sf/openrocket/gui/main/SimulationEditDialog.java:
>
> > @@ -673,7 +673,7 @@ public void actionPerformed(ActionEvent e) {
> >  		label.setToolTipText(tip);
> >  		sub.add(label);
> >  		
> > -		m = new DoubleModel(conditions, "TimeStep", UnitGroup.UNITS_TIME_STEP, 0, 1);
> > +		m = new DoubleModel(conditions, "TimeStep", UnitGroup.UNITS_TIME_STEP, RK4SimulationStepper.MIN_TIME_STEP, 1);
>
> It's a bit ugly to tie the UI to RK4SimulationStepper. Should we just 
> allow zero value, and take a max(value, MIN_TIME_STEP) every place we 
> use it?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/plaa/openrocket/pull/4/files#r1779414>.
>
I'm in your hands. Whatever is best you think.
The return value from readHeader is just ignored here when it's overwritten with the return value from readData.
This is a FatalError if you don't have a logger as the typehint does not accept ``null`` (see, a test would be needed)
please use an event listener class instead of a closure.
why limiting this to ``_context`` ? Your implementation will accept any iterable element
why throwing an expression instead of returning `null` ?
Yes please. Put the throw statement at the bottom in the interface and use
the inheritedoc docblock


2012/11/19 Terje Bråten <notifications@github.com>

> In
> src/Symfony/Component/Security/Core/Authentication/RememberMe/DoctrineTokenProvider.php:
>
> > + *  `class`    varchar(100) NOT NULL,
> > + *  `username` varchar(200) NOT NULL
> > + * );
> > + */
> > +class DoctrineTokenProvider implements TokenProviderInterface
> > +{
> > +  private $em;
> > +  public function __construct(EntityManagerInterface $em)
> > +  {
> > +    $this->em = $em;
> > +  }
> > +
> > +  /**
> > +   * Loads the active token for the given series.
> > +   *
> > +   * @throws TokenNotFoundException if the token is not found
>
> Should the comments in the interface be updated then?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/symfony/symfony/pull/6057/files#r2170325>.
>
>
I can do this. One sec.
missing `var` ?
yeah see when i do that i get this error and not sure why im not mapping admin im merely tryin to make the current user making the page the admin

Caused by: org.h2.jdbc.JdbcSQLException: Value too long for column "ADMIN BINARY(255)": "X'aced00057372000b6d6f64656c732e5573657282deeb20cec5364202001846000874696d657a6f6e655a000876657269666965644c000b616e6e6976657273... (1445)";
I thought Heroku cron was deprecated in favor of Heroku scheduler?

https://devcenter.heroku.com/articles/scheduler
this one too. Then ready to merge!
It is not an owning pointer, but an alias. 

Moreover, the pointer should point to const data (const ImagePointer myImagePointer).
They can provide other formats but I've always seen jpeg available.
I suggest it's fine like this until we discover a need for something different.
I've not wrapped my head around what's going on here well enough to be sure, but you might be able to rewrite a lot of this code by creating a set of custom validators - http://www.perfectline.ee/blog/building-ruby-on-rails-3-custom-validators  and/or looking at the validates_timeliness gem https://rubygems.org/gems/validates_timeliness
Réglé
I'm not sure this is safe. If it's a LazyBs, don't you need to call ensureEval and then getReferenceKey again?
fixed, good point
Can we make this smarter to not recreate `_emptyView` if it's the same?
done done ? ;)
I would say debounce this method call since there may be many objects being added consequently which would only need a single sort.
not sure for this, but you can separate case when no feature filter is present, it would avoid iterating on all features
fixed, now the @ServerSetup is used
StringBuffer uses locking which isn't necessary. StringBuilder should always be used over StringBuffer()

However in this case if you want to copy into memory it's much better to just use an array or array list of strings that you have already read.
As I mention above, this bit has to change, otherwise every other subsystem has to work around this, which is not going to happen.
Should this now not allow for exceptions? The validator and model description both seem to indicate it should
тогда наверное лучше это в контроллер перетащить, не хочется конфигурацию гонять через JSP. Хотя если считаешь что так лучше то давай оставим, напиши
Comma is missing.
This string should also be localized, allowing to change the order and separator for every locale. We should pass month, day and year as variables to the locale.
We can `if (!callScreenWindow)` here and remove it from the 2 handlers.
test is a leftover?
Just done some testing and there seems no disabled attribute for a tag
Concatenating localized strings is a bad idea in general. Please use:
```javascript
name.textContent = _('others', {
  name: contact[0].name,
  n: contact.lentgh - 1
});
```

where `others` is defined like this:
```ini
others        = {[ plural(n) ]}
others[one]   = {{name}} and {{n}} other
others[two]   = {{name}} and {{n}} others
others[few]   = {{name}} and {{n}} others
others[many]  = {{name}} and {{n}} others
others[other] = {{name}} and {{n}} others
```
… and this is what makes the text inputs much easier to use.
@basiclines any ideas?
Confirmed. No issue here.
Please provide localization entries for this page.
I happy to hear that!
needs an updated test
dead code, we are using it later :)
Why this 1sec timeout? Is that waiting an animation?
Wait, I'm stupid. It's in a trans block. Yes, you're right, I just didn't see that. CARRY ON.
Shouldn't this be `:white`?
should be `fakeRequests` -- nit-picky, I know, but I like to be consistent with naming; classes are uppercase and methods/properties are lowercase.
do you prefer this over `i -> j`?
iLines
what if it delete item in source and does not delete item in  cache... maybe we should handle it somehow too ?
`@box` à la place de `@box.id` devrait suffire normalement ?
Shouldn't this be `user` in the first arg (using the 3-arity form)? I'm assuming you want to get around any IO-redirection.
This will explode if an application is not using Auth.  This should be wrapped in an if block.
http://codepad.viper-7.com/W4GWUR

```
<?php
    $packages = array();
	$packages['dontreplaceme'] = true;
	$packages['dontreplaceme'] = false;
		
var_dump($packages); // array(1) { ["dontreplaceme"]=> bool(false) }
	
	$packages = array();
	$packages['dontreplaceme'] = true;
	$packages += array('dontreplaceme' => false);

var_dump($packages); // array(1) { ["dontreplaceme"]=> bool(true) }
It love to; however, since this is really meant to be put into your shell configuration (e.g. $HOME/.bashrc), it would be difficult to predict what "." actually is. I assume you are thinking that "." means, current project directory. If that is the case, I can't think of a sound way to make that happen other than doing odd things like overriding the shell's `cd` command. I certainly wouldn't want to go there.

That being said, there is nothing wrong with someone throwing into their project's `makefile` a line like:

    completion:
      complete -W "$(php ./composer.phar completion)" php composer.phar

This way, they could type the following once per session while in their project directory:

    make completion
This is actually intended, as each section is wrapped in a <section> tag, and therefore has it's own internal document outline, as does not affect the global outline. As per:  http://coding.smashingmagazine.com/2011/08/16/html5-and-the-document-outlining-algorithm/

On a side note to others we are also doing this outline hierarchy on the network front and section fronts, from <h1>'s to <h3>'s
Could you please fix the indenting here?
And you don't need spaces inside `()`, but you do need them around `}{` - so it should be `} else {`.  Sorry to be so picky, but it will really bug me otherwise.
`user` object is required for adding a new participant.
I agree that this should be fixed, I don't have an immediately simple way to do this that doesn't cause browserify to bloat the bid-bundle, and the goal was to keep that minimal. Are you okay with my delaying a fix for this until the next rev?
myfavoritebeer.org tests are failing here.
This isn't really "porting to Windows", but rather, "this segfaults ruby-1.8.7p370", right?  Probably best if the text matches the actual root cause.
convert this (and the others) to 

  if !is_upstart?
    return super
  end
  ....
Rely on the type passed from find / findall /query instead of indexOf
hopefully not tho
@ralphschindler Can you review this change?
Travis tests CS at PSR-2 level with [PHP-CS-Fixer](https://github.com/fabpot/PHP-CS-Fixer). More details on this [section](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#46-method-and-function-calls).

How Travis can pass the last commit? Maybe we have an issue.
Do we want to assume that `ConfigurableInputInterface` should be part of the default?  If so, why not just put it into `InputInterface`?
These need to be commented please
How would the opponent know if the snowball hit or not? :p
no need to specify that  the types of these members are elements.
global leak.
cache length
Maybe this should be a bit simpler? E.g., `<Permission(1, 2)>`
yeah sure.
@anti Osaako tää käyttää yhtio_tyyppi_tuoteno indeksiä vai menekö suuntalava_index indeksiin?
Shouldn't you use the parser here instead of accessing directly the lexer API ?
We could introduce ?g option and then remove it in translate_options.
I'm also implementing following options for 'run' and 'scan':
 - :global 
 - :return
   - :list
   - :binary
Let's use `@opaque` in here please ?
The reason why I chose `or`:

```elixir
iex> true | false
** (UndefinedFunctionError) undefined function: Elixir.IEx."|"/2
    Elixir.IEx."|"(true, false)
    erl_eval.erl:572: :erl_eval.do_apply/6
    src/elixir.erl:95: :elixir.eval_forms/3
    /Users/yrashk/Code/elixir/elixir/lib/elixir/iex.ex:81: Elixir.IEx.do_loop/1
iex> true or false
true
```
Yep, I was freaking out because I thought my list code had slowed things down...when in fact it just made the .inspect call much slower because of the additional references (inspect doesn't seem to be all that intelligent of an operation).

Btw, when we use the built-in ERB templates, we are 4x faster than asciidoc at this point. When I use haml, I'm about the same. If I switch to slim, then it's back down in the ERB range. And we haven't even optimized yet :)
issue a pull request with this at getpelican/pelican-blog
label and data appear to be unused
code style: spaces around '='
Can you call it MessageTypeParser instead?
Is `display_name` guaranteed to have a value? It didn't look like it to me. If it doesn't, and the user is autovouched, there's no way to click through to see their profile.
This css was copied from the persona brand guidelines page.
I don't understand your comment. This is what I'm doing in my mind. list comprehension to get a list of tuples --> throw it into a dict to get a dictionary, return said dictionary.
Hm, interesting. So yes, lets do it. Thanks for the catch!
shouldn't this be 1.9.3
最後のself.phrases.createの挙動のみを考慮してset_phrasesとしてしまいました・・・
retrieve_phrases若しくはretrieve_phrases_on_timelineとします。
it looks like auth_klass isn't being used either, unless I am missing something
One other thing. Be sure to test your implementation on text that isn't valid utf-8. We encounter this surprising often. This is why the original spaceCompress has the try/catch block:

```python
    def spaceCompress(text):
        try:
            text = unicode(text, 'utf-8')
        except TypeError:
            text = unicode(text)
```
In this overall and here, you can greatly simplify the logic if you just hold onto the app object reference. Then, when you get to this point, call uninstall on the app object reference.
Almost good:

    """
    Description

    Keyword argument:
        arg1 -- Help of argument
        arg2
        arg3

    Returns:
        Dict
    """
Please enumerate the valid attribuets instead of using kwargs here. Also document them. This makes the external interface clear to the implementer.
No need for the hyphen in `wide-variety`.
const State&
please don't add non-POD statics. as well as the reasons I already outlined (static initialization order, and it is in a header, so it *could* become a problem), there are other costs: http://suasol.wordpress.com/category/lowlevel/
Doh! Actually, I'd rather use the same logic that I used in Gemfile:

 Dir["/etc/stickshift/plugins.d/*.conf"].map{|x| File.basename(x, ".conf")}.each {|plugin| require plugin}

Are you saying that development.rb should include, for each plug-in foo, a foo-plugin-stub.rb that checks whether foo-plugin is installed and requires it if it is?
What will this return when the login has been unsuccessful? I'ts easier if we can record the failure in the status code.
+1 for renaming `addRole` to `setRole`
We've lost a lot of sanity checking on this test (e.g., verifying the response data to make sure it is the authenticated user). Maybe a request to the Me feed would be a good sanity check on the user session actually existing.
This shouldn't necessarily be changed but it is a remark for later.
The test.js utility will be used to create the keyspace and will run all the install.js files.
This way people don't need to create keyspaced or CF's in their unit tests.
We can probably use `commentsRows` here.
Can we change "metadata information" to basic profile? This would make it consistent with what's used in the UI.
Ah, I see- the diff-view let me think this was defined on `RelationshipInstance`. The use of the `RelationshipInstance` for queryset state is disconcerting (one class should have one purpose), but fine for this pull.
I would not say "It's possible, however, the first few maintenance releases can contain some changes to existing features, based on community feedback" any more. We did that for 6.0 but it created confusion which is why we did the betas and RC's for 6.1 instead. Once 6.2.0 comes out, for example, we should not be making any changes unless it's for something considered to be a bug.
як в документації:
http://symfony.com/doc/current/reference/forms/types/file.html
// extension cannot be guessed
лишилося зайве, забув видалити перед комітом