labofsahil/github-event-dataset-2012 huggingface dataset:
i believe relativedelta is more fully featured / covers different timespans
than timedelta. although in this case, looks like timedelta would have been
fine.

On Thu, Sep 27, 2012 at 5:40 PM, Cory Zue <notifications@github.com> wrote:

> In logistics/models.py:
>
> >                  return True
> >          return False
> > -
> > +
> > +    def is_stale(self):
> > +        deadline = datetime.utcnow() + relativedelta(days=-settings.LOGISTICS_DAYS_UNTIL_DATA_UNAVAILABLE)
>
> personal curiosity - what's the difference between timedelta and
> relativedelta?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/rapidsms-logistics/pull/3/files#r1712523>.
>
>
@shacker negative, Racc::ParseError parse error on value "(" (LPAREN)... (more mumbo jumbo here)
Just to be on a safe side - do you mean to use invokers ?
Did this sneak in or is it intentional?
Glad to have this conversation rolling! Björn is a dear friend, a meditator
and programmer living in germany. Travis is currently my closest
enlightened structure collaborator, and has had a lot of a chance to think
about unbounded graph structures while working for archive.org.

This is a bit about the bottom of the rabbit hole where travis' work and
mine meet:

http://enlightenedstructure.org/BaseParadigm/

Warmest regards,
Harlan
On Jul 21, 2012 2:34 AM, "Björn Grossmann" <
reply@reply.github.com>
wrote:

> > @@ -32,20 +33,16 @@ We can have this now.
> >
> >  One and two I'm working on, in the context of a project called
> [Software Zero](http://enlightenedstructure.org/Software_Zero/) --
> currently pre-alpha and fit only for ruby coders.  [Contact me](
> https://github.com/harlantwood) if you want to play.  Three and four are
> dead easy.
> >
> > -## What's in it for GitHub?
> > -
> > -I'll tell you why I would love this idea if I were them.
> > -
> > -GitHub makes their money from private collaborations, while public
> projects are hosted for free.
> > -If the forking of content takes off like the forking of code has, there
> will be many use cases for private as well as public collaborations around
> written works -- potentially a much larger market than that which exists
> for code.
>
> yes, it totally makes sense to me, to use github to store comments and
> changes.
> I was impressed the other week when I listened to a talk given by Dan
> Yoder, http://fluentconf.com/fluent2012/public/schedule/detail/25798
> He suggested storing all client data in gists on github or on similar
> services and doing all the heavy lifting on the client side, so that
> projects could be done without expensive server infrastructure. He said,
> his clients loved the fact that data was stored on github and even
> considered this a plus.
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/harlantwood/Harlan_T_Wood/pull/1/files#r1210659
>
If you think this should be removed, please delete rather than comment out.
you should ignore these files locally instead of adding them to the project gitignore file, as they are not related to the project but to your local setup. See https://help.github.com/articles/ignoring-files
Обычно происходит наоборот — выставление модификатора пораждает событие :) Как минимум это позволит более гибко управлять событийным API.
I think that this should also be user-settable.
I'd rather use ``:class:`DictVectorizer` `` so it is a clickable link.
```
if (Type *t = targ->trySemantic(loc, sc))
    targ = t;
else
    goto Lno;    // errors, so condition is false
```

Shouldn't I be able to just do:

```d
if (!(targ = targ->trySemantic(loc, sc)))
        goto Lno;                       // errors, so condition is false
```
is container_ garanteed to not be NULL here?
title should be better affirmative 'USE HTTPS', explaining why is in the paragraph
sh-deps commiters
That's odd... I'd count things like quotes, brackets and others as "word" separators, but I guess we're not talking about the same "word" here. ;)
From the class name, I didn't expect that `Fiddler` would do recording instead I'd expect that from `MockServerFilter`
Please do! The fewer places we need to specify things like this the better.

You could even do it as a macro if you want.
as an FYI, this will not be compatible with the folsom refactor. you should pay attention to code that is about to be pushed out to the folsom branches.
Je vais changer "élément" en "balise" afin de garder une certaine consistence.
Trying to understand why this is being removed -- is it because it's not used?
$data is the array. We need `$model = new ViewModel(array('books'=>$data));` I think.
Don't cast. put the methods in the interface.
@shacker which `parseURL` API function are you talking about, do you mean the `getURLParameter` method?
I haven't tested this yet, but I think that "i" that got stuck on the end of password is a typo.  Can you fix that, test it and let me know if it works?  Otherwise I don't have a problem with the changes you've made.
I think you meant the greenlet module :-)
setPhoneNumberType
hah, troll. OK, I will fix.
Note here that there's no `*@args` like there is for svn. Anything other than `--git-branch` and `--git-remote` is ignored.
I don't think session (client) can be null :S, can it?
The description does not seem to be as detailed as the other recipe descriptions.
Yes, find_objects only takes integer images. I changed that.
Can we add a describe block here?

