labofsahil/github-event-dataset-2012 huggingface dataset:
What happens if a dependency isn't a brunch plugin ?
How about adding a brunchPlugins: {} to package.json instead of looking at the dependencies ?
Aye. It's cool now though, working fine.
use ByteBuffer.allocate()
Need a version number
For this segment,  since every time new server in this cluster is added , the metadata of the function will be override. it will cause terrible problem if new servers' codes have version problem.
On the other side,we can't remove that metadata update, if so , we have to restart zookeeper server every time when any tiny code change is commited.

This will not be a big problem if we assume that every server has the same code.But still need to keep it in mind when we are going to add multi-version code support.
@peillis , why do you type hint the $manager? any special reason or you just like it like that?
// you can also pass parameters to roles's bizRule
Well this brings order back. But if i remove some items from my inventory (either through dropping or through storing in storage) i have some gaps in the inventory. Are those intended?
Vieze hack omdat course.getCategory() niet werkt... iemand een idee ?
Same comment as for ArticleView - checks for eink screen all over the place are not nice. I don't know if there's a good way to create device-specific activity instance in Android at all since activity class is defined in AndroidManifest, but at least all e-ink specific initialization should be isolated and triggered by a single "if"
@richarddong 其实只要是单对单的，就是除了上述的最后一个，现在的函数已经可以处理了，感觉用了个很伪的多态，所以现在就差list了，然后别的地方需要提醒的都可以参照onr_to_one_notification，我写个文档把～
@bcxx Learning Python 中译版的权威性有待考证不过赞同弄一份术语翻译清单作为对照，这个事情可以提交给华蟒社区看看。
almost epic ^)
thx man.
So help me to fix Venugh, I just copied from the skyfire.
Is our copysign also efficient?
So, your problem is gone now?
nothing has to be saved there, elementals spells are in spellbook only while the pet is active
помоему мы уже обсуждали database.yml
1. приложение не должно содержать пароли (если надо установить пароли то это делается в файлах которые находятся в гитигноре)
2. надо дать возможность другим разработчикам самим выбирать настройки для БД: имя, пароли, возможно сокет и т.д.
heh merges can be tricky, np
I did actually! Right here:

https://github.com/mdp/rotp/pull/4

