labofsahil/github-event-dataset-2012 huggingface dataset:
I think escodegen should ensure this guarantee to keep escodegen as a simple module for using this as a tool.


`esprima.parse(escodegen.generate(esprima.parse(code))) === esprima.parse(code)` guarantee is pretty understandable and developers using escodegen can expect escodegen's behavior. If escodegen does additional works, developers cannot expect the result of escodegen.  I think this unexpectable nature breaks value of escodegen as a infrastructure of JS tools.

We can create pass for optimizing AST(mangling etc.), I think pass and escodegen role should be divided to keep them simple and clean (like a compiler frontend, optimizing pass and backend)

Is it a good answer to your question?
for these long lines, I would have separate strings for each line, and concatenate them. the backslash escapes the newline, but not the indentation, so you end up with a string with huge amounts of whitespace.

It doesn't really matter, because we happen to be dumping it into HTML, but in general, I think it's better to avoid sticking in a ton of random whitespace.
Might want to s/nova::nova-common/nova::ceilometer-common/ for these log messages... otherwise it might be confusing where something is coming from in the log files.
What does this -1 mean?
could you please remove form.getVar? it is unneeded :)
I guess it should return the error so you can display it in a flash message
could replace [\.] with either \. or [.]. The former would probably be more efficient, since the latter would be a single option character class.
Can use `d.depth` directly, `depth` isn't used again.
Could there be a toolbar switch to show relative times instead? The zoomed-area scale could then start from 0, so I'd also see how long the events took all together. We'd miss the UI string too, but I think that's less important.
Added 'Space Arm', 'Defense Wing' & 'Battle Flight' in e2c3a94
Could you add the authors section? When a module is created we need someone responsible of the said module to contact in case of troubles.
ok - changing this.
this is generated by the build. this shouldn't be committed.
spring injection
windows-1256 character set? Dude...
Probably worth moving this calculation into the constructor, so we only do it once.  Likewise for maxK, although I think since maxk_threshold is a compile-time constant, the compiler might already be doing that calculation at compile time for us.  stepK is almost certainly too complicated for the compiler to figure out that this result could be figured out at compile time.
Why did you change this and the following change? This was fix to base rendering.
Not quite sure why this method needed rewritten at all... but this line specifically introduced a bad change in behavior... so before, you could say pad(123, 2) and get back "123"... now, with this check / edit, you'll call pad(123, 2) and get back "12"
I did verify that quotes around a part of the name works.  that's how 
rfc822 specifies quoting special characters, actually. with double 
quotes around the characters themselves.