```ruby
describe "#description" do
  it 'states that it only allows numeric values' do
  ...
  end
end
```
CS issue here. The curly brace of the method should be on its own line (and indented one level less)
It's a typo. They're supposed to be `EPSILON14` in `PerspectiveFrustum`, but for this test they are exactly the same.
Perhaps we need a helper function that goes from (x, y, z) to (0, x, y) so we don't hard-code this everywhere.
Since the WireBox config contains all default values, how do you feel about omitting it for both sample apps?
Sure. This should also come in handy for Win64. I'll close this pull for now until I figure out a nice interface which can be used with multiple linkers.
这里不能这么干……因为OrigSpeed是没武器重量影响时的速度
依然已经把keysetting和mousesetting都归为controlsetting了，那么KeySettings类和MouseSettings类就可以删掉了。
One too many tabs.
For all of the places where you write ${foo}, you can write $foo instead.
Removed this part, as noted in the incrby part as well
Note: This spec is wrong, yes? It should be "south". Or am I missing something? This error does not persist in the new code.
Total guess. No categories like the Tulsa site so I thought the scores would look more interesting in steps of 3. 

On Oct 22, 2012, at 8:32 AM, luke crouch <notifications@github.com> wrote:

> In restaurant_inspections/sanantonio.py:
> 
> > +import re
> > +
> > +from bs4 import BeautifulSoup
> > +import requests
> > +
> > +from storage.csvfiles import get_csv_writer, get_value
> > +
> > +logging.basicConfig(filename="scraping_errors.log")
> > +logger = logging.getLogger(__name__)
> > +
> > +URL_ROOT = 'http://samhd.tx.gegov.com/San Antonio/'
> > +PAGE_SIZE = 200
> > +SECONDS_THROTTLE = 2
> > +
> > +
> > +DEMERIT_MULTIPLIER=3
> interesting. how did you decide on 3 for the multiplier?
> 
> —
> Reply to this email directly or view it on GitHub.
>
Про запас. Реально только для first.
общепринятое название size
as a general pattern i would rather make the final fields public to avoid adding extra getters + setters to the code. it makes everything a bit harder to read and doesn't really buy us anything, esp. for trivial map access.
No, I meant only instead of line 10
Yep, that's exactly what it means, and it matches the current behavior.
Note that you can use `xformat` at compile-time/in templates which should be faster than appending strings like that (I think xformat may already be used internally by `format` from a recent pull).
Please do not put braces on the same line as other stuff like this. The lines below are okay, because the whole thing is on one line, but other than cases like that, braces need to go on their own line.
Cheers, merged at elbertf@0e215c6c45f1abdc29141736a4aa0040a362a0ee
This breaks a spec?

```
1) Measure validations 
     Failure/Error: it { should validate_validity_dates }
       expected Measure to validate validity dates
     # ./spec/models/measure_spec.rb:391:in `block (3 levels) in <top (required)>'