Irregardless, I also submitted this pull request, which lets you run rspec as default rake task (which wasn't working for me):

https://github.com/mdp/rotp/pull/3
yep,
You have the right idea. On forum, posted the complete conversion of this script using Stephen's relatively recently created htmlcpecialchars/xl combo functions. Much less cumbersome and relatively quick with those functions. Also updated the howto security page regarding this on the wiki: http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan
I'm hoping to be able to spend some more time on this stuff in the codebase; I actually converted several modules in the codebase back in June of 2010, but stopped when the MU stuff became a much higher priority.
-brady
Thanks Jeremy; I'll definitely check out the quick Stroop test tomorrow and see if I can't fold those MRSettings tests into it. 

As of now I have been unable to get all the tests passing and app building when using a cloned repository because of my architecture and python setup issues, but am working on resolving them (I'm also keeping a list of fixes for my specific setup that I'll hopefully compile and share).  I think the list of version libraries on the installation page on the wiki may need to be updated (maybe just the output of the last known good `pip freeze`?)  e.g. wxPython 2.9 appears to me to be incompatible for example (it uses a .color method that has been removed, I think, although the wiki simply says 2.8 or greater).
Sorry that's off-topic; look for some updates later.

Anyway, I will continue to work on the tests, and hopefully will have one built soon. In that case I'd hold out on merging the commit until I can get some testing. Soon, hopefully!  Thanks,
Erik


On Feb 21, 2012, at 9:28 PM, Jeremy R. Gray wrote:

> thanks for working on this, Erik! 
> 
> tests would be great. note that there's an existing test that simulates user key responses in order to test a small version of the Stroop demo. its testRun_FastStroopPsyExp() in testTheApp/testbuilder/testExperiment.py
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/kastman/psychopy/commit/f203ee6929daefa2b999d8af73f419c55a6b358c#commitcomment-994315
It was throwing an error in specs, so I assume that it's not.
sounds good i think it might be more "correct" to be without it since someone may well have intended to assign `null` or `undefined`
preparation for #41
> Opera will support IndexedDB in a future release.

[reference](http://chat.stackoverflow.com/transcript/message/2447763#2447763). 

As for Apple, well isn't indexedDB in webkit? So it will be in Safari eventually.
name="polls"
title="Голосования"

Name используется в пути; например, /{sectionName}/{groupName}/archive sectionName и есть name
Title будет использоваться везде. где надо визаулизировать название секции. Не отображать же английское название?

Если убрать последний коммит, то в списке секций для фильтрации будут отображаться англоязычные названия.
it looks like you fixed this so you can remove this comment
Uh oh... please not this way. Read https://docs.jboss.org/author/display/AS71/AS+7+Testsuite+Developer+Guide and fix.
The implementation looks good, but code style should be made a little bit more PEPpy, and [0] inserted at beginning of bit vector seems unnecessary and confusing, though I am still not familiar with the full algorithm. And I do think this would be better in a module with related metrics, and with external doctests in nltk/tests Thanks.
I'm an administrator, I think admin should have deletion permission :)
of all the pun opportunities you come up this? don't be square!
Hi, i face some issue on CI 2.0.. about hosting problem.. i need some one help.. this is my first project developed using CI. so i face index.php issue in base URL.. ple help me... i used .ht-access file but that file cant removing index.php.. in localhost base_url working properly, but when i hosted base URL should not work without index.php. ex: www.xxx.com/index.php/search. 
 .htaccess file...
 RewriteEngine on
 RewriteBase /xxx.com
 RewriteCond $1 !^(/index.php|resources|robots.txt)
 RewriteCond %{REQUEST_FILENAME} !-f
 RewriteCond %{REQUEST_FILENAME} !-d
 RewriteCond $1 !^(/index.php|css|images|js|robots.txt)
 RewriteRule ^(.*)$ /index.php/$1 [L]
 i face page not found error... 
 if i use base_url in config file like $config['base_url']= 'http://www.xxx.com/index.php'; the link should work properly.. 
 please help me, how can i remove the index.php in CI framework when i hosted..
I don't know about your background, so I have no option but asking: Are you familiar with the concept of YAGNI?
So THAT is why you didn't need to handle regular expressions - you wrote your very own Scanner! (I didn't even think of that.)

If you prefer, we can adopt it.
ok, now i see... the jquery mentions input should be using the jquery auto-expander plugin, because it's way better and less buggy....
Why is that necessary? Load order? If loaded in one chunk the load order will still be preserved.
I just upgraded an "older" Rails 3.1.3 project on which I had scaffold view tests generated with
rspec-rails 2.7. They now generated a large amount of deprecations. I was able to remove them
manually, thanks to help on the mailing list.

Is there a documentation patch we can apply somewhere to explain the problem and the solution
to rspec-rails users?
In `variables.less` there is: `@gridTotalWidth: @gridRowWidth;` ...so they cancel each other out on this line, meaning it could be reduced to:

    (@gridGutterWidth + @gridColumnWidth) * @columns - @gridGutterWidth
Found another "Rain Showers" also exists. I think Google must have literally hundreds.
Will try and track a few more down.
other branch just for test
It is not possible actually to get backtrace from freeze it just stucks ... and no other info thats why i only posted laste debug lines ...
To get the user profile in Live, use "http://apis.live.net/v5.0/me/picture". The problem is, the URL of image comes in a 302 response.
raise ArgumentError
I agree with Phillipe. Not that I don't like "**" of types when needed then Yes.

But in this patch the pointer-to-pointer are not used.
Every where that had:
- char* was changed char**
- pass of text => &text
- text = X => *text = X

No where is it used that that text = new_buffer . That is we return a changed pointer.

If you need char **text to be an output parameter anywhere it is not seen in this patch. Where is the use case of this change

Boaz
Either syntax works.
They're present in how stylus handles options being passed in with plugins from the command line. Copied it. Just any plain ol json. 

-- 
Andrew Cates (iPhone)


On Jan 6, 2012, at 5:59 AM, Joe Hudson<reply@reply.github.com> wrote:

> Where do the options come from?  I didn't know we could do this and I need to add this to the documentation...
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/walmartlabs/lumbar/commit/aea776e30c91a3cc58228f18ca0684b3817eb697#commitcomment-843711
Aye. There are reasons, but none strong enough to excuse such an atrocity as this. I profusely apologise to the wider developer community, and will surrender my Rubies and SQLs and NoSQLs and Pythons at the next available opportunity.

(Using env variables was annoying everyone)
Good job @robyurkowski. :heart:
Хорошая работа :)
Acá no falta un salto de línea adelante?

Selección de la herramienta no debería ser de nivel 2? es decir con dos ##?
Let' say 0.5 is dev and 0.6 will be tagged
I'm not sure is that's easier because you're putting some of the code inside the <<-TEXT block?
That does work. Thanks!
fetched from studyers/instanotes master
... the current route
Make it clear this is for CouchDB, so use:
'CouchDB Hostname'

And do this for all of the titles below also.
The "flying motion" of Death Grip, that has been fixed long time ago, you have to look in the repo...
I was thinking of a true DSM UI not the Transmission Web UI
Why not put that back, was part of the cleaning :)
We can use Buffered output stream to make the transfer quicker
# It's time to stop up and reflect on where this is going

