labofsahil/github-event-dataset-2012 huggingface dataset:
Awesome!

On Apr 24, 2012, at 1:06 PM, Arash Rouhani wrote:

>> 
>> 
>> -- Given the errcode and stderr, return error-value
>> interpretError :: Int -> Text -> KerberosAuthResult
>> interpretError _ errmsg = fromJust . msum $
>>     ["Client not found in Kerberos database while getting" --> NoSuchUser,
>> -     "Preauthentication failed while getting" --> WrongPassword,
>> +     "Pre-authentication failed while getting" --> WrongPassword,
> 
> hmm, on second thoughts @ahodgen , this isn't a typo!
> 
> It's just that we determine error type based on the string output of the program (at least my `kinit` command doesn't give error messages). Can you revert this "correction"? Otherwise I can if you don't want too. :)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Tarrasch/authenticate-kerberos/pull/1/files#r727457
Ok, that makes sense. Let's leave this pull request open for a day or two and see how it fares on Stanford's site then? We can try changing the 'in' keyword if it doesn't resolve the IE issues.
perhaps renaming NORMAL --> 2PC would make more sense?
"The commit or the rollback command can be received before the prepare message" <-- the prepare is always sent sync and pnly after that the commit/rollback is sent. I think this guarantees the ordering between the prepare and commit (or rollback)
Are we using this anywhere?

`currentPageShown.safeToEdit`
This is part of the topnavigation, not the $rootel.
You're right and it's good to not have the duplication of the same body.
It's however (for me) still harder to read than using the regular if statements.
what is broken?
Probably just an oversight, why not add the type for currency?
>  It worked for me and I worked on things way cooler than string formatting

I hope this will be another pull request. :-)

(You can have multiple features in multiple local branches, developing them independently, while one branch is being in the pull request, pulling to your development branch as you clean the pull request.)
No

If we're going to do that we should do it across the board...

Also, I think these are things that are better configured in akami etc...
why this should be static?
I could be wrong (I wrote this code a while ago), but I think the reason I chose to do it this way was mostly to override redirect_url_for (defined in paypal_express_common).  I think sending cmd=_express-checkout caused PayPal to use regular Express checkout instead of digital goods.
Perhaps put these an array or something? This is pretty huge :)
This explicit cast is unnecessary.
We also moved "azure site portal" to "azure portal".
It's equal to `None` it looks like if it's not set.
You have an indentation issue here: one tab and then two space characters.
Put this file in an 'examples' folder and call it 'plot_fft_image_denoise.py' and you'll have a useful web page automatically generated from it.
Tbh, the mutable map still tastes nasty. I wonder if a better solution wouldn't be making this `flatMap` a `map` that returns either a `List[(String, String), List[(String, String)]]` that we then create two maps from, or a case class with String pairs and a List of Strings that does the same. We would then flatten the second component of those to create the overall bundle list, and use the first component (which could be an Option) to create a custom bucket `Map` with all its relevant entries in a map statement. Thoughts?
think zaim cut and pasted
Removing the max() worries me. It was there to make sure we never returned a grade < 0. With your code, what stops a response being graded < 0?
I will change it to image_use_gravatar.
Cambiándolo.
for the request code, the word "return" (or its variations) doesn't really sound right.
This only applies to JBoss AS 7
Explicit private indicator would be nice for the field.
Имена тегов принято писать с маленькой буквы.
Will this never change?
why this file in the root?
looks like a mistake. Is this intentional?
you should remove the ``\`` before the function calls
everything should use the same appender.

roll > 100MB
roll > day
Public variables are really a bad idea i think. Try to avoid them if you can.