```
why ship the src dir to npm?
In Symfony 2.10, the inverse side is'nt updated, i don't have test with master. So if this solution isn't obligatory now, fine.

For the deletion i don't understand why don't make that before the persist? It cannot be used elsewhere in my mind, but maybe i don't see something. Could you to light me about that, this would be really kind.
82b623f4f9d61565826ac0b9a41e5a53aa9152de
@Baachi is right, this one should be reverted, change you are pointing is corretly change few lines below, but this one is wrong.
Do these generated files need licenses too?
It's be nice if the error codes are collected into a constants file and the used from there. This will greatly improve the code readability so I don't have to go to azure web site and check what this error means
Looks like the last section is gonna have an unnecessary divider?
@kriswallsmith the rationale is that you don't want to check something that is not in the cache `$key` and `$key.expires` are distinct entries.
Please pluralize the helper module name: `ResourcesHelper`
I wasn't sure.  but I think you're right.  moved back.
Should this also be a `{String}`?
Многовато, лущ 40-50.
But, the rule is that this "quick" fix was to correct the reported problem in the JIRA (stop the exception) __without__ changing any existing behavior.
Also, the timeout should be decremented by the time used to receive stale messages; with recursion, the clock starts from 0 for each receive()
Not sure about "Mozwebqa's py.test plugin". Also, we should probably mention that py.test itself is required. How about "py.test (plus a few plugins)"
This belongs in the `before` block, not in `specify`.
Убрать «наш».

зависимость от jQuery → зависимость от пакета с jQuery
Jackson claims to be thread-safe, as long as you don't try and re-configure the mapper from multiple threads. What one is supposed to do is: create a mapper once, configure it properly, then (de)serialize as much as you want from any thread at any time (but NO changes allowed to the mapper config). That's what this approach tries to do. 

As I said elsewhere, if we were to use a non-sorting mapper, we should create a separate non-sorting instance (and not play with how one instance is configured...).

See http://wiki.fasterxml.com/JacksonFAQThreadSafety and also http://wiki.fasterxml.com/JacksonFAQ#Data_Binding.2C_general (search for "thread-safe", no direct URL to that particular question unfortunately)
Move all of these validation regex's to presenter (and test)
name=%page.checkstyle.name  (to reuse externalized string in plugin.properties)
Seems like most of the stuff in this file is duplicated in puppet_spec_helper?
this will delete all resources which are ignored as described by .gitignore, but not anything else. This should avoid deleting people's untracked .tex files if they forget to add them before running clean.
For code consistency, I would suggest using a similar approach to what's in WritableField.field_from_native

e.g.

replace
        value = data[field_name]

with
        try:
            value = data[field_name]
        except KeyError:
            if self.required:
                raise ValidationError(self.error_messages['required'])
            return

Or, if you want to avoid the try/except block, consider at least factoring out the common condition in the if, like this:

      if field_name not in data:
            if self.required:
                  raise ValidationError(self.error_messages['required'])
            else:
                  return

If you choose the latter approach, you could also update the code in WritableField.field_from_native.
i mean the calculation should either change to subtract more for the longer path (`17 = sqrt(2) * 12`), thus if you want the straight reduction to be `3`, the diagonal modifier to subtract would be `sqrt(2) * 3 = 4`.

Because manually calculating this is tiresome, i suggested a multiplication instead which would keep the scale between straight and diagonal intact.
Summaries here seem slightly extraneous, but this one should be corrected.
There are two goals:

1.  Give the developer the opportunity to exercise some code-reuse / extensibility in their template design (e.g., maybe this template consists of only a block override, a `<ul>`, and pagination links)

2.  Check for a basic understanding of Django's aggregation features
Shouldn't this be in `config/environments/production.rb` with all the others?
mysql_escape_string() deprecated, replaced with closest non-deprecated method.
This is a little hacky but it's needed in order for the magnifying glasses to show up/work properly both as inlines or on the individual model admin pages.
They weren't, but were included in the JBoss repo. That's why I added this - so that the project can be built directly after checkout.

I couldn't build the project because the org.jboss.ejb3:jboss-ejb3-api:3.1.0 is missing in the central.
use a code-block for the URL
Вообще можно, но тогда будут сохраняться оригинальные имена.

Вообще, можешь убрать
<pre>/**
 * @param {String} name
 * @param {String|Blob} val
 * @param {Boolean} isFile
 * @param {?String} fileName
 * @param {?String} fileType
 * @return {undefined}
 */</pre>

И заменить `fd.append(arr[i].name, arr[i].val, false);` на `fd.append(arr[i].name, arr[i].val, false, null, null);`. А вот `@constructor` надо оставить, ибо без него компилятор будет ругаться.
Что удаляет этот `xPath`? Нужен ли он вообще?
This test should read-
  it("readonly fields set", function () {                
            expect(mockedWebworks.defineReadOnlyField).toHaveBeenCalledWith(client, "hardwareId", null);
            expect(mockedWebworks.defineReadOnlyField).toHaveBeenCalledWith(client, "softwareVersion", null);
        }); 

This fixes-
1. somevalue is undefined, should be null
2. hardwareid should be hardwareId
3. version should be softwareVersion
I have no compelling reason...
No longer need weblauncher, but keeping the file since native packager complains if there is no entry point file
please group all the variables at the begining. Makes it better readable
Sigh. I don't know. I'll merge this, but we should really detab all these files.