Hmm... The Transform interface does not really buy one much in the way of abstraction, but instead detracts abstraction by requiring concrete return types everywhere. In essence, covariance is bullshit and I think it's probably a better solution to just redefine `transform()` methods for each of the collection abstractions, rather than in a single place. We might also need specialised transforms, sort of like what was added for tuples, and not being locked into the Transform interface would permit that.
I implemented the macro CLEVER_TYPE_ARG() for helping us to retrieve the type argument given the type and its index.  In this case, we just have to set the return type according (in this case the Map's KeyType which is the 0th type argument).
@pixelfreak It appears that dragging simply isn't added to this map. Can you explain your other issue more thoroughly? Is the behavior incorrect, and could you submit a pull request for a fix?
maven-dbflute-plugin を利用するため、 [Seasar](http://www.seasar.org) のMavenリポジトリを pluginRepositories に追加します。
Nice one!  So in theory it should still be compatible with commons-pool 1.5.x (the Pelops changes are just syntax candy)?
Hmm gimane ye
Also a copy-pasted copy call.
checkInFileEntry() actually needs to be much more complicated than this one.  Because, you are assuming that the last approved version has the same preview images as the current one – which is most likely not true.  In fact, if somebody checks out the file, they may change the binary multiple times in the PWC until they check it back in.  However, this may also mean that they may change it back to the original file before checking it back in.
Awesome thanks! Would you be willing to do a wiki entry about some possible use cases for this?
As seen in JavaMexico +1

http://www.javamexico.org/blogs/sr_negativo/problemas_al_usar_ceylon_ide_no_ceylon_runnable_element#comment-18942
What happens when $EDITOR is not set up?
Shouldn't we 'seed the database with data' as well here?
Take a look at the refactored code :)
I was going to look into making this libary 1.3 compat but based off of this note it seems that you think it isn't needed with 1.3.  What file manipulation functions did 1.3 add exactly?  I know `clojure.java.io` has some new functions but not as many as `clj-file-utils` provides.
I think a doc on measuring performance would be incredibly useful in its own right, outside of the context of parallel collections. Instead of throwing this into the overview on parallel collections, maybe we could collect these points and eventually make a new doc on measuring performance? (For now, whatever you'd like to say about measuring performance can stay in this doc, we can work out organizational details at some other point.)
I double checked and you are correct, sorry for the inconvenience.
Great idea...  I'd change the name though.  "help" might not play well.  For example in the debugger help is actually a keyword.  Not sure if this would have any issues.  

namespace :admin do
   get "help", :to => "help#index"  
end
should say chunkHeaderSize ?
yeah, sounds better :)
I'll change it in the morning.
This doesn't work in a 256color terminal- bug or expected behaviour?
Why is it not generated on Windows?
Hardcoded path might give problems. This can either be changed to relative path or user can be given config options at the time of installation.
I think at some point we'll have to organize Levels into Courses, since one rarely plays a single hole of golf.
yup - the name is off. This should be fixed.
прикольный коммент к данному коммиту :)
insya Allah bisa direalisasikan... :)
The indentation seems off here - are you mixing tabs and spaces? Also, don't you need to import urlparse? something like from urlparse import urlparse (And then you don't need to do urlparse.urlparse)
I'd use FILTER_UNSAFE_RAW for passwords because we don't want to strip any characters from a user's password.
Faut-il qu'on commence à préparer un rapport sur ce qu'on fait ? (Pour la présentation du projet que nous aurons à faire)
I dont think this is superior to the other approach, I think its more complex and harder to read for other programmers. This approach requires a ton of sirch and breaks, wheras the other approach is implicit and contained. var next_step = step + 1; which step is step #4 again? see what I mean. I am sorry I just dont think this code helps.
Aquí se han colado varios espacios en blanco. Pásale un ant format-source antes de enviármelo.
![MEME FIGHT, GETTING INSULTED  WIN LEVEL: EPIC](http://i.qkme.me/3647vn.jpg)
Honestly I'm not sure currently how you would upgrade, and I'm unhappy
about this :(
Just curious, why did you add the 'mysql2' gem to your Gemfile? The 'mysql' gem is fine enough. Did you have a particular need for 'mysql2' ?
Аще цимес в том, що brd на некоторых бордах может содержать '/'
Have u tried OC'ing to 1.7 Ghz?I m gonna give it a shot :-
what does this line mean exactly?
found this hard-coded location of "target"
I agree with your suggested change and I updated. Please see this [commit](https://github.com/snkinard/steam-condenser-java/commit/195d21eb0b7d5bbcf18dce011de94b864de41f3c)
The driver in official P500 kernel does not looks on .metods. It uses interrupts in any way.
I wrote in my commit message, that some attributes are experimental. 

Fire up your Safari installation, search for something and the magic begins … ;-) 

(Autosave takes care of the elements in the last search list, so that they are not forgotten when the page is being refreshed.)
why enable a non existing module ?
When i look at the code in 1.6, seems to me like it was a a bug, AddressSupport was intended to be instantiated as a singleton, but the way the code was originally written, it had a public constructor and a getInstance() method which is typically contradictory to the singleton approach. the getInstance method also throw an exception when the singleton is null instead of creating it.
This is an update that ports over the differences between the Firefox 4 theme to the Firefox 10 theme. 

Hopefully it's mostly bug-free. There were a lot of changes to be merged in, so there's probably something a little odd. 

Please test, and we'll merge in this branch when we fix whatever errors there may be.
Thanks man! I love what you're doing :)
/usr/lib/i386-linux-gnu
Got switched up here: http://goo.gl/2mZ4T
On 01/15/2012 10:37 AM, HowUpDoHighKnee wrote:
> Sorry, I'll correct this with my next commit
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nightingale-media-player/nightingale-website/commit/6e2ae6db2fe3eae212691d4b69900fb8756070a9#commitcomment-868939
No worries - we can just rewrite the paths if/when we upload it...just 
keep doing what you're doing, you're doing a good job :)
@MichaelCastelbuono Yeah, I think you are right. `->filterByStartDate( date("Y-m-d"), Criteria::LESS_EQUAL )` is really saying "filter on date(Y-m-d) less than or equal to start date" which does seem awkward.
Clayton always harangues me for this (and I know this is my code so sorry for critiquing it).  Unless ssh_keys is used elsewhere in the function this should be result = rest_client.sshkeys.inject('') do ...
По смыслу здесь проверка отложенных ссылок только в одну сторону (зависимости) или в обе?
Кажется, достаточно проверять в одну.
Thanks for the additional context.  Switching over the existing stuff would be a pretty big undertaking, but I certainly see how it could be of benefit in terms of simplicity.  I think this may be a good testbed for the time being and then once we get this merged and off the ground we can revisit it once it has had a little time in the wild to work out any remaining kinks.
Testing factor replicates stuff on local machine with library(gtools) commented out, get error message:
```Error in as.character(con) : 'con' is missing```
Does not happen with library(gtools) uncommented -- so we DO need it! Not totally sure for which bit though. Will investigate it later...
If these are constants, why not have them be all upper case (e.g. `REGEX_RGB`)?

I'd also use `REGEX` instead of `REXP`.
I assume "wikimarkup" no longer applies here.
Den e najs
To check that there is a member in the node ring with the status Exiting. Is that not clear?
Is it only `!clusterView.isLeader` that's problematic after shutdown? We're checking `clusterView.status == Removed`  right after the `!cluster.isRunning` check.
Some[Double] instead?
What if members is empty?
think EndpointException is not a fatal exception so you don't need the
> "_: EndpointException"

The idea was to catch the exceptions that might slipped out that were not  
wrapped in EndpointException, but match the NonFatal pattern. Currently  
NonFatal and EndpointException is treated the same way, so these lines are  
surely redundant. I will remove it as you suggested.
Needs explanatory comment
What's your thinking on that?  Should we make a branch for that?
missing a '.' before UploadManager.
Something feels wrong here. The return value is set to None, which more than anything indicates a plugin writer error. It should always return a report, we just have code in there to use unknown for the summary/details and log a warning. I'm not sure how you're getting cancelled as the result, but purely looking at the test that's incorrect and misleading. I'd rather call it an error if the plugin didn't return a report than cancelled.
hrmm hidden variable setting.  don't feel pressured to fit it all on one line :)
not in this branch...
Nice catch ;) Kind of stupid that it's using private members, it should returns checksum directly instead of assigning it to a private member.
Added to my planbox. This will be done after this pull request as this is not critical. I really want to merge this pull request RFN and concentrate on testing SCN fail over which has been delayed way too much to my taste.
if you think we don't need a cache reset, i will remove this .. in my tests this has no negative effects
It's not working on Win32.
Seems that compiler can't distinguish between string and string&.
These could be helpful for you:

- <http://hg.python.org/releasing/2.7.3/file/7bb96963d067/Python/import.c#l785>
- <http://nedbatchelder.com/blog/200804/the_structure_of_pyc_files.html>
With a correctly migrated database, please run:

rake db:schema:dump annotate:models

then check in the modified files
What would the type of that be? Would it just have a filter for `Procedure`?
Require was removed.
Prefer `\A` over `^`.
This part should be DRYed out via a decorator.
Slightly better:

```javascript
/markdown\s*=\s*('|")?1\1/.test(atts)
```
`const string &conf_val = iter->second.value;`
will avoid a copy of the string object, here.
:metal: Thanks for removing these!
Because adding a rounded rectangle mask to a text field doesn't "fit in" on Mac OS X. On iOS it'd blend in fine, but Cocoa, by default doesn't have masks.
No space between `^` and the brace.
Please remove the debug lines.
Yes.  V8 is happiest when the `arguments` object is only manipulated in clearly low-impact ways, like reading properties and length.

Setting properties or accessing caller/callee kicks it out of Awesome Mode.
This seems to be the only use of askMaster instead of askMasterWithRetry. Should we just do withRetry every time? TBH I'm not sure Akka could even fail on one try and then succeed on the next one but I guess it's worth it to retry.
The else should be on the same line as }
zend_llist doesn't have any api to remove more than one element like HashTable's apply. so I simply do it my self (original macro DEL_LLIST_ELEMENT(current, l)). if there is a easy way please tell me.thanks.
в скобки
I think in Rails we don't indent the case like this, but instead we simply do:

    normalize = case fragment
      when Regexp
        ...
    end

Could you please adjust?
You should be able to use `assert_nil`.
I had to add this explicitly because many of the AR tests were failing without it. I believe some of the associations are passing in their contents. Will look into it further as I would like to see many of the calls to IdentityMap.remove removed!
I think we should not `send`. Why not set the type as was did in `becames`?
Can you change this to not use mocks?  I'd rather we don't depend on internal implementation.  Something like this:

```ruby
cache_miss = false
assert_equal 3, @cache.fetch('foo') { |key| cache_miss = true; key.length }
assert cache_miss

cache_miss = false
assert_equal 3, @cache.fetch('foo') { |key| cache_miss = true; key.length }
assert !cache_miss
```
As a general rule I find type annotations with refinements offensive non-DRY, but given that this is an implicit definition, I have to agree.
Is this the desired behavior? Should we simply adjust the docs?
Yes.  Forgot to move it back.  Tried using logpdf to try out the scale keyword.
Good point. I prefer FOO *foo for purely historic reasons (based on K&R and many many years of use), but I know most people prefer FOO* foo (Google style guide allows both), so I try to stick to the latter. I'm so used to the former I often just don't notice I used it.

I've fixed my code to use the latter.
I suspect you copied this from the Mul code above. Mul and Pow are structured differently. 

Mul.args contains all of the elements of a product i.e. ```w*x*y*z``` all of w,x,y,z are in args. The Mul handler will need to ask about all of them. 

Pow.args contains only two elements, the base and the exponent. You can access these using the .base and .exp fields. 

