labofsahil/github-event-dataset-2012 huggingface dataset:
Yes I know I think so too that's why the teacher writes all these kinds of funny stuff like Prog = Decl+ or Stmt* because it only matters for the parsing but not for the internal representation (like some kind of generic domain in Advanced Programming). In order to define FunDecl we need to define Stmt in the first place. In Advanced Programming we did some kind of semantics for if statements,while statements... (I've attached the file). By the way that example code was copy-pasted from here I think this might be useful later on (so another good reason for using haskell instead of clean is available documentation on the Internet): http://alephnullplex.appspot.com/blog/view/2010/01/12/lbach-1-introduction


________________________________
 From: Ben Ruijl <reply@reply.github.com>
To: sebitzutzu <sebitzutzu@yahoo.com> 
Sent: Wednesday, February 1, 2012 10:42 AM
Subject: Re: [spl] Added a rule for FArgs. (a554204)
 
I think we should be very careful to separate the grammar of the parser and our internal data structure. What this file is, is basically our internal representation.

--- 
Reply to this email directly or view it on GitHub:
https://github.com/benruijl/spl/commit/a55420436c069b91e4d1e1a1615d6223772fe3cc#commitcomment-921930
You need to invoke the anonymous function here.  That fixes the failing frame/script tests.  I'm getting the usual 4 failed,  plus jsdom/index/issue_338_internal_nodelist_props (ERROR 1 vs 3).
It does, however, seem to check for non-existence. So using an order-only dependency won't actually work.
This appears overindented
Основан на патче отсюда:
http://forum.psi-plus.com/viewtopic.php?f=8&t=94
Взят только базовый функционал. Для проверки используются все доступные в системе языки.
Really no, not without a client change.
Great code! So we don't need to call the __init__.py
I may even have a contribution in it ;) At least someone who was working on  C++0x  talked about using my ziggurat code for generating random normals, but I don't know if anything ever came of it.

I think the NULL data pointer possibility needs to be carefully thought out and coordinated with the Cython folks, and maybe we should take a look at the numpy.i swig files also (scipy contains swig generated wrappers). The potential impact of this change could be enormous.
There are instructions in the Makefile to "uncomment" stuff for OS X.
According to Those Who Know™, Win32 should never be `posix?`.

I'm a little less certain about what's not safe about it.
Dom do you get these messages?
What if mmll['min_lat'] is 0 ? This expression wil always be False and the resulting min_lat, max_lat, ... will never be set correctly?
I'm getting a tab or a space or some other garbage at the end of the code.  I think there is a bug.  I'm trying to track this down now.
We should keep PR_DUMPABLE set as soon as possible, especially before any plugins are loaded. This will allow someone debugging to get a coredump.
In which case? VE or non-VE? I'm not exactly well versed in the ways of setuptools, so there's quite possibly a better way.
oops, should be 1
окей, я тоже раньше думал короче что ник уникален, Димсон пендаля выписал сказал что id уникал и все!
You committed hitran-things together with nodesoftware-things. I'll pull it, but please use separate commits in the future :-)
@Th0ar 其实用了bootstrap的话，ie老版本基本上就无缘了.
I *did* try to use `super()` -- but these are oldstyle classes.
it's somewhat counterintuitive that it works this way…
Thanks dimiandre.
Should this be :tag => '2.2.2' instead?
@kabum  kannst du bitte mal genau erklären, was du jetzt in der COllection alles geändert hast?
Ich net :D
@bramswenson if you're changing this, could you move the setting of defaults into the 'setup method? See  https://github.com/pry/pry/wiki/Custom-commands#wiki-Complete_example
I think that this is not a good idea to make it dependant on luck.
In my opinion this solution looks better:
<% yield_name = "table_headers_#{model}" %>
<% @view_flow.set yield_name, "" %>
...

Bet even better would be to move the header in a separate partial and then just call it at the beginning and at the end of the table.
Also it would improve the ability to customize Typus, as it becomes possible to change header without overriding the whole _table partial.
this fixes #2 by the way :)
Have u tryied deleted the repo u have and recloning then rebulding dbcs? ?
When i Enter combat with npc 18326 i have this error again :
Check existing of map file '../../data/maps/5563231.map': not exist!
Spell::DoSummonGuardian: impossible place for create creature entry 18701, spell 32663.

but before this line i have this line :
Loading map ../../data/maps/5563231.map
VMAP loaded name:Auchindoun: Sethekk Halls, id:556, x:32, y:31 (vmap rep.: x:32, y:31)

now can any one tell me why this error ?
no.
кстати, World PvP это несколько больше чем Outdoor PvP вообще-то. Ну что, кто готов портировать ОЛО с трини? уже можно.
упс. я не добавил пока речек смены State в Update - вот там оно и должно будет вызываться. будет в v2 на днях. заодно верну некоторые сообщения из гм-команд... Однако и так работает без особых замечаний :)
What about something like:

post.to_json(:include => {:author => {:include => :achievments}});

to do nested includes?
alter table rename would be nice
Do you actually use Spamblock Bayes *without* the standard Spamblock plugin? I still have both of them. o_O
Landed on 35b3d151949e023eee07c118bf89826519fee488 thanks.
@johnelse Will this disable the event.from logging to xensource.log as well? I found the entries in xensource.log are quite unnecessary too.
Looks great to me. I haven't tried it though. 