On 9/25/12 10:29 PM, Sef Kloninger wrote:
>
> In main/courses/email_members/tasks.py:
>
> > @@ -170,7 +170,8 @@ def course_email_with_celery(hash_for_msg, to_list,  throttle=False, course_titl
> >      staff_email = 'noreply@class.stanford.edu'
> >      if course_handle:
> >          staff_email = re.sub(r'\--', r'-',course_handle) + '-staff@class.stanford.edu'
> > -    from_addr = course_title + " Staff <%s>" % staff_email
> > +    course_title_no_quotes = re.sub(r'"', '', course_title) # strip out all quotes
> > +    from_addr = '"%s" Course Staff <%s>' % (course_title_no_quotes, staff_email) #make certain that we quote the name part of the email address
>
> Surprised you didn't put the quotes around the whole email title, like 
> "Networking Course Staff" foo@bar <mailto:foo@bar>.
>
> The way you did it, "Networking" Course Staff seems strage, like the 
> name of the course is sarcastic or something. And did you test quotes 
> around only part of the name? I've only seen around the whole name.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/Stanford-Online/class2go/pull/830/files#r1692209>.
>
This class is going to go away.
I personally prefer `Closeables.closeQuietly(connection.getInputStream())`
I worry we're being too clever with this MapConfigKey stuff. It does solve a problem, but I wonder if it will confuse a bunch of users.
Recommend using AtomicReference<Task>, rather than Task[].
The run() may not see the initialized value of t[] as it hasn't gone through any synchronized/volatile.
Good question.

It is strange the the running directory is behaving so unexpected btw. I would expect that it is the maven module it is running in and not the root directory of the project. Perhaps it is related to way we have configured maven to run its test.

I'm going to try something out.
Perhaps use guava with Charsets.UTF_8, rather than relying on platform's default?
Ah, yes, of course. Hm.... maybe that needs to be conditional on finding superfeedr creds in the settings? That way if you've not got any it just silently doesn't do anything.
こういうインデント意味あるんだっけ
And really, if this isn't a good test, that's fine. I was also trying to test the clearing of redis did something. If you ramp this up from 10%, then more than this test would fail if you didn't clear redis. So whatever you think is best.
Okiedokie. I'll take a look.
None of this is going to work anywhere other then your localhost.
Please use the configuration variables from tests/mongo-test-cfg.inc
So these used to be a@b.com and then I changed them all to test.com for that purpose because my brain replaced example.com with test.com.

I need to get off these bath salts.
Will not produce the (somewhat unspecific) log-entry, should be at least as easy to scan/ edit and is compliant.
small nitpicks but I'd prefer `function setRawMode(mode) {` here and `if (process.stdin.setRawMode) {`
Probably better to do the `request.user.get_profile().timezone` in here and not use a cookie at all?
Please add dateAdded and caption to this entity.
Is this an actual TODO or something you just copied & pasted?
They should be in theory yes, otherwise we'll have massive duplication for each O/S.....
I don't even think there is any imagery outside this.
This range can go away, now that it's wide open.
这个block基本没看懂啥意思， 你merge吧：）
Yeah, I'm used to seeing xxxx xxxx xxxx 1234
Это что произошло?
Ok, I'll fix it!
Сделать, конечно. Copy-paste - это страшное зло. К тому же ещё и хуже читаемое.
Spelling error - ConditionHeader, instead of ContitionHeader

The error's really in the method name.
Why you've changed to use NotSuccess instead of Error?
You don't have to, but what about using a `<tabbox>`?

See https://developer.mozilla.org/en-US/docs/XUL_Tutorial/Tabboxes and https://developer.mozilla.org/en-US/docs/XUL/tabbox
Would it better to do an element_is_not_present check rather than a type with an expected exception? That seems to be the standard for this type of thing. I believe that the type + expected exception comes from an older version of the page that did have an input, but the input was disabled.
including の訳がないので、"以下を含む@DomainClassUnitTestMixing@を使用することで" としては。
that also takes a ``sparse_format`` argument. As I guess your implementation doesn't support sparse matrices, You can set it do ``dense`` so that a sensible error is raised when given a sparse matrix.
A Python property. Have a look at `coef_` in the SVC class for example.
1.

it should be cross-validated anyway
@ogrisel : that's the line doing it
I would set it to `True` by default. I don't think the overhead is that important and it can save hours of tracking down random bugs by making it as deterministic as possible by default.
--gpl plutôt qu'une double négation ?, à propager un peu partout...
I copied that line from somewhere else in the code if I recall.
Do you actually want to pull this change? Should I fix it up?
Composer does not use the UniversalClassLoader but handles these conflicts as well.
With this moved, please remove the ConversionHelper files.
Also, unregisterForPushNotifications looks like it's for iOS only, can you please check with Appcelerator?
Please don't duplicate privacy labels in class definitions
oui cette méthode getAuteur
Ajouter @GeneratedValue
For python version >= 2.5 and < 2.7, without this line will `slimit compressor` module, other than `slimit` library.

http://docs.python.org/whatsnew/2.5.html#pep-328-absolute-and-relative-imports
Is this bug logged and/or fixed in gerrit?
Commit 8097b80 pulls the 'yourkit' class out, but app/server.pp still needs to take in conditional values so that it may generate the YourKit start java options in the sakaioae startup script.

Now server.pp only has a soft dependency on yourkit.
Ag can search binary files. This will break that.
Should be eclserver, but see note above - should really be cluster/queue.
Do you also support grouped hash dedup?
Should check (and output) any error text from pipe->readError
the extra if (toremove.ordinality()) { seems a bit pointless here.
Like I said: ugly solution, but I use this to print out relatively nicely
Why do you need this?!
This blows, but I guess that's just the way it is. We'll have to see if there's any easy way to run the tests against multiple versions of Qt. We could also consider dropping support for <4.8 after our 1.0 release.
global :D I've got it though
nit: would be easier to read if you move `options = options || {}` on a seperate line, before this.
We probably don't need the license in the yaml file.
I'm sure you can do that too!  Consider it as an exercise in modifying your pull request before I accept it! :-)
What is this? I don't need it.
as assert on the `BlobEndpointUri` that it equals to `https://myaccount.blob.core.windows.net` will be useful because the connection string should support that users can set their own blob endpoint uri like this:

    "BlobEndpoint="http://fooaccount.mycore.windows-int.com";DefaultEndpointsProtocol=https;AccountName=myaccount;AccountKey=mykey"

Though, the Table and Queue endpoint will be the default one.
Perhaps this should be called `URLColumn` to match the model field name?
@yuhunglin 
is_all_day

```
    {
      "id": "h1ifrb5qm7ubd6ue713o45m9t4",
      "status": "confirmed",
      "html_link": "https://www.google.com/calendar/event?eid=aDFpZnJiNXFtN3ViZDZ1ZTcxM280NW05dDQgc2hhY2tlckBiZXJrZWxleS5lZHU",
      "summary": "Scot out (Stone Soup)",
      "is_all_day": true,
      "start": {
        "date": "2012-11-16"
      },
      "end": {
        "date": "2012-11-17"
      }
    },
```
@shacker appending to the script, i should probably just regen
Sekvenssejä voi vertailla suoraan:

~~~clojure
(= [5] s)
~~~
ops, looks the former java programmer inside me is still alive. :wink:
Is this a bug or just confusing code? The method is named ``make_cache_``**key** but you're storing what it returns in **value**
is that update any better?
Remove the debugging error messages into the text file.  You can use `$this->error("message")` or `$this->info()` or `$this->warning()` or `$this->debug()` to send messages to the transcoder.  Those messages can then be acted upon by event listeners registered elsewhere (such as for logging, or outputting the text into the command line.
@mrmattf Now that we can add images to the texture atlas on the fly, can any of this label collection logic be cleaned up even more?
The problem is we're not managing the queue, the brower is, and it gives us no way of knowing the current state of a web worker's queue.  If you want everything you schedule to be queued up with no questions asked, just set the maximum number of tasks to infinity.  But for the use-case in this pull request, returning undefined is exactly what we want.
We usually call these `Mock`.  `Fake` is OK, but less consistent with other tests.
That makes sense.  Merging.
Yes but they also dump a stacktrace, which looks ugly. Besides, I wanted to customize the error handling: if wrong command line arguments are passed, it calls showhelp; but if an error occurs in the main demangling code (e.g. input file not found), it prints an error and exits. Showing a stacktrace makes it look like the program crashed, when actually it's just a fatal error and normal abort.
you should never focus or blur an element yourself. Only Ti.UI.TextField/TextArea is allowed to do that.
KEYBOARD_APPEARANCE_ALERT,  KEYBOARD_NUMBERS_PUNCTUATION, and RETURNKEY_EMERGENCY_CALL are not supported on mobile web. Should either mark the example as platform specific, or choose a different type.
Oops... Heh. Leftover fragment from debugging. I'll fix that right up.
ah ok, must have missed that.
Да, есть такой баг у хрома и сафари http://www.quirksmode.org/dom/events/blurfocus.html
That's good, although same comment as https://github.com/jackalope/jackalope-jackrabbit/pull/2/files#L0R20 applies, but it should also be required by something, I haven't seen a package requiring it at the moment. Probably phpcr/phpcr should require it?
Can we move this file in the correct folder (desktop) Don't know how it get here(Thanks)
IMHO, proper indentation > 80-column rule
As I mentioned above if there is `hasElaborateCopyConstructor!T` than it must be called on `result`, not temporary.
I think we should have a separate unit test method for each of the given/when/then blocks. Not only does it make the intension of the test clearer, it also prevents side-effects. The failing test might just be such a side-effect.
Do you have an answer to this question?
なるほどー。。。コードであらわしたいところですね。`@log`を`StringIO`的なもので上書きして、エラー出力に吐いたのを目視確認するだけでなく、文字列キャプチャしてテストしたいですねー。
Needs to be same as repo location (which will eventually be phpbb-blog with github.com/phpbb-blog/phpbb-blog-for-3.1 as a mirror (Pushing the repo to both locations, locking one).
Not ideal, but I'm fine with it. @hafriedlander Any objections?
Can you use `protect_identifiers()` in here? Also - it would be better to replace `_protect_identifiers()` with it, so we can make the latter protected at some point.
The reason I made this a separate file is so when you import `main_box`, you get the constants as well.
Weirdly, this did not install diff or debug for me -- I had to do a manual installation. However, this looks like an npm issue.
this is to address one of the previous checkstyle comment. If we put an extra 4 spaces there, checkstyle will complain :(
Me: `http://www.microsoft.com` no more exist and it's replacement is `http://www.microsoft.com/en-us/default.aspx` so I've replaced it with one that exists
Someone: this doesn't look like good software engineering practice as the good practice is to make your test framework doesn't depend on random data.
Me: OK, so what do you suggest now?
Fixed in this and the Blob and Table versions too.
I suppose sorting will be better regardless
remove frequent output
I'm not really comfortable with adding this here. 

If this is valuable, then we need to think about adding a dedicated builder object for the serializer.
Почему бы не сделать

    EntryStatus newStatus = state;
    switch (state.State_)
    {
    case SDND:
    case SXA:
        newStatus.State_ = SAway;
        break;
    case SChat:
        newStatus.State_ = SOnline;
        break;
    default:
        break;
    }
what are you appending to the result here? Results only happen with the import method, not when you're just adding data, right?
https://admin.fedoraproject.org/pkgdb/acls/name/qxmpp-dev?_csrf_token=974530dd90c0386bf1e47f8c2d0bd3b543bbf480
Well, that's how it was until but. . .
Gary and I talked about it and he made a valid point.
Let's say you have an SLA of getting a reply in 5 seconds or you no longer interested. (e.g.. why would you care about finding out a stock quite from 2 hours ago as a current request when you already have an updated one), but the consumer was out temporarily and the request queue was populated with thousands of messages. On top of that the consumer is rather slow and not very concurrent so it takes it 2 sec to process each message. This means you have 1000 messages that needs processing before yours which would be 2000 seconds. . . that's way past your SLA. Obviously to really fix the problem the request queue needs to be drained, but that is another discussion.
I DO believe that we MUST honor the receive-timeout otherwise its useless.
What happens is, whenever your identity_column is username or email, a doubled column with same name.
So whenever the query builder tries to SELECT * FROM (SELECT email, username, email, ...) fails under Oracle because it doesn't figure how to handle 2 columns with same name without an alias.
This code is therefore missing a check on identity_column to determine if column email, or username or any chosen identity_column has to be set in the column list of the SELECT statement.
Temporarily,  I aliased it as identity_column so it does not crash anymore.

( Also, I will no longer work with ION for the actual project, so I might come back later and continue Oracle integration as for now, we'll implement our simple required solution by hand. )
Use CSS for space.
This could be split into multiple-lines if.
as an FYI, the inifile based native types for openstack have gotten great reception this week!
Missing parameter doc.
maybe put it into a method of Client, to make things more readable
this one is not needed at all as it is already handled by Composer: https://github.com/symfony/symfony/blob/master/composer.json#L59
It's a good thing to add this as proposal and gather some feedback.

Personally, I'm not a huge fan of these contracts, but I don't have a good alternative to offer either :) Probably I'd leave this out in 1.1 and wait for BV providers experiment with this sort of feature.
This will break the test. It should be "file:///" + blackberry.io.sharedFolder + "/downloads/windowcover.png", like was done for the ics card test.
That name is a bit unclear to me: what does 'sss' stand for. Schwart-like schwarty schwartiness?
500 errors from the exception propagating up. Presumably because the call stack at this point hasn't been wrapped in api_call_handle_error. I'll check if it works that way around.
Thinking about this, we might want to keep the size option because what you haven't done, but need to do, is to fix `app/views/user/api_read.builder` to generate a gravatar URL when appropriate, and that should probably be at a larger size.

For our own images we return them at original size, but I don't think there's any way to do that with gravatar...
rename to votefav
Should this be tmp-dir?
Yes, good idea.  I've changed this just to be !pair.name since I convinced the upstream addressparser lib to just provide falsey empty strings instead of omitting or deleting properties.  If we have any internal logic like this, we should be doing similar things.  (But in general, we'll just be using that function because I'm going to expose it through MailAPI too.)
хочется insert сразу многих без промежуточных версий
it can but still you need your own person object which is linked then
with a central one

On Tue, Jul 24, 2012 at 9:02 AM, Peter Grman
<reply@reply.github.com>
wrote:
>> + *     http://www.apache.org/licenses/LICENSE-2.0
>> + *
>> + * Unless required by applicable law or agreed to in writing, software
>> + * distributed under the License is distributed on an "AS IS" BASIS,
>> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
>> + * See the License for the specific language governing permissions and
>> + * limitations under the License.
>> + */
>> +
>> +package org.openengsb.domain.requirement.events;
>> +
>> +import org.openengsb.core.api.Event;
>> +import org.openengsb.domain.requirement.Person;
>> +import org.openengsb.domain.requirement.Requirement;
>> +
>> +public class PersonCreatedEvent extends Event{
>
> what do you mean if i plan to throw them myself?
>
> the idea was, not to have the person object in every requirement and so tons of redundant person objects but only a reference - can it be like this in OpenEngSB??
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openengsb/openengsb-domain-requirement/pull/1/files#r1222619
Guess this would be
```scala	
def pad2(width: Int) = { 1 to width - sb.length foreach (_ =&gt; sb.append('*')); sb }
```
Is it okay that this switched?
Thanks for the walk-through!
Yes. LANGUAGE_CODE is needed. Sorry for missing that.

--
Henrique Bastos <http://henriquebastos.net>

Curso Welcome to the Django <http://welcometothedjango.com.br>
Aprenda Python e Django na Prática!

Twitter: @henriquebastos <http://twitter.com/henriquebastos>
Skype: henriquebastos.net
+55 21 9618-6180



On Fri, Dec 21, 2012 at 5:11 PM, tonylampada <notifications@github.com>wrote:

> In djangoproject/frespo/env_settings.py_template:
>
> > -FACEBOOK_APP_ID              = 'XYXYXYXYXYXYXYXY'
> > -FACEBOOK_API_SECRET          = 'ZWZWZWZWZWWZWZWZWZWZWZWZWWZZWZWZWZWZWZW'
> > -GITHUB_APP_ID                = 'XYXYXYXYXYXYXYXY'
> > -GITHUB_API_SECRET            = 'ZWZWZWZWZWWZWZWZWZWZWZWZWWZZWZWZWZWZWZW'
> > -BITBUCKET_CONSUMER_KEY       = 'XYXYXYXYXYXYXYXY'
> > -BITBUCKET_CONSUMER_SECRET    = 'ZWZWZWZWZWWZWZWZWZWZWZWZWWZZWZWZWZWZWZW'
> > +LANGUAGE_CODE = 'en-us'
> > +
> > +TWITTER_CONSUMER_KEY = 'XYXYXYXYXYXYXYXY'
> > +TWITTER_CONSUMER_SECRET = 'ZWZWZWZWZWWZWZWZWZWZWZWZWWZZWZWZWZWZWZW'
> > +FACEBOOK_APP_ID = 'XYXYXYXYXYXYXYXY'
> > +FACEBOOK_API_SECRET = 'ZWZWZWZWZWWZWZWZWZWZWZWZWWZZWZWZWZWZWZW'
> > +GITHUB_APP_ID = 'XYXYXYXYXYXYXYXY'
> > +GITHUB_API_SECRET = 'ZWZWZWZWZWWZWZWZWZWZWZWZWWZZWZWZWZWZWZW'
> > +BITBUCKET_CONSUMER_KEY = 'XYXYXYXYXYXYXYXY'
> > +BITBUCKET_CONSUMER_SECRET = 'ZWZWZWZWZWWZWZWZWZWZWZWZWWZZWZWZWZWZWZW'
> >
>
> But there's the LANGUAGE_CODE = 'en-us' change.
> This is needed, right?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/freedomsponsors/www.freedomsponsors.org/pull/101/files#r2491794>.
>
>
Any reason why the ``AMQPBackendDispatcher`` is not in the constructor ?
This is not proper way how to deal with it. You cannot send user to `#`. This line should look like this:

```
<a<!-- IF sn_user_menu.URL neq '' --> href="{sn_user_menu.URL}"<!-- ENDIF --><!-- IF sn_user_menu.EXTERNAL --> onclick="window.open(this.href,'_blank');return false;"<!-- ENDIF -->>
```
Still trying to solve this one. Was finished then the browser stopped supporting parameters in local url. Bryan is still working on it.
I think we should lock the concept dictionary at this before calling getConceptsToExport to avoid the scenario i mentioned on line 87, what do you think?
I just had a look at this and I'm thoroughly confused. I didn't alter any files in the infusion directory. And when I went to look that alteration isn't there.