Also, if you wanted another test case the following also fails 
```
In [3]: ask(Q.integer(x**4), Q.imaginary(x))
Out[3]: False
```

This should be None because there are cases when it's true (such as when ```x==i```) and cases when it's false (such as when ```x==sqrt(2)*i)```)
See discussion comments on this. Everything is handled immutably.
Why did you indent this? We often leave this flush left.
Add the same test for rewrite of `erf` in terms of `erfc`.
Not exactly common knowledge. I'll see what I can do.
isn't the grouping wrong for the ? in `\\(([^)]*)\\)?` i.e. doesn't the ? just apply to the final ), and not the whole parenthesized section?
Is this check expensive?
also, chmod is not affected with umask. so there are no side effects
Ah, and as mentioned in #927, the pecl installer doesn't write to STDERR. Ever.
Should be tab indented.
You can get rid of the type definition here, i.e. remove ": SerDe".
that helped me figure the visitor that was missing
I know, but as this is only true for closures used in the service container, I prefer to be consistent and always inject the app via a use statement.
so should I put a hold on my default thing?
(well I have to anyways, cuz functional tests are kicking my butt)
megabytes?
we should wrap both strings in {{ _('We\'re taking snippets ...') }} for l10n.
Again, "before". Or "prepend" as used above.
I think this is somekind of misleading.
We introduce soft-delete feature for post, which should use this feature in that way.

This is about content handling, not user handling
Fix grammar in this sentence.
I don't see why this would be a good idea.
Well yeah i checked it with a couple of extensions enabled. Output was fine.
Pass $db as an option instead
hehe, I copypasta the comment from an previous function, will correct :)
seems like we should process this flag absolutely last?
Looks good to me, did you test it?
tag scriptesh koo ?!
по сумме 2х? т.е. ты хочешь в базе еще и держать запись с суммой показов за прошлые дни?
тогда получается в твоем варианте дневная статистика обновляется в реальном времени, а общая статистика берется суммой текущей и прошлой.
в моем случае в реальном времени будет обновляться суммарная статистика, а дневная считать уже потом.

вообще, зачем нам обновление дневной статистики в реальном времени? крону все равно для проверки лимита так и так нужна только суммарная. а если в админке нужно будет показать статистику за текущий день то там же ее и посчитать легко можно будет
There is no variable by that name in this scope
as I understand this means that currently min api level is defined by native ocv sdk part.
Just do the following. AppSettings.Browser is already a BrowserProduct.
BrowserProduct browserProduct = AppSettings.Browser;
Remove leading space before `adding`.
Using statement should be placed inside the namespace declaration
Ideally would use store + model for this in MVC, so later we can sync offline/online easier.
This is the same field every time?
See pp.py for a suitable docstring.
any reason for 0xFFFF here instead of say, 1 ?
That declaration hides the one on line 2793.
Setting sqlHash to itself if non-null to avoid a conditional seems too clever by half.
this comment might be good, just remove "And" part
The file loaders will absolutely need to know things about Cyclus core!  At some point they need to report what they have read to something. The reverse dependency is important to avoid - the core should not need to know anything about the file loaders.
that adds overhead. we've already grabbed it with the iterator. any reason you want to use that instead?
goclus - I'm telling you.  If you don't think it will be better - I'll take your word for it.  It seems like there are a lot of not-as-helpful-as-you-want methods in IsoVector then.
Fixed in b93208ee
This seems to be using Proc
We use >= here for fixing MADLIB-449: AR: We choose rules which is larger than minimum support instead of larger and equal than minimun support.
Do you mean RPM package by "the other deployment targets"?  The difference from it is that PGXN package needs almost raw source tree.  I'm not sure about "install" process in cmake, but as I created "extension-install" target for EXTENSION, cmake install process is not applicable here, I think.
Oh snap. I'll fix the leakage. 
I was kinda waiting for someone to call me on style :sailboat:
It's only for a test of `dict` returns from plain object functions works, ref. the discussion around assuming that returned values with a `get()` method is futures.
Well, `fjord.search.index` shouldn't be importing anything from fjord, and if I follow that rule, it works fine. But `register_live_indexing` needs to have a reference to `_live_index_handler`, which needs to have a reference to `unindex_item_task`.

`unindex_item_task` should live in `tasks.py`, and I think that `register_live_indexing` should be in `index.py`, but then it can't get a reference to `unindex_item_task` like it needs.
Oops. These two lines are remainders from when there were views for the 404 and 500 pages. After that I learned that Django has some built in views, and that only the templates need to be made.
Good catch, thanks.  I think the node ip address and localhost makes sense given the reluctance to change /private/etc/hosts.
Can we extract the JS out into its own file. 

Would be nice to have it wrapped in a `(function(jQuery) {})(jQuery)`, so that we aren’t adding everything to the global namespace.
I would love to use `prepend` from Ruby 2.0 instead ...
Since we have to use Manager.calls_manager for both riak and redis I guess this needs to be factored out into a common module so that it's obvious it can be used for both? Doesn't this look for .manager though? Hmm.
i think you must specify the class name too for the ``:js:func:`` stuff to work
The implementation follows the distributed session proposal.
Fair.  Done.
Is there a reason why `!=` is better than `!==`?
Yeah I figured that much when I committed it, but was in a hurry last night :)  I was more concerned about it actually working at that point.
I was thinking we'd upgrade translateKeyboardEvent() to return "Enter" and "Esc[ape]" for these cases.