I would have removed all the method javadoc you added. It just adds clutter to the code.
Thank for your quick answer :)
exactly what i was looking for! ;)
I also had to make this same fix after nearly all my tests failed when running headless in a debian environment.  Really was against modifying the core jasmine file, but that was the only thing i could do to make it work.  Running on OSX is fine.

Any ideas why this doesn't work?
I've not done any further investigation. The common thread seems to be Ubuntu/Debian. It may be time time to take this thread over to the main Jasmine repo.
is this combobox functionality ? e.g. http://jqueryui.com/demos/autocomplete/combobox.html ?
@vanraz writeToConsole() тогда зачем?
Pushed with a small change to fix access to getstr.
Warn so we get it in exceptioneer
```java
// and here is a java block
if (true) {
  int i=0;
}
```
But....what....is different? Am I blind? :S
We have to change this after in some time.
For warning/error in runtime, probably we'll go with exceptions. (not implemented yet)
mvn findbugs:findbugs
mvn findbugs:gui
@rafaelgg note that we do not distribute mailboxer migrations. We call mailboxer generator from our initializer.
In the windowlist applet we have worked around this problem by defining a seperate function to which you reference with Lang.bind. But of course doing it this way is also allowed.
Ahh, ok. I saw the gunship_battle.cpp with the //, so I thought that you forgot about the function xD
I don't see the value in changing from 'oecuser' to 'OECusername', especially as this is inconsistent with all the other property files naming the database user.  The same applies to the password change.  In production, a REAL password needs to be used that should never be committed to github.
squadzone Mr. hello:
Here to express my sincere thanks and gratitude to you. There is a small problem, I hope you can give to solve.

From February 28, in the XDA forum that use them to update the software, you have not been able to update, always reality of broken links.

I hope you give the one I like the hair on the previous XDA forum download link in this thanks!

Can also be given on this page to your download link.

Here is my email address: lgj6kxdfr@gmail.com

Once again, thank you for giving us such a good work!

English is not good, please understand
I'd give this one more round of testing. Permalinks work with my code now, we should probably add a test, though.
Fixed all issues from code comments. Pull request coming.

As to font size.. I settled on 40 as a good default. As to the unit, it's the same unit as else where in the program (pts).

Since this is a niche feature it makes sense to give the user a bit more control on the size.. she may just want a smidgen of marking, or want them to be huge. If the watermark seems to be going out the page, the user can try with a smaller size. By doing size calculation ourselves, we're reducing flexibility a little.
replace this with

```ruby
assert_equal File.readlines(gold_file).each(&:strip!), out_lines
```
To make it cleaner we should store the current value of the pref. With this._isEnabled it's hard to follow the code path.
finding a hardcoded path on a test, that point to a home folder.
It appears that intelligent guess works differently in Firefox 9 and 10 depending on whether the request is to a local resource vs a remote resources.  Chrome parses the response as JSON whether the request is local or x-domain whereas Firefox will parse the response as JSON on a local, but on cross-domain it will be left as a string.  forcing the dataType will address this issue.
Step 1: SET @OGUID = X; -- Set guid (1 required) <--- Error: Column "X" not in Field list

Step 2: SET @GUID = X; -- Set guid (2 required) <----- Error: Column "X" Not in Field List

Step 3: (@GUID,@PATH,1,0, ''); <---- Error: Column "guid" Cannot be Null

Step 4: Line 115 <----- Errot: Column "id" Cannont be Null

plis fix T.T
I would be glad to merge it! Can you make a PR ?
имена методов должны быть по-нят-ны-ми
пиши maintainable код
INSERT INTO `achievement_criteria_data` (`criteria_id`, `type`, `value1`, `value2`, `ScriptName`) VALUES (7567, 11, 0, 0, 'achievement_the_hundred_club');
INSERT INTO `achievement_criteria_data` (`criteria_id`, `type`, `value1`, `value2`, `ScriptName`) VALUES (7568, 11, 0, 0, 'achievement_the_hundred_club');
Unless I am misunderstanging the use of autotagStopAt, the code to set it should be.

```python
      if vim_global("StopAt"):
         self.stop_at = vim_global("StopAt")
      else:
         self.stop_at = self.home_dir
```

Currently it just gets set to True if user hasn't set it.
woot, i can just say: Agree to disagree :)
why?

we had errors without this on the sheevaplug (not on my notebook)
Well we don't need the banks on our server so those have been removed completely.  We don't currently use any plugins that hook into banks but if we install one in the future it could be a problem as the iConomy5 implementation in Vault would get a NoSuchMethodError if it tried to access them.  This will also probably be changed around more to fit our needs.  Would the simplest way to keep this compatible with Vault be changing the package name?
need to implement this
> default="http://disney.com/
ಠ_ಠ
oh ty
Ah, fair enough. In that case, it should be a constant.
Fixed that.

So, you have decided to do cherry-picking in the end?
@Rob: This is probably more in line with what you had in mind, right?

I only changed that one method as reference, can do more when we agree on the general picture.
jaime pas ton commit
Are you planning to adapt these patches to 2.0 ?
I don't trust any of the hthor code that looks like this - the whole way stand-alone file names are mapped to physical filenames needs redoing.  
However if it is the same that is probably ok.
oic.
@seamusabshere: Um, yikes, do we really have to embed this "embedtweet" thing on every page?
+1 e000...