Ideally, all "non-printable" keys should be mapped to friendly names by translateKeyboardEvent() or else we won't be able to register shortcuts for them (I guess you _could_ register shortcuts like "Ctrl+\n" or something, but it feels wrong).  We could start by just adding support for specific keys as needed (like Enter and Esc in this case).

But in the interest of getting this patch finally landed, let's just do a TODO her for now if that works for you?
I think this is the only string we need to translate, but I'm ok to leave it as is for now.
Typo? Should be “Доступна”
Oops, yeah... I forgot to comment on that. The previous logic was wrong and I wanted to check with you the best approach to take here.

Now that the panels have `min-size`, the only way for minSize to be chosen here is if we change the minimum size for the panel to a bigger value than the one stored in the preferences in the previous session. With that in mind, we can do two things:

1. Don't check for minSize. This shows the panel with the stored sizes and the moment you start resizing it jumps to the new minimum size directly.
2. Check for minSize (like previously), which should automatically resize the panel to the minSize.

There were a couple of issues with 2 before:

* If you resize the panel to the minimum size (in which case `size==minSize`), `contentSize` is always smaller than `minSize` and we are forcing always a bigger size on it (the scrollbars aren't quite right).
* If you raise minSize, the panel is resized correctly, but again the content isn't and it is always getting resized to minSize. We'd probably need to set it to `minSize-(size-contentSize)`.
* We may want to save the preferences in this case, although the code won't complain.


Which behavior do you prefer? If it's the second, I could push it along with the rest of review changes.
see response above
I sent you a sample page that does not agree with that theory :)

Also, any element can be made block-level (display:block) or inline (display:inline) using CSS, so I think this needs to be done for any element that is not position:fixed.
Yeah. Some of this is because I'm new to rescue. If they work as local, let's keep them local, and work on improving the others. :)

Thanks for paying attention to the style, though.
Format this plz
you could just put these styles on #result instead of adding a countdown class
This is probably not needed? Why have you added it? Are we doing POSTs in JavaScript anywhere now?
agree, much better also because now i had option to keep same distance between words (also unlinked). Thanks
Be careful. I just renamed `facebook_tags` into `facebook`.
does this listener ever get removed?
Why two var lists? If one is a set of CONSTS then put them before the Dialog.register.
`plrojectName`?
The name value doesn't give me much indication of what this is? Maybe percent instead?
could you add some info to this exception message so we know what tier couldn't be found? That way we'll see it immediately in sentry
Not your fault, but this comma here makes no sense.
Perhaps having a ``self.use_ssl`` boolean would make more sense? This way it could get set as soon as the pool is created and we won't need to inspect the url later.
Duplicated code?
https://github.com/sonatype/nexus/pull/582/files#L1R342
Add a tag reference here
Can you fix the indentation to be 4 spaces here? Also, use more generic names such sourceStreamFiles and destStreamFiles instead of local and merged here.
This is a lift of the type-based pattern-match on a sum type you might find in Haskell, Scala, or ML to the function dispatch level. I approve! I think I need to start a new wiki page for code patterns and include both this and Jeff's elegant `eltype()` implementation, both of which really show how to get things done with multiple dispatch.
Yes, this is not a great long term strategy. It would be good to find a way to learn the true value of `L_tmpnam`.

But, for now, how about being a little wasteful and using 4096? Based on this list from Wikipedia (http://en.wikipedia.org/wiki/Comparison_of_file_systems), no existing filename should be longer than 4096. While this doesn't apply to full paths, it's at least a little safer at the expense of 4x as much memory.
The window target hardcoded here breaks `GAIA_DOMAIN` parameter for `make install-gaia`.

Just use `*` here, I don't think we specify the domain will make us safer in any way.
On 12/04/2012 07:33, James Lal wrote:
>> +    launch: function(){
>> +      var event = document.createEvent('CustomEvent');
>> +      event.initCustomEvent('mozChromeEvent', true, true, this.getDetails());
>> +
>> +      window.dispatchEvent(event);
>> +    }
>> +  };
>> +
>> +  function MozAppGetAll(){
>> +    setTimeout(this._loadManifest.bind(this), 0);
>> +  }
>> +
>> +  MozAppGetAll.prototype = {
>> +
>> +    //relative to homescreen
>> +    appManifestFallback: './apps-manifest-fallback.json',
> I am not worried about performance I am worried about the maintenance of the webapi.js file.
> There will be one extra .json file in homescreen that will include all the manifests for gaia.
> Its not code and its auto-generated I don't feel like it adds overhead.
>
> That said if you feel strongly about keeping everything in webapi.js I can include it there
> but I don't feel good about mixing the auto-generated data with the hand maintained code.

right. so what about creating a compatbility folder and put webapi.js 
and your generated file there?
This way the code will be in 'one' place while there will be no mix of 
manually written and autogenerated code.
Yes, we're using the marionette client that lives in the source code of Mozilla-Central (http://mxr.mozilla.org/mozilla-central/source/testing/marionette/client)
Perhaps a wild suggestion, since I'm not terribly familiar with IPAC tables, but maybe a short example would be helpful here to explain when one would use `definition='left'` etc.
you're right about that, now that I go back and look at it... the trouble is that this is a sphinx extension that isn't terribly adjustable.  There might be a way to play with the graphviz properties to make it work (http://sphinx-doc.org/ext/inheritance.html), but I'm not sure off the top of my head what to fiddle with.  Perhaps you can add an issue to address this, @cdeil ?  It will take some changes in the sphinx extensions to get it to work (probably), so I think it's better to do it after merging this PR.
原来是写的的merge方法，都一起合了，但发现比如加置顶，补信息什么的并不想影响到totalcount，不然很麻烦，所以写了mergeIds，以后可以再写merge
换掉var_dump把
恩，那还是直接放在前面吧。
fair point, changed it to *bookmark_for (video)*

*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 Sun, Dec 2, 2012 at 2:10 PM, zsherman <notifications@github.com> wrote:

> In app/models/user.rb:
>
> > @@ -34,5 +41,15 @@ def self.from_auth(auth)
> >      end
> >    end
> >
> > +  def bookmarks
> > +    #returns a user's bookmark List
> > +    lists.where(name: "Bookmarks").first
> > +  end
> > +
> > +  def bookmark (video)
> > +    #returns a user's bookmark Listing for a video (if it exists)
> > +    bookmarks.listings.find_by_video_id(video)
> > +  end
>
> +1
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bitcast/bitcast/pull/22/files#r2287984>.
>
>
even if it is TMI, we don't leave scattered bits of dead code all over the place
As the windows gitian builds use 1.47, I guess we can safely ignore the case of pre-v3 boost fs on windows. Updated to use .string().c_str() everywhere.
It's a mess! sorry, b75a6b440abfbaf7af3ef82dd19eb4aff4be5847 is an extract
Sorry I don't understand. Both `gridByType` and `currentType` are object literals so talking about "first" sounds wrong to me.
Minor code duplication.
Either extract  [ self setWeekday: 1 ]; before 'if' statement, or create separate functions without a boolean flag
*I just followed the legacy code above.
*
On Thu, Aug 16, 2012 at 5:33 PM, Sven <notifications@github.com> wrote:

> In emesene/emesene.py:
>
> > @@ -172,6 +178,10 @@ def _setup(self):
> >              extension.register('session', xmpp.Session)
> >          extension.register('session', dummy.Session)
> >
> > +        if webqq is not None:
> > +            extension.register('session', webqq.Session)
> > +        extension.register('session', dummy.Session)
>
> You're registering dummy twice
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/emesene/emesene/pull/1460/files#r1390952>.
>
>



-- 

*Best Regards~*


*Xiang Wang*

*CDC, TrendMicro*
Should probably link to the call you need to call first. At least mention it by name, since it doesn't seem to be MailHeaders.
Was terrible!  1ms was a proof of concept that I had never addressed again - switching to 50ms dropped CPU from hovering around 10% to negligable.
Can we name this file `parallel_magics.ipynb` to match the naming convention of the other example notebooks?
This changes the behavior slightly, as before we'd guarantee the exception's `args` and `kw` attributes would be as passed to `__call__`, but now we only take the results from the raised exception (which likely have `args = ()`, and `kw = {}`).

Is the ability of the exception to change the calling arguments ever actually used?  I can't seem to find an instance of this in the IPython source.
I'd say we shouldn't use DeprecationWarning for that reason. The idea is that developers can turn on a debug flag and see where their code needs to be updated, but users aren't troubled by it. But in this case, we want the user to see it.
I don't think the TemporaryUrlKey annotation is required for RestAnnotationProcessor
My bad. Good catches. Sorry, bad copy'n'paste, there.
'Bahagian' or 'Medan', 'Bahagian' seem to be more appropriate to section in english.
Just verified that we have permission.
>@nulltoken already states that this part has been run successfully.

I indeed assume this. `Checkout` is an operation of its own, after all ;-)
works here with gcc. But i will add the include.
I think I'd have to disagree. According to the spec, “the q element represents some phrasing content quoted from another source”. As this is not a quotation (that I'm aware of), marking it up as such would be inappropriate.
You should really wrap these lines. Also, "when you want to start contributing" is better.
sounds good.  yeah, I'm going to have stephen donner's team write some selenium tests for these things.  thanks!
Not needed. And probably never will. However, leaving it in make us far-future proof.
I fixed #1, won't fix #2 for reasons you wrote. Also, given http://initd.org/psycopg/docs/cursor.html#cursor.close I think explicitly closing the cursor is not necessary and can be removed, which is what I did.
I don't like this. It's ambiguous. Within the code you can always reach the same with `self.config.logger...` and a code-reader might wonder (outside the scope of the `__init__` method): "What's the difference between 'self.logger' and 'self.config.logger'?"
For my understanding: Are all of the aspects of
* faithfulness
* plain-ness
* exactness
completely independent from each other, and can thus be combined arbitrarily?
missing in the new pull-request:
javascript:
This should be lower case (isUnassigned)
well, with the old code for the loop, it was impossible as this case would be an infinite loop, thus never reaching this line :)
No, I forgot to remove the verbose. :P
Ah, you're right, I misread it.
why putting a default value here ? The constructor always erases it
`Symfony/Component/Console/Output/StreamOutput` used these for method `hasColorSupport()` which I tried to model when adding this. But I'll remove them.
it should be ``{@inheritdoc}`` as this is already documented in the interface
Should be: `null === $options`
Yes, it was on purpose but I looked for it in more detail and I realized it not necessarily written that way... so I reverted those changes to be more consistent. Thanks!
You're right, fixed now. Thanks.
At the time you asked, no. With the changes below with the custom error handler it should work as expected.
yeah, I'm not happy with this solution either. Will remove it.
Seems like image should be the `Reminder#senders`rather than the event owner's image, given the copy which follows.
not nul
Oh.  That's a holdover from before I cleaned up matching on add/remove a bit (G and K didn't need to do anything on remove, so it didn't get anything applied to it.)
I'll be nice and fix it on g:lines and k:lines as well.
I guess the text should read 
"The cloned rule will be created disabled", or something like that; this first wording got me confused
Isn't there an "it" missing after remove ?
It seems to me that a more foolproof way to do this is

    ::strncpy (this->creator, creat, sizeof(this->creator));