Lets play ping pong :)
Did you try running this?
I was planning on using it in the azure service bus project.
i think to thank Shauen and Subv -_-
For SQL, ANY kind RCS, let alone DVCS is overkill. You can't exactly edit a SQL file -- you have to create a new one if a mistake is made, so there is NO reason for revision control. FTP would work just fine. But since we do DB and core in one repo, it is needed evil.

Anyway, learn the CLI dude, you will be much happier in the long run -- the GUI sucks ass.
It will be a good day when I rewrite these tests with fudge.
Hi, I am reading through some of the code to learn about the algorithms - they are very clear and informative. Yet I am a little confused here and would really appreciate it if you could explain a little bit.:)

It seems that the way you solve the least square problem is by fixing either item biases or user biases in tern and optimizing the other. But why aren't user_biases and item_biases' value set to zero before every time they are updated? Is it some "smoothing" technique to prevent the value from changing too much?

Thanks a lot, and happy new year:)
I do agree with you to a certain degree. At least it is documented thoroughly.
тут можно сразу при инициализации
user_items = Array.new(3)  { FactoryGirl.create(:item) }

а  name: "Name #{n}" - вынести в фактори sequence
Looks good to me except "if" and "then" on different lines. But Ville already commited another variant of ssh-add completion. Your patch needs to be updated now, I suppose.
Good catch, fixed and rebased
Not exactly “all” possible events, but close enough I guess.
If I understand correctly, you are taking notes about what remains to be done. No issue here, but add "FIXME", this will let you find things more easily.
whoops, I think I added that ._.
Just make `enableTLSConnectionManager` private. It's already called within a `synchronized` block.
Am 26.01.2012 21:49, schrieb Anthony Theocharis:
> All of the files in /public/admin/ should be GPL'd still.

Completely fine with me - why did you commit that to a separate branch?

Felix
Nit: space after //.
In the code, GetOptions doesn't use for getting config path anymore.
It only use in buysell_deploy.pl in order to choose between test and production environment.
I replace Readonly package to Const::Fast package and use it in this file
I put the first translation files you sent me into a /lang subdirectory. Probably not a big deal, but I'll have to figure out how to merge these updates there.
Good idea, I'm gonna steal this up from you.
Sort of but they are specific to this particular use. I wouldn't write this out here except it may help clarify the RPC stuff a bit.
i think that you should change "P500" to "THUNDERG"
On Feb 14, 2012, at 4:54 PM, Peter Kleiweg wrote:
> At home, I have the latest version of Debian, Boost 1.47.0, and I don't have libboost_thread-mt.so, just libboost_thread.so

You should:

http://packages.debian.org/sid/amd64/libboost-thread1.48-dev/filelist

> You can just leave it out of setup.py, so why bother?

Well, for one, it breaks on other systems. E.g. the module will not load on OS X.
That's a mouthful. Can you try and rephrase or put some `,` to let the phrase breathe?
Had the same idea about pictures, but got my N900 back from repair after a short while, so didn't get around to it :)
thanks that was very informative
Currently using OpenSwift libaudio,and adapted a lot of stuffs for making CM6.2 work - they are all included in initial commit.
In any case, I've reverted this: https://github.com/DarwinAwardWinner/volatile-highlights/commit/6e18deac98c2e162490df13c30eb4f9b3cdbef4c
can you simplify as:
for (Run old : job.getBuilds()) {
?
Балшой стал регексп. Оче балшой. Slow as hell.
As a new user to Sublime, it took me a while to realize where the Packages folder existed. After you know where it is, it's straightforward, but it would probably be helpful to add something like:

`open /Applications/Sublime\ Text\ 2.app/Contents/Resources/`

but to open the packages folder. 

Also, you should mention something about this plugin building the `subversion.pyc` file. 

All in all though, LOVE this package. Makes my life so much easier. Thanks for putting it together.

Another side note: Are you able to commit multiple files at once with this package? I've not been able to figure that one out yet. Cheers!
ewwwwwwwww! this is wrong on so many levels...
Wait, no, I'm wrong, you're right.
i maybe have touched, no thing changed.
I was thinking more along the lines of having the conditional branch based on `e.attrName`, and perform the conversion in each block of the conditional. They way we'd have all the new values setup and we can do one big `setAttrs()` call at the end of this function.

```javascript
_afterUnitChange: function (e) {
    if (e.src === 'conversion') { return; }
    
    var newVal   = e.newVal,
        attrName = e.attrName,
        celsius, fahrenheit, kelvin;
    
    if (attrName === 'celsius') {
        celsius    = newVal;
        fahrenheit = (celsius * 9 / 5) + 32;
        kelvin     = celsius + 273.15;
    } else if (attrName === 'fahrenheit') {
        fahrenheit = newVal;
        celsius    = (fahrenheit - 32) * 5 / 9;
        kelvin     = (fahrenheit + 459.67) * 5 / 9;
    } else if (attrName === 'kelvin') {
        kelvin     = newVal;
        celsius    =  kelvin - 273.15;
        fahrenheit = (kelvin * 9 / 5) - 459.67;
    }
    
    this.setAttrs({
        celsius   : celsius,
        fahrenheit: fahrenheit,
        kelvin    : kelvin
    }, {src: 'conversion'});
}
```
Why do we need to run with this modifier?
Agreed. The setter() wasn't part of the interface.
Thanks, I understand. Could we perhaps extend HashTable to get around this 
issue?



-Misagh





From: Eric Domazlicky [mailto:notifications@github.com]
Sent: Friday, September 07, 2012 2:07 PM
To: Jasig/cas
Cc: Misagh Moayyed
Subject: Re: [cas] CAS-1187 Add a Memcached based map for ClearPass (#121)



In 
cas-server-integration-memcached/src/main/java/org/jasig/cas/maps/MemcachedBackedMap.java:

> +
> +
> +/**
> + * Memcached-backed implementation of a Map for caching a set of Strings.
> + *
> + * @author Eric Domazlicky
> + * @version $Revision$ $Date$
> + * @since 1.0
> + */
> +public class MemcachedBackedMap   implements Map<String,String> {
> +
> +    private final CacheMap cacheMap;
> +
> +    private final int mapTimeOut;
> +    private final String mapPrefix;
> +    private final static String nullPlaceholder = "#null#";

Null's can't be stored in a Memcached server because everything stored there 
must be serializable. The nullPlaceHolder gets around that restriction and 
is the reason for the replaceIfNull, checkIfNull functions.

—
Reply to this email directly or view it on GitHub 
<https://github.com/Jasig/cas/pull/121/files#r1558840> .

  <https://github.com/notifications/beacon/J6T91GIPIyhU-8ti4GCGP-_jbPsNU1nRwZIJ46-275BoBxQ1s_qTQIFQgbL-oVXN.gif>
I measured this a while back and it's substantially cheaper, but I think it's fair to argue it's premature optimization.
Turns out this change didn't need to happen :P
I'd use formatting style that is followed in other files in this commit, i.e. when using chaining each function call is in a separate line.
ah, okay, wasn’t looking at a large enough piece of context here.
This shouldn't be done here, ActorCell should be agnostic about dispatcher implementations.
remove parens around msg
when changing to returning a `Future`, the timeout parameter could still be passed into the created supervisor, so that that actor uses it as a receive timeout and shuts down the whole operation when that fires
I generally see them with the namespace nesting explicit, but we can look at this more for our uses.
s/Since/Because of/
Do you mean in the docstring but not in the parameter or both?
In order to express it in another way:
everybody can call this function.
However, he will only be able to modify his subscriptions as the rest gets filtered out automatically.
Не понимаю, почему вы решили отдельным методом, если внутренности одинаковы, может лучше добавить хотя-бы private метод реализующий основную внутренность. И почему user_id, если зачастую он $this->user->id
Почему бы не сделать как в методе выше, возможность вывести например только один параметр.
I guess if we're not doing #318 now then it makes sense to keep the attribute name the same as the config option.
if-clause to check if file still exists?
Getting warnings without this.
This is wrong. It should not force the dev version. This forbids the beta versions (and the stable versions when they will be there)
c'est ce que j'avais fais a la base mais ça marchait pas, donc j'ai décidé de décompresser le zip dans un répertoire puis de le supprimer ensuite
Better: return a bool for failure/success and add the temporary file as an output parameter (std::string *local_path)
I commend your use of Popen.
Multi-line `if` statements should always have braces.
It still requires full attribution.
Hm, I've never seen `||` and I don't see how it differs from `|`: `all.equal(c(F||F,T||T,T||F),c(F|F,T|T,T|F))`

Also, this is my first foray into pull requests, so I don't know how to change `options()$cores` to `getOption("cores")` (and `as.numeric(options()$cores)` to `getOption("cores")`).
Also, I prefer an iterative approach over a functional one in tests. It's easier to debug when something goes wrong.
Agreed with Tom on this. Otherwise it looks good.
This shouldn't be the case; can you profile to see where the slow-down is?  Calling the cached version should be faster, if anything.
Я имею ввиду kvadratik, а не ps
Now that you said I think the assertion was right since `'foo'` is not a number either if it casts to `0`
Should this method be

    cache_if !condition, name, options, &block

?
I think it helps newbies, who forgot to add polymorphic option.
but should be better that ```@errors``` is cloned also ?
Grrr, sorry about that.
If you want to go through the pain of macrodef, feel free.

And no, you don't have to use the one in starr, just don't delete the one
in locker.  We can put it in a different classfile directory.
I agree with @xeno-by that the wording should be changed.
More illuminating comment: "For hermitian matrices, singular values equal abs(eigenvalues)"
AFAICT, these are not used
Please explain why you are deleting all the zones here. I don't see any reason to do this in this test.
Or if you want, just make a similar change here on your branch and we can just throw out #1481. I don't really care.
Then this is a hack, right? Perhaps this would be a good time to use a XFAIL doctest to show how some routine doesn't work unless the space is there and to include a comment that says to "fix line in latex.py when this passes". Best would be to fix it so the coeff doesn't get stripped.
Getting a bit ahead of things perhaps, but I would also like to suggest that instead of 'measure_numbers' the method be named 'components' or something of the ilk.
I am working on removing most of the non-Basic instance, however on many occasions forcing the use of Basic is exactly Inner Platform Effect http://en.wikipedia.org/wiki/Inner-platform_effect 

I would prefer fixing the parts of sympy that do stupidly naive recursions of the expression tree instead of continuing this.
@goodok ok with this. Because, his earlier suggestion was to deprecate this.
and attach event handlers as declared in {@link events}. 
Make sure you've declared handler as public and call the parent implementation if you override this method.
Same here. Is it for saving a function call?
Okay, I think I figured out a good solution that works well for both finite and infinite iterables.  Let me know what you think.
Indeed it is, good catch.
ja PDF does not yet exist, and PDF genereate is very difficult...
Temporarily, PDF link switch to en.
I don't like "angle1" and "angle2" because it's ambiguous what that means.  Perhaps `latitude` and `longitude`?  

Note that for all the comments from CoordinatesBase on down, it's probably easier if I take care of it with the transform stuff (so our changes don't collide), so I'm fine with merging these parts as-is... I'm just mentioning it now for discussion so I know what to do in the transform branch
I would reset it just to be safe, because lolappkit.
Well, I need to rename it then.
In pretty much all cases, the methods on CategoryFunction being used were actually the ones supported in ScanAggregation - it was just we'd never pushed through the simplification of the type.
the use of `errmsg` in this context uses a literal term in a sentence, which is unclear and contra indicated in the docs style.
Only if it's unreadable to the user but DOES exist - same answer as above for the rest.
Il faudra éventuellement changer ça. Peut-être quelque chose comme *OODA Technologies*.
This is not really 'my solution'. I just modified the logic slightly and moved things around, the actual $message stuff was already here in its present format.
Have you tested this? It doesn't look like it will work.
Moderators should not be able to assume all user level permissions, so if they do not have permission to view all other topics they should not be able to.
User submitted filenames should not be used on the filesystem at all. Look at how files are stored by the "old" mechanism.
to not be
Not sure whether these function calls are separated on purpose. I've got the feeling calling str_replace with arrays is fast for big `$data`.
`$('textarea[name=message]').value()`
@padolsey i'll try it in the worker now
We do this in our app now and was introduced before we moved all affinities from server to common. This could cause issues with mojitProxy.destroyChild and possibly introduced memory leaks when refresh view is used. Its worth looking into the affects this has on the client before its baked into Mojito.

@kfay may have more info here.
This is a better means to load dependencies into lib from any new jar projects that are added to the reactor under dspace/modules/... without needing to be specific about there being a dependency clause in dspace/pom.xml
Is this code now functionally different?
$dbw is used for reads (then writes to cache) so because sometimes loading the model from a non-master database will result in outdated information.
Potential bug? Should m_parameterizedXmlSQL keep its value if preExistingParams cause it not to be used?
There IS a seemingly significant null check for it later.

Should it even be duplicated when it might go unused?

It seems like all of the issues around parameterizing could be handled more cleanly "in one place" by something like:
        m_parameterizedXmlSQL = static_method_that_wants_a_name(m_xmlSQL);
        if (m_parameterizedXmlSQL != null) {
            //     log and return as parameterized
        }
        // return as unpameterized

I suppose you could make that method a static method of Parameterizer, if you think that class needs to exist. I've got some doubts.

I DON'T see what having an INSTANCE of Parameterizer buys us -- to me it just means we have to call a constructor and a method, which seems like 1 call too many.

In any case, one place in the code should take care of detecting and dealing (or not) with pre-existing params and deciding duplication etc. as the
design may evolve -- right now it seems too scattered between here and too many exposed Parameterizer entry points.
Good call Alex. You are right.

Benjamin A. Petersen
ben@benjaminapetersen.me (mailto:ben@benjaminapetersen.me)
linkedin.com/in/benjaminapetersen (http://www.linkedin.com/in/benjaminapetersen)
facebook.com/benjamin.a.petersen (http://www.facebook.com/benjamin.a.petersen)
twitter.com/#!/bapetersen (https://twitter.com/#!/bapetersen)
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)


On Wednesday, June 27, 2012 at 11:49 AM, Alex Ehlke wrote:

> > animate = animate || this.options.animate;
> > 
> > + // var keyword to avoid global!
> > + // var tag = $(tag); 
> > 
> 
> 
> However this should never be undefined unless there is a programmer error, since `tag` is required, and so this will always bind to the `tag` parameter passed to removeTag. So no need for `var` here.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/aehlke/tag-it/pull/82/files#r1061040
> 
>
commit=True is the default
In the past we haven't, but the deps should work out so that the header is generated before riak_kv is compiled.
Should I put the match back in, then? The tests will fail with the "already registered" error until the `deregister ` magic is in, and then they'll be happy again.
no need to pass empty string to constructor.  `stringstream input;` would suffice.
oops, went down a path that didn't work and forgot to cleanup. good catch!
I don't think it's accurate to query the database at init time, as we are not even sure yet that we will need this info. I'd rather have a cached property, so it will only get fetched when we really need it.
Right, I understand what you're trying to do *however* if we expect the two height values to be different then this test would fail in all browsers that don't adjust their innerHeight on scroll. So, yes, that would pass for iOS, but fail for everyone else - including all of the desktop browsers - because when you scroll 150px in Chrome, the height is still the same.

Unless there is a way to target one individual browser for a unit test, that's no good.
It *is* a magic number, and... no. The choice of how good a fit to demand while protecting ourselves against nontrivial loops is arbitrary.
yes. I add more more indentation for the line below if statement.
On May 1, 2012, at 7:07 PM, Hitoshi Harada wrote:

>> +	plpy.execute("""INSERT INTO rtbl(seg_text, label, value) 
>> +	                SELECT segment_hashtbl.seg_text, features.label_id, features.weight 
>> +	                FROM   unknown_segment_hashtbl AS segment_hashtbl, 
>> +	                 """ + featuretbl + """ AS features 
>> +	                WHERE  features.name = 'U';""")
>> +
>> +	# Wordfeature
>> +        if document_id == -1:
>> +		plpy.execute("""INSERT INTO rtbl(seg_text, label, value) 
>> +	                        SELECT seg_text, label_id, weight 
>> +	                        FROM   segment_hashtbl, 
>> +		                """  + featuretbl + """ 
>> +                                WHERE  name = 'W_' || seg_text;""")
>> +
>> +	# Factor table
>> +	plpy.execute("""INSERT INTO """ + viterbi_rtbl + """(seg_text, label, score) 
> 
> The indentation here is ambiguous.  I assume this line is not inside "if" block but should be run always.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/madlib/madlib/pull/103/files#r759169
>
Copy not needed when returning the first parameter (state) of an aggregate function.
I tend to agree.  The old code was using C sort order, which is what we are fixing.  If we are adding proper collation support then we should presumably handle this case, otherwise we aren't fully addressing the reported bug due to inconsistencies between db sort order and lda sort order.
This .replace('T') nonsense is crap :-) Remind me to look into convincing the validation to accept valid ISO 8601 (i.e. with T)
I agree. I want to simplify this so that it only exposes the public methods directly on the object that was "mounted". This way, what's exposed also matches what the inspector returns.
I think "memcached" here is the name of the daemon which is "memcached" on my machine. Is it "Memcached" on yours?
Don't delete this here.  In rare cases usage will be printed out by rhc so we need to keep this whole until we can remove the whole thing all at once.
You're right. Was rushing out the door, so didn't have time to look at it. The code is scheduled for refactoring, so haven't changed much lately except permission checks. Can I get you to enclose the prepare and execute statements in a try/except block and return false on error? It's easier if you do it in your branch ;)
Good solution, though weird that it acts different from the original.

Thanks :smile: 

PS: There is a findWhereDAVDataIs() method, but I guess querying directly causes slightly less overhead.
a prendre en compte
Maybe save `now` at the top so we don't get an 11 second range if the tests run slow?
Can anything other than a str come out of request.args?
Can you move this to above the `from warnings` import line?
Regarding "if defined $state[$_]", this is just bad copying from me.  The dither routine explicitly uses the 'undef' to handle dither parameters changes that don't toggle state.  It would have no effect in the radmon section (if we're looping over the states, I don't see where one of them would be undef), but should be cut.  It should also be explicitly commented in the dither routine.

And thanks for the $time_violation feedback.  I'd like to have this code throw more errors instead of passing back flags, but if I've got the flags it definitely makes sense to use them to stop further checking with possibly bad data.
Why does fProvidedPools affect whether we wait for the futures to finish?
Makes sense, but `actionStart` doesn't read well and `actionTaken` goes against `appeared`. What about `startAction`?
this if need to flip the logic, and only active the new path if it's for /oauth/authenticate, and everything else fall through to the existing/old block... the reason is, there are numerous other paths through finishAuth() that are also not oauth/authorize and depend on setting auth.account to force the account (search for .account above to find examples)
I hadn't realized that we're barely testing this. I don't think it should block this pull request, but we should get more tests behind `_Adapter` and `toura.adapters.tourjs`.
For consistency, please call it `onclick`.
It looks like this will only ever reopen the most recent file closed. Is there a reason to keep anything but the last item given the current functionality? Or was the intent to support other functionality in the future?
I don't see where the provider being registered for "all" modes gets added to the ``hintProviders.all`` list. Should that be done here?
No need to add this constant.  The idea with menu sections is that they describe a group of related menu items.  Save and Save All would both be in the FILE_SAVE_MENU section.  (Although support for using these constants isn't yet implemented).
Once Harry's Ace update PR is merged, I will merge master in and change this to be `"ace/mode/css/csslint"`
Line 143 - 146: Isn't this already taken care of in the ExportService ? Beside, the if statement is useless...
On two lines, please : it does not fit in my screen. And it is quite large...
Exactly, you will have to override `__init__`. But you do not need to pass `FUTUREDATE`. We do not need this. Just pass the `upper_limit` value, which is some date. And you can do simply:

```
def __init__(self, upper_limit=None, lower_limit=None, *args, **kwargs):
   self.upper_limit = upper_limit
   self.lower_lmit = lower_limit
   super(LimitedDateTimeField, self).__(*args, **kwargs)
```
I noticed that `type` is capitalized in the error that's thrown. Might be better to throw out the original value of `type`.
these two if blocks can be merged
`var timeStamp = new Date( 0 );`
If memory serves me correct these are the transitions you wanted for titles. I think you felt that slide up/down wasn't appropriate for titles. If we want them in then that's an easy change.
Are we inlining styles now?  This won't get any CSS lint goodness.
Why the second switch statement?
PartialEvaluator here cannot be reused, since page and annotation have different font references, so my vote is for getAnnotationOperatorList().
Is there any reason, for doing that check? The old/fallback implementation doesn't explicitly ignores `ca_certs` if `cert_reqs` is `CERT_NONE`. So why do we do so, here?
Worth looking into how much work it would be to use util-logging instead lf println:ing stuff. That way we can have the default to log to the console but can also redirect to local file or via scribe.
This test doesn't really test anything
This happens on initial page load - setting autoActivate to false deals with that particular circumstance, but I put this here as well in case there are other situations where it could happen.  It might not really be necessary, but I figured it wouldn't do any harm.   I can remove it though if you prefer.
This looks expensive!
My note was more about the naming convention. We do not change file names for any template in Katello. Therefore if you need to find foreman.conf, you wont find it. I would rather see it in templates/etc/httpd/http.d/foreman.con.

Of course what you point above is not correct and should be fixed. But config_dir does not change.

I like to find config files by their filenames very much.
Set your editor to remove trailing whitespace. :)
% has to be outside of _(), otherwise translations can't work
I have found variant that works like you proposed: %config(missingok), this works like %config but if the file is removed, it doesn't try to recreate it again. But if it exists, it updates it as needed (or creates .rpmsave file). But I wouldn't encourage users to edit the file. But they still can remove it if they don't like it.

I wouldn't use noreplace, because it's more script than a config file and it's not expected to be edited so often. What about making a symlink instead of copying the file to cron.daily, which would disable file editing, but still enable control over existence of the script in cron.daily?
Hrmpf...didn't need it, so I've removed that one
I know uint isn't CLS compliant but our assembly isn't even CLS complaint. We'd just need to verify that it works from vb
https://github.com/andreasgal/gaia/pull/1004
Can you attache the bug number for it?
I mean when we switch the write to haojing, we may now make the interface without implement.
I could put it in the view or the user model I suppose....which is best?

*Nikhil Goel* | Vanderbilt University, 2013 | Computer Engineering,
Economics**

LinkedIn <http://www.linkedin.com/in/ngoel1> |* ngoel36@gmail.com | ( (256)
503-9183 | NikhilGoel.me <http://nikhilgoel.me/>



On Thu, Nov 29, 2012 at 6:32 PM, zsherman <notifications@github.com> wrote:

> In app/helpers/videos_helper.rb:
>
> > @@ -1,2 +1,9 @@
> >  module VideosHelper
> > +  def bookmark_listing
> > +    return current_user.bookmark_listings.find_by_video_id(@video.id)
> > +  end
> > +
> > +  def bookmarked?
> > +    return bookmark_listing
> > +  end
>
> I've always put functions like this in the helpers, where are they
> supposed to go instead?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bitcast/bitcast/pull/22/files#r2273341>.
>
>
The original idea was to reduce the risk of merging the code, in case there were issues with LevelDB [on some specific platform] we don't want to hold up the release or do a potentially messy revert.

I agree it's irritating and a burden, but it'd suck if all of ultraprune ended up getting reverted due to unanticipated issues with LevelDB. Once 0.8 has been successfully rolled out to the userbase and things are quiet it could be deleted at that time?
Sorry guys for the inconvenience, how could I have just renamed the file without replacing it?
btw, the name is quite confusing compared with ``getProxyClassName``. You should try to find a better name
sumthin's up here
Мне показалось или оно пробелы на табы заменило?
If they are different things they should really have different names to avoid confusion. Something along this lines of `DeploymentZone` would be more helpful.
It's available.

http://ruby-doc.org/core-1.8.7/Object.html#method-i-tap
Documented in the Javascript object docstring.
`generateHTML`, `parse` and `highlight` all call `getLanguage source`.

If generateDocumentation took code rather than a filename there would be no source, i.e. a language object would have to passed through. Would it be better to just pass through the language object (maybe `config.language`) anyway than make extra function calls?
these parsers should be in xml package, and the name should be DeleteResultHandler

there should be a test for parsing in the parse package like ChangeMessageVisibilityBatchResponseTest in sqs
yes, we should have our own.  our computeservicecontext.unwrap() should return the backing compute service
not sure what you mean. you mean testing the org.apache.commons.codec.binary.Base64 directly?
"which should see its note removed" seems highly awkward to me.  However, "of the git object" seems like an informative addition.

How about "The OID of the git object to remove notes from"?
This API makes little sense anymore. We used to have this two-step lookup (`find` + `get_byindex`) because you could `find` a pathname to get its index (location) inside the array, and then lookup that index **and the consecutive ones** to see if there were entries with a different stage after it.

Since now we're querying by path and stage, this function can return directly a `const git_index_entry *`, because there will always be one and only one.
Keeping it in `mwindow.c` sounds good, but the mutex will have to be initialised dynamically, as Windows mutexes are handles (presumably to kernel objects).

There's a couple of optimisations we can still explore to avoid taking the lock, like making the usage counter atomic
also, the current way could cause two wildcards to match the same field causing two calls to addField, while the inverted approach could iterate the match first exact names and only then iterate wildcard fields in order and choose the first match.
Yeah, we still have the class when we need it, but this also gives us a handy way of making a list that gets striped automatically. I can put classes on every other item on the FAQ, but it's all hardcoded so if we ever add another section we'll have to rearrange the stripes. But that's not so hard when it's just a few sections, and we'll get stripes in IE. Though I thought IE8 supported nth-child... I'm probably wrong.

Except... with .expander-odd plus nth-child(even) I think I messed that up (mockup had the first and third items with no background). So maybe the class won't play well with this selector. Doh! 

Anyway, not a blocker for merge but we'll fix it up later.
_page_name is my own new thing, and I need to record it differently. However, I could set _view_name instead of replace / with . in process_view. I'll have to set _view_module to a constant though, so its redundant but it probably would be better than duplicating code here.
I think it's just the "hide more" that doesn't make sense to me... "more" implies an addition, but hiding it is a subtraction.

What about "Click to show less information on this topic"?

Or maybe just "Click to show information on this topic" and "Click to hide information on this topic" (no "more") are actually best since it doesn't imply any amount, not more, not less, just "info".
You don't need the parens around `m.hexdigest()`.
Not entirely sure how this got here, it is a mistake.

I blame vim.
Oh. :(. Maybe during the work week.
Same comment here re efficiency; do a single db query for all the cases rather than one per case id.
What if it's a product with no build in any of the featured versions? Maybe that's a little bit too much, but I'd rather get the has_builds property of each version and use an OR to compute that value. We don't really want to display a link that goes to an empty graph.
There is no need to document what the code historically was, just 'why' it is now what it is. Commit messages can report historical notes.
JSON doesn't allow string `"true"` and `"false"` to stand in for `true` and `false`--so no, the capitolized variants aren't valid, but neither are these.  We need to allow strings because that's what a params hash gives us.  We can decide what variants we allow.  I like keeping it pretty strict and just using the lowercase strings.
@drak it is static because @vicb asked to move the instantiation of Mongo to ``setUpBeforeClass`` which is static in PHPUnit
You're correct. I'll change the commit to using just trim instead.
Please don't remove spaces around ``?:``
fast as always :)
the condition should be added to the previous condition.
so that it cannot be instanciated. Too force only static methods.
AFAIR should be `} else if () {` here and below.
You're right but i was pointing the consistency here. No other bundles are using keys for translations.
It's different from before. You shoudl probably first trim and then replace, otherwise the port is not removed when whitespace is at the end (because of `$`).
you need to do something proper on the exception. It is very important to handle it well. We don't want to show the user the exception trace. Handle it, show a proper message to the user.
eclipse didn't like it wen I tried to return List<Comment>. I'll take another look at it and see if I can get it to work without using Object.
Double end: syntax error or formatting issue?
The api should return just the sessionId. This change should not break your code, because you did a very smart thing by setting this.sessionId into the opentok object. 

With the new api, users who call generateToken(...) will be passing in null/undefined value for their session, and there's the if statement in there to catch that and use this.sessionId.
If I am on the graph view and click on the graph menu element I would actually expect to land with the default options, that is without the current query string. It can be seen as a way to "reset" menu selection. Same for the base view: Only add `query_string` to the non-active views.
why not use streetAddress or streetName?
Yeah there is not too much around, but I found this: 

http://www.nfc-forum.org/resources/AppDocs/NFCForum_AD_BTSSP_1_0.pdf

I was thinking on testing this stuff on the phone as well, but have not gotten that far.
Did you try something like this?

       <configuration>
          <compilerArguments>
            <endorseddirs>${project.build.directory}/endorsed</endorseddirs>
            <AgeneratedTranslationFilesPath>${project.basedir}/target/generated-translation-files</AgeneratedTranslationFilesPath>
          </compilerArguments>
        </configuration>
That would still be fine
<module xmlns="urn:jboss:module:1.1" name="org.jboss.as.domain-management">
    <properties>
        <property name="jboss.api" value="private"/>
    </properties>
It would break setting of the `dialogClass` option, as we read the old value to remove the class. That's the only case where an option is read before being changed.

Looking at tabs, I'd move the `dialogClass` update into an `if` before the `switch`, then call `_super` inbetween. Correct?
Вроде вернёт 2 (id анонимуса).
Проверю.
do you really need `alarm.js` here?
I can easily change that to use a data attribute that won't change the UI. I'd prefer that to removing/adding the DOM listener.

Note that the advantage of using disable is that it reflects what we are doing in code. It prevents the user from modifying the input during the process of saving to pending messages. If he manages to write something during this (sometimes the phone has slowness and you can do so), these modifications won't be used as we just clean the input just after the saving to pending messages.
Weird intent
External images definitely won't load synchronously, and I don't think embedded images will either, so this will have no effect.  It might not be noticeable because most HTML e-mails probably explicitly size their image containers.

It seems like the only real option is to have MailAPI's showEmbeddedImages/showExternalImages take a callback and add "load" event listeners on the img tags they fix up and invoke the callback once each load occurs.  According to https://developer.mozilla.org/en-US/docs/Mozilla_event_reference/load_%28ProgressEvent%29 the "load" event from the Progress spec does not bubble, so we can't just add a listener to the body/document and just have it bubble up to us.  That will require a pull request against gaia-email-libs-and-more, but it should be pretty simple.
Here if you detect that some app is installed you call directly to homescreen in order to launch the app?
@dcoloma It could not be part of the image itself if you want to always have a shadown on every icons on the homescreen. The icons on the marketplace will be of all different types :)
this was causing the creation of a TODAY header even when no messages sent/received today
That's what it says in the "spec" https://bugzilla.mozilla.org/show_bug.cgi?id=802545#c33
Everything related with old edit mode has been removed
Plese describe a little, one line commentary at most, the behaviour of each function.
Nit: don’t leave dead code. Looks like you want to remove the whole animation?