Then nobody has to look at this line and wonder, "is that the right size, or not?"
I can't see too much benefit from the conditional import here (it doesn't import matplotlib does it?).
Would you mind commenting in the docstring that there is some caching going on?
nitpick: could you remove the parenthesis? It makes it look too much like a tuple to me
Note:  Logging in the server is a little broken right now.  I commented this out for now and will fix later.
Why is this a property if it is initialized in __init__?
Good point.  I'll comments demarcating it.
Please leave spaces around the `==`.
These files had to be ignored on Unity 3.4.
On Unity 3.5 they were moved elsewhere so I does not mater ( although it pollutes )
Ok I haved aligned the exceptions - they now meet my current requirements. We'll add more later.
I don't think so. KEYSTORE_TYPE is a simple string like JKS, PKCS, etc. Besides, the validator and model description both indicate this attribute is nillable, not that it should allow expressions.
This one should have a confirmation as well.
Damn, I suck at proofreading. :)
why don't we have here a module as we have in other apps?
Mozilla is the first place I've experienced people asking for less comments in code! :P
but don't leave the anonymous function ;)
revise, use `getAllContactsCachedSync`
no `kb` prefix needed
`dbData`?
Or, switch back to using actual img elements instead of divs with background images. I'm not sure what benefit you get from using background-image on those, since the aspect ratio is the same, so you don't need to do any cropping.
Just a suggestion, it would be better to remove the element selected by class. This might deleting the wrong element if the list item structure changed.
That make sense.
Remove this line from `bootstrap.js` if you move it here.
hmm.. why would we need this? :)
There's a fix for this on L47.
Why not use getElementByID('sendRequest') or querySelector('#sendRequest')?
Same as above, don't test for the exact message, just if it throws.
Why is there a fat arrow?
You are right, my mistake. Thank you.
check the marker and find the unset - it's one of the ugliest things - we should automate it with a function ... but I was always to afraid to break something
This seems important enough to escalate to an issue on GitHub, so add it there.
Consider raising a `NotImplementedError` since this doesn't do anything, or at least mention in the comments why this does not thing and what your plans are for it.
Describe this better.
it must  be:

var myOptions = {
			zoom: $map_zoom,
			center: homeLatlng,
			mapTypeId: google.maps.MapTypeId.$map_type
		};
Maybe ARCHIVE_CONTENT_TYPE just to be unambiguous.
tInitialTextにしてもらえますでしょうか？
You can just do `if not association_data` for the same check.
lemme count:
1 array index (if mapCount wasnt properly set)
2 dereference (array entry was null)
3 invoke readLock (anything in that method could throw)
4 dereference (readLock returned null)
5 invoke lock (anything in that method could throw)
what does isSequential=true mean?
This is for the tests. I'll reorganzie the commits when it is a bit more polished.
The syntax should be the same, between each protocol. And this is what handles the parsing of the messages.
I copied the trace layer and removed everything but the eye nose and mouth paths.
can you revert the end of file modification? It's to avoid future merging of branchs
mode is undocumented
also, register both extensions in init() with ServerInstance->Modules->AddService(...);
There's an extra comma here that breaks CSS validation and may have side-effects in some browsers. Otherwise I think this is good to merge after that!
If you want, you can do that, but then I think we lose the ability to determine if its a dev version? Have to double check how we numbered the dev versions.
Yep. You must add catid.
Oh so the original hack is to get a reference from a JSM!
What about just doing this in api-utils/utils/data ?
  `const { atob, btoa } = Cu.import("resource://gre/modules/Services.jsm");`
Services.jsm is here to stay and is already loaded by Firefox.
And its implementation is really simple and won't mess with these methods:
  http://mxr.mozilla.org/mozilla-central/source/toolkit/content/Services.jsm

Do we have a platform bug opened in order to expose them in all sandboxes?
It sounds like a really simple feature!!
Ahh sorry I just have not seen it in this request and assumed it was not used.
Yes I remember but I also came to realize that than you carry XPCOM like API all over the place, for example I never ended up landing custom-protocol  because after our decision, I was not able to make API nice anymore. I can remove `id`, `contract` `register` and `unregistered`  from options before passing it to Class so that they don't get included into `Component`.
Can we use `cfx-js` in all other places instead of `cfx.js` then ?
Agreed, UNCLE is silly.
didn't touch this line, just copied it from Criterion
Agreed. The whole spec file needs a bit of work. I was mostly concerned with the shortest path to unblock our certificate management work. :)
Not clear to me either - so don't know what to actually test. Suggestions?
The real reason it's here is that we used to have pooling, not that we plan on adding it.