labofsahil/github-event-dataset-2012 huggingface dataset:
Review: fix Gabor noise numeric instabilities.

Squash numerical instabilities that could lead to inf values in Gabor -- basically by turning off filtering for very tiny filter values (which should not be visible, since filtering does very little when derivs are small, and even speeds things up by not filtering needlessly), and when the determinant of a troublesome matrix is too small (other things will be divided by it later). 

In the process of debugging this, I realized that I never implemented operator<< for the Matrix22 class, so that's mixed in here as well (even though it's not actually used in the code now that I've removed the debug scaffolding).
Test Fixes

portlet/assetpublisher
LPS-25036

LPS-25036
Added to students

added myself to students
GPDATABASEMIGRATION-102

http://jira.grails.org/browse/GPDATABASEMIGRATION-102
INT-2446 - Twitter: Allow unauthorized operations

* also improve test code coverage
6.x

Changes to allow wildcarding is advanced searches
Int 2351 maint
Int 2400backport

* MessageChannel was moved from core to the base package.
* ChannelResolver moved from core to support.
* TcpConnectionInterceptorFactoryChain was incorrect.

Backport to 2.0.x
Adding more UT coverage.
Fixing README.md for Service Bus
LPS-24660 Remove table usage to allow easier styling in themes.
Exporta3d

Example on exporting the current scene to an .a3d file
Updated Spanish comments to English in XmlRpcConnections.js

Thanks for doing this.  I am trying to use this framework from a mobile app to talk to WordPress.  Anyway, while reading the code to try to understand how to use it to post a comment, I saw the Spanish comments and converted them to English to help myself understand using Google Translate.  Anyway, I thought it fitting to share back, even though it's just a small thing.

Thanks again for your work.
Rename sqlite3 gem

Removes the post-install message from Bundler:

```
#######################################################

Hello! The sqlite3-ruby gem has changed it's name to just sqlite3.  Rather than
installing `sqlite3-ruby`, you should install `sqlite3`.  Please update your
dependencies accordingly.

Thanks from the Ruby sqlite3 team!

<3 <3 <3 <3
#######################################################
```

Cheers,
  Lee :beers:
Functions done'd

Kaikki paitsi viimeinen tehtävä tehtynä.
User.js  and TeamTest 

@ymanyakina : I wrote a TeamTest code and edited User.js Please review the code.
Weird search result for guests

When you are not logged in, you visit activitypage.php and type anything in the searchbox and submit it, it will redirect you to `socialnet/activitypage.php?mode=search` and content of this page is:
> {"user_id":"ANONYMOUS","more":false,"onlineCount":0}

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4649
Zoomable site for mobile devices

When chat is disabled for lower resolution, than we could allow zoomable site.

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4490
Implemented "delete" and "edit"
Rewrite of Annotation parser

This is the rewrite of the Annotation parsing code.

* Rewrited lexer and parser almost from scratch
* The new parser is designed to parse any annotations that is possible to write within Symfony2
* Added an annotation model which represents the annotation and its sub-annotations
* Changed and added tests for AnnotationParser and AnnotationCommentParser
* Added an AnnotationCommentParser responsible to parse annotations from a comment
* Ported code to use the new AnnotationCommentParser
* Tried hard to put comment parsing stuff in AnnotationUtils

Tests are passing except one which is also failling in the master branch. I tested what I could (highlighting, report unresolvable annotation, template view extraction). However, for some features (like the F3 for template), I was not able to test it until the end of the chain due to what I suspect are bugs.

Moreover, the last missing bit is the error reporter which is not appended to the parser for now. I think it is not that bad for now since the code for reporting the error in the concrete IErrorReporter is commented anyway.

I will send a PR to the Doctrine-Eclipse-Plugin since it depends of code from this PR.

This PR will fix gh-50. Don't hesitate to ask me if you want to change some stuff inside the PR or if you want me to squash the commits.

Regards,
Matt
Useragent

Phonegep now identifies itself via the user agent. This will help greatly with traffic analytics.
Update README.md

include linux instructions
Фикс для компилирования

Теперь собственно всё. Единственный варнинг о Blob, то что он не принимает 2 аргумента - не обновили ещё компилятор в соответствии с последним драфтом спеков.
Fixing upstream URL in README.md

The upstream git URL was wrong in the README file.
String lengths, extension functions and Select modification

GetUndefinedColumnDefinition now honours the StringLength when creating a field for a serializable blob.
First and FirstOrDefault extensions now support predicates.
The Select check for full select statement now ignores white-space at the start of the string.
Windows fix

Fixes issues #183 and #167
Linux setup updates

Hello,

I made some minor changes to the getting started guide for Linux. Copy-pasting from the doc to a udev rules file should work now. I also simplified a couple lines to use the auto-generated symlink to get to the beaglebone serial port.

Frank
addressing rt #77264 and # 77265

Hi,

I created some patches and tests for
https://rt.cpan.org/Ticket/Display.html?id=77264
and
https://rt.cpan.org/Ticket/Display.html?id=77265
some month ago. I created a pull request for zby's repo, first, but now i was told that this is the right place.

None of the existig tests fail, but additional review might be required, since I don't understand everything whats going on behind the scenes of RU.

cheers, Lukas
added channel and channelsettings
Branches/symfony2.1

Hi i just updated my fork to be compat with symfony 2.1 
no deps file needed anymore, composer does the work! Security Factory loading is done slightly different in symfony/master
This should be considered as Answer to Issue #7
Fixed issue with artwork
Updated German Translation

Hi,

I've updated the german translation again.
Master

Added add message forms, with hex error checking
Neue Pakete und Verbesserungen im Text(satz)

Mit den Commits habe ich nochmal versucht, einige Änderungen einzubauen. Dank eines besseren Editors sollten diesmal auch die Umlaute erhalten geblieben sein. Der letzte Commit (e60927d46cecc2db8b8e21159feee5b220f021d0) bringt einen Kommentarblock im TeX-File mit einigen Anmerkungen und Fragen mit. Es wäre schön, wenn die kommentiert würden.
Make Generation Slightly More Extensible
Query#save takes a user object (to save user.id)
Permit custom macro definitions in the commentPreview module.

I added an options setting to the comment preview module that allows the user to define additional shortcut buttons.
Initial HTML5 App and Android Project

## Branch Master
The UX folder holds the main HTML5 application. The app I have create codenamed ("littleBlueRocket" after the the spaceship in the logo) contains the main logic, images/icons, fonts etc. To use it you need to copy the contents of ./ux/littleBlueRocket into ./assets/www folder in your phonegap project.
## Android_master
The android phonegap project folder set up for eclipse. The UX stuff is included.
## The code
All the code is commented so take a look at index.html, theme.js and theme.css to see what has been set up. I'm currently using Cordova 1.9 and jQuery 1.7.2. The ui is custom made since I started before the design issues had began, Changes will be made later.
Add Portuguese (Brazil) translation

Hey, I translated this applet for my native language. Would you please add to your repo and repackage it? Thanks!
Update patch-5

Added ":" to internationalization functions as in some languages (for example in french) a non breakable space is needed in front of ":" ";" "?" "!".
画像作りました

昼、夜、処刑の画像です。
Fixed reliance on uniqueBy fields containing no null values.
大域変数と定数に関する項目を訳しました。

大域レキシカル変数というものを初めて知ったので、後半はかなり訳が怪しいです。
多分このあたりの話だと思います。
https://groups.google.com/forum/?fromgroups=#!topic/comp.lang.lisp/d5bsSR0o4Ps
Match version-TRIAL entries

Also match change log entries when the version has -TRIAL appended
which can come from the `%V` code of NextRelease.
Admin-able video galleries for pages and blog posts

* Most of this code was ported from [my fork of refinerycms-page-images](https://github.com/phiggins/refinerycms-page-images), so it suffers from the same limitations that project does. This includes:
  * There are no tests for the caption stuff. I have no idea if that works.
  * The tests for the admin UI are very minimal, only testing that things are generally in the right place. It does not, for example, test adding videos to pages, deleting, reordering, etc. Those functions are tested instead at the model level.
  
* A lot of the code in refinerycms-page-images was inconsistent in the usage of 'page_image' vs 'image_page' for various things. I've settled on 'page_videos' in this project, since that follows the usual convention for join table naming of alphabetical ordering of the joined things.

* The following things are missing / do not work:
  * Lots of locale stuff, so a lot of the labels for things are not correct.
Add AutoEnchanter plugin and description

Needs an image :)
Identifikation eingebaut
Solved the problem with the System.Web.Razor couldn't be found.

Added a comment in the web.config:

      <!--<dependentAssembly>
        <assemblyIdentity name="System.Web.Razor" publicKeyToken="31bf3856ad364e35" culture="neutral" />
        <bindingRedirect oldVersion="0.0.0.0-2.0.0.0" newVersion="2.0.0.0" />
      </dependentAssembly>-->
Added prefix to constants and functions

Just added prefix to constants and function names to prevent conflicts between plugins that are using the notifier. Also added a constant to store the codecanyon author username (used for Envato referral program)
This unset is dead code. No need unless you're doing references.
Updated Commit: 391fb3a05a6fe6e1e02ae0f6d849b7260326a358
Okay, just curious - why do you prepend the # character here? I'm pretty dense when it comes to lisp.
In general, I think more investigation on the semantics of mongoDB's find (if it throws an exception at some point) is a good idea, but for now, we only raise the current label (to the collection and db label) when reading from the cursor. (The previous implementation raised the current label when doing a find as well.)
How did you resolve this? I've got the exact same error, but I'm not sure what old code would cause this.
Stable. I don't foresee major changes.
Tohle znemožnuje i přesměrování nově přihlášeného uživatele.
Šlo by to takhle?
<pre>
   if (isset($session[$key]) && ($session[$key][0] === NULL || $this->getUser()->getId() === $session[$key][0])) {
</pre>
This is nice! Great job. Can we use `public_send` rather than `send` to set the attributes though?
this appears as an AttributeError on my machine, not NameError. (Python 2.7.2 Linux x86_64)
Yes, they are actually dict keys. The options object explicitly sets any parser attributes in its __init__.
and what about relog or alt+f4 player on ship ?
T'es bien sur que `lookup.Reached.getPeer(1)` n'a pas déjà la value ?
但你在客户端用的是QUrl。。
Fix spacing at comma
"It should be removed later ;)". Yeah, I forgot about that.
Maybe I should clarify, you had some scoping with the immediate function you used but I just removed all the functions that were attached to the global Contra obj and instead only return addHandler. I figured all the other functions do not need to be exposed within the global Contra obj and can live just being private functions. This prevents users from just typing Contra.win() in the console.
How is "md4.rs" different? See also #1254
why is the `mapping_.size()` before the insert returned (rather than the value after)?
Note that this broke Riak installs.
http://buildbot.springrts.com/builders/full-windows/builds/2454/steps/buildConfig/logs/stdio :
INFO Parsing source file "/slave/full-windows/build/tools/unitsync/unitsync_api.h" ... 
INFO gccxml cmd: /usr/bin/gccxml  -I"." -I"/slave/full-windows/build/rts"   "/slave/full-windows/build/tools/unitsync/unitsync_api.h" -fxml="/tmp/tmpgOPQ6V.xml"
INFO GCCXML version - 0.9
/usr/lib/pymodules/python2.7/pygccxml/parser/scanner.py:337: UserWarning: unable to find out array size from expression ""
  warnings.warn( msg )
Scanning dependencies of target pyunitsync
[ 34%] Building CXX object tools/unitsync/python/CMakeFiles/pyunitsync.dir/unitsync_python_wrapper.cc.obj
/slave/full-windows/build/build/default/tools/unitsync/python/unitsync_python_wrapper.cc:62:33: error: Numeric/arrayobject.h: No such file or directory
make[2]: *** [tools/unitsync/python/CMakeFiles/pyunitsync.dir/unitsync_python_wrapper.cc.obj] Error 1
make[2]: Target `tools/unitsync/python/CMakeFiles/pyunitsync.dir/build' not remade because of errors.
make[1]: *** [tools/unitsync/python/CMakeFiles/pyunitsync.dir/all] Error 2
Being an acronym, I'd prefer you use `ACL`
#\n\t放在同一排的输出效果不一样  且在三引号中的缩进最好只用\t来表示以免产生错误
have you tested it? in the previous code, there wasn't stop before travelling back, but i thought it might be desirable
have a look at Enum type
Never knew this was possible, awesome. Knowledge++
Why is the scheme specified? For future use, or symmetry? I have not yet found where exactly it is used.
Ok for an enum. However I would prefer:
ExitNotRequested = -1,
ExitSuccess = EXIT_SUCCESS,
ExitFailure = EXIT_FAILURE

terminate() would still take an int though ( to leave the option to pass any other value if needed)
006+       ["repositoryformatversion"]=>^M
007+       string(1) "0"^M
008+       ["filemode"]=>^M
009+       string(4) "true"^M
008-       ["filemode"]=>^M
009-       string(4) "true"^M
012-       ["repositoryformatversion"]=>^M
013-       string(1) "0"
wow, there is also an AvailabilityMacros.h, reading...
I think that in dungeonschedueler is a mistake, because we have millions of sql-queries per seconds. 
The Freeze Detection is right.
Thats no real solution, maybe a bad workaround.
Это обязательно нужно делать в блоке finlly
So for L14, I add a boolean has_subtypes:

```python
has_subtypes = models.BooleanField(blank=False)
```

And for L29:

```python
related_name='Skill.subtypes',
```

But, there is still no Skill.subtypes field present, what type of field should it be and what parameters will it need?
Why did you have this?
No. I hate the warlocks who kill you after you kill them :(
JS is indented by four spaces
Add the games to the compatibility page perhaps?
Is it the only point of this modification?

```cpp
if (enable == isFlying() && enable == canFly())
        return false;
Pull request looks good in general (will have a closer look later), but I wonder why you don't respect dependency ranges when checking myParentsModules?
Con le modifiche effettuate nel file class.template.php è stata introdotta la possibilità di modificare i blocchi del template.
Good call.  I pre-allocate a properly-sized seed now and fill it accordingly. Didn't bother declaring DYNAMIC-EXTENT; do you think it's worth it?
evet.  her slayt için (`desc` yapılmamış olduğundan `rake -t` dediğinde göremeyeceğin) bir görev tanımlanıyor, dinamik olarak.  sonra bunlar birleştirip tek bir göreve dönüşecek.  burada parçaları hazırlıyoruz.
ну тут, логика реакции не трогается, в данном случае исключению просто добавляется дополнительный признак http status code, в дополнение к остальным
Hi @pmuir, it's looking good, running tests now. Just one doubt, why remove the getName() from the Configuration instance? even if it's not required, wouldn't it be useful to have?
yeah sorry I should have said this is with windows :)
 I read that 'putenv' might be able to replace it, but couldn't get it to work.
We need the replace the current command line argument evaluation to something similar to what is used in OpenMW anyway. So if you just use boost program options you should be able to copy and paste a good chunk of code.
This is just a whitespace commit, because i found indenting etc to be really inconsistent across the gem. 

Visit https://github.com/imajes/geokit-gem/commit/cd14aa660c415c437493fb80070286ca02a99e63?w=1 to see that there are no substantive code changes here.
Added testcase 2 with a UUID as 'id' instead of the objectID (which is default). This works also in corrected and uncorrected code.
For anyone CC'ed, discussion https://github.com/jnicklas/capybara/issues/619#issuecomment-3579942.
Indeed, but this is used in replace method... I just used the same way...
WHERE AND HOW DO I DO THIS?
@mikeprimm Why do you think the world height variables are gone?

I see them presently hardcoded at nms.World.getHeight() and obc.World.getMaxHeight() and they seem to be there since the 1.2 update.  I'm not following your logic of hardcoding the number also in these classes when you can at least centralize the magic.
Ok nice detective work :)
have you populated the facilitator and participants parts?
Действительно
this fix makes no sense.

c/c++ PEMDAS says multiplication before division so the original would result in a truncated conversion.. Nrmally you want rounding but here, the returned value is a time in ms, that is typically used in timing, so by truncating, you are guaranteed that you get at most T time and never expire before that time.
Ok.. figured this out. Only had to use util.cleanInput.

The misc.toPath sent me on a goose chase.. I didn't need to do this because util.writeToFile takes care of that.
You're right: a save action which is not an *editor* command should set dirty state to false, so public API so no underscore.
I kept wanting to reload `package-build-alist`, so this made it more convenient.
These dependecies is just list of names of **Debian packages**. I just puted this list in single line and added apt-get install at beginning.
What? Your comment says "in 3 hours", I'm in the present :P
Tento blok kodu je zbytecne ukecany.

   @article.comments.new(params[:comment].merge(:user => current_user))

by melo byt postacujici.
とても良いです！ありがとうございます。
作成するプラグインによってテストの粒度に差が出るのは仕方ないかもしれません。
ある程度の数のプラグインとテストが出揃うところを目指すのをまずは優先したいです。

spec_helper.rb で require している environment がまだ無いですね。
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -1,7 +1,7 @@
-require 'environment'
+#require 'environment'
Also, it seems like we can not merge that pull request right now, as there would be conflicts with other commits.
It seems you will need to rebase on top of a more recent version.
Тут еще не хватает данных о том, как ставить фигурные скобки...
foo{
}
или
foo
{
}
А как же эти замечательные функции в одну строку?
Is there a path to the confirm instructions page anywhere now?
Just sent you a mail on LinkedIn!
We really should take care of adding specific heroku stuff to the project.
Don't see any workaround here. Am I missing something?
I'm fine with this, but can you provide more details about why you want this change?
Для mockedAccountService попробуй тоже использовать spy чтобы использовать имеющуюся заглушку. Хотя я не уверен что в этом есть глубокий смысл. Подумай над этим.
Yeah, we can write our own parsing logic for this. Split on capital letters, insert underscores, downcase.
I have made some changes in my `sabrahams-jython-i18n` branch which improve this a little...
Il faut _vraiment_ éviter `pushback`, c'est très mauvais pour la performance. Pourquoi y aurait-il besoin de `pushback` ici ?
Ehhh... Yeah you are right. Sorry. I was panicking :/
It might have been that loop i fixed that somehow cleared the board due to a crash.
Oh? How come?
Note for later: *If it ain't broke, don't fix it* (even more if you don't use it)
cvxopt is still in tree for prefix mpfi could go or could be bumped to 1.5.1 ahead of the main tree.
I am unsure how this will work. How would this element get focus to begin with, purely by keyboard navigation?
1) snake_case, not camelCase for method names.

2) Please define one test_ method for each file, so the test output looks pretty. You'll have to use define_method and a bit of meta-programming to get this done.
Huh, so it is. I missed that in the docs the first time around. I probably won't bother releasing a point version just for that, but I'll fix it for the next release.
I've already integrated your menu into the main development branch. Could you please have a look, If I did everything correctly and if it is equal to this "new" commit?
https://github.com/TrinityCore/TrinityCore/commit/7ce78536069233deff626b70ac914423a23b2492
Oh I see...  the title isn't the title...  this is the <h1> tag....  hmmm   Is it possible to do something like:

helper_method :title, :only => index

I don't think so.  OK maybe this is the best way.  hmmm
The idea of dummy nodes and arcs is nice. Not sure about the term, though. Here is an explanation of the differences between stubs, mocks, dummies and fake objects:

http://martinfowler.com/articles/mocksArentStubs.html#TheDifferenceBetweenMocksAndStubs

I would also think about combining DummaryArcContainer and DummyNodeContainer into something like DummyGraphStructure. You could set it up with n nodes, each node would automatically have a number between 0 ,,, n-1, and then you could have a createArc(int sourceNode, int targetNode) that creates an arc. Might be easier to read & understand.
@BubblegumBalloon Create an issue for it :)
The right way to address this assertion is with self.assertRaises, not with a try/except.

This could also be a separate one, not in the same test as the other assertions below.
If you don't mind, I would rather (instead of wrapping os.path.basename()) add:

```python
if file:
    file = os.path.basename(file)
```
to the top of _print_message(). Wrapper functions usually seem ugly to me.
This seems to be a good approach.  useUnicode.py acted like a runtime hook and we forgot to use it again.
wow ! x30 speed up with brain grids?
The message text clearly says 'warning', which leads me to believe I'd want its output on console.warn, especially  in the case I'm logging console output by levels. Seems very rational and obvious to me. However if core is putting warnings on the console.error channel, even if it seems weird, I guess sticking with precedent is better. So you can just close this one. No problem!
I think this should be fine:
//#if TARGET_IPHONE_SIMULATOR
//    [NSClassFromString(@"WebView") _enableRemoteInspector];
//#endif
And it's often like: casting doesnt work over and over again until I restart server.
Sometimes it works every time, sometimes it works only once... Sigh, I will need to do some deep research into that ;/
do we want to keep this at the widest?
Yep, that's what I documented in issue #2

How about my suggestion there, just handling the IPv4 address at the end of an IPv6 address specially and nulling the last IPv4 byte?
Yeah, and it's nowhere near finished yet. ;-)
nextSuggestionNumber and previousSuggestionNumber two similar. can you make re-factoring here?
恩，extra/package/android/configure.sh 的确还有用，我理解错了。

Android.mk里指定LOCAL_CFLAGS 倒不是为了计算快。
如果我这边不指定的话，会有链接错误，提示undefined reference vlc_entry__xxx。

简单一点，就是 LOCAL_CFLAGS := $(CFLAGS)，只是不知道会不会有什么副作用。
顺便说，我的环境是 MacOS X 10.7.3， NDK r7
The diff here is showing all the dashes turning into bunged up character sequences.
This hook is not intended to be part of the DSL; merely to be called by the testsuite class.
on my end this specific change on my 35 base caused all audio to fail.
Nvm sorry.
Why not use ramzswap instead of this ?

See http://code.google.com/p/compcache/ for reference.
This is problematic. You assume that the clearing the index is done within one second and simply start the indexing process. You should start the indexing in the return callback of clear() instead.
Hmmm, I cannot seem to duplicate the issue now...
attrapé (donc "attrape" sans caractères accentués) plutôt, non ?
Yeah, I was wondering about that. Thanks
On Dec 31, 2011 9:00 PM, "preludedrew" <
reply@reply.github.com>
wrote:

> You really should add this to the devices overlay instead of here.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TwistedUmbrella/android_frameworks_base/commit/d7139752b3be440acfbb0316284867f99568242d#commitcomment-830079
>
No need to insert config params if the defaults are ok.   User should reference the docs for customization as with other plugins.
I just checked the pinmux from Viewsonic's latest patch and it is muxed...here it is:

#ifdef CONFIG_I2C_TEGRA
#if (defined(CONFIG_TEGRA_ODM_VENTANA) || defined(CONFIG_7379Y_V11) || defined(CONFIG_7373C_V20) ||defined(CONFIG_7564C_V10)||defined(CONFIG_7546Y_V10))
static struct tegra_i2c_plat_parms tegra_i2c_platform[] = {
	[0] = {
		.adapter_nr = 0,
		.bus_count = 1,
		.bus_mux = { 0, 0 },
		.bus_clk = { 100000, 0 }, /* default to 100KHz */
		.is_dvc = false,
	},
	[1] = {
		.adapter_nr = 1,
		.bus_count = 2,
		.bus_mux = { NvOdmI2cPinMap_Config1, NvOdmI2cPinMap_Config2},
		.bus_clk = { 100000, 100000 },
		.is_dvc = false,

they have it set at CONFIG_7373C_V20.
Terremark has two set of relative URLs : /api/v0.8a-ext1.6 and api/extensions/v1.6/ . The former is used most often while the latter is required mostly for network configurations. So individual calls can override the path as desired.
Agreed. I like the idea of `Y.soon`, but I think a more pragmatic implementation than its current one is the right balance. Unless of course there's any real reason for wanting faster callbacks that the hacks provide.
If the findResult is null, it means we don't have such type of head.
You can use "htmlSelector" to force it to append to input
We can delete this function  after generalize the SimpleList.init() function
Dead code should just be left out probably.
Why not use http://commons.apache.org/lang/api-2.6/org/apache/commons/lang/builder/CompareToBuilder.html ?
Just realized I need to account for someone wanting to store the value of nullPlaceHolder in the Map. Working on that now.
Should we put this in the base class so we don't need to override it and type the same thing every test? 
We can still override if needed.
Ok, I started out with a state-chart which is in the draft doc, and  
followed that. It made sense for me to have one initial state for both  
cases. But I can change if you like.
Isn't this the same as:

    seen -- removedMembers.map(_.address)

I think that is easier to read than foldLeft for simple removal.
ah, sorry, didn’t properly boyscout yesterday on the plane. (this must have been a white-space only change due to slipping on the keyboard and accidentally touching cmd-shift-f, I guess)
I thought this would be solved with Rolands in-mem db/registry. Is it implemented?
Yes it should, otherwise it will be surprising that the feature doesn't work. We want least surprise.
Can be turned off as optimization if not needed.
hmm, this looks like duplication, will also be done by `RemoteActorRef.start()`
The reason to catch the TypeError is to wrap it in the same DynamoDBNumberError exception, so that all our number validation yields a uniform exception type to the user.
You mean I did this?  Sorry.  I'm sure I never meant this as a final solution.
4.5 is an in-place update of 4.0, so most of 4.0 is good, just some version numbers change.
Maybe one more line in here to indicate that the options would be used to drive how the handler performs the installation.
ah, i see, so the `outerWidth` call sets the width on a `<th>` element, and the `_setRowWidth` is setting the width of `<td>` elements. that explains why you have to check for `NaN` above. so two things:

 - [this rule](https://github.com/siq/gloss/blob/master/src/widgets/powergrid/powergrid.css#L11) needs to be updated to include both the header and the table body (which allows us to remove the `NaN === 0` assumption from above)
 - `_setRowWidth` needs to be changed to only modify the top row
   - additionally, we don't want jquery to walk the entire table (and instantiate a collection with every row), so you'll prob want to do either bare DOM API calls, something like `$(this.$tbody[0].firstChild)` or use a selector like `this.$tbody.first('tr')`.
   - can we still change the name of `_setRowWidth`? something like `_setTableBodyCellWidth` or something?
 - we still need to remove the DOM API calls from `td.mtpl`, so `_setRowWidth` will need to be called after every render
   - a good way to do this will probably involve adding something to `grid.js` that the columns can hook into like:

               _.each(this.get('columnModel').columns, function(c) {
                    c._postRender();
                });

does that make sense?
When I say "don't update referencers", is that just wiki-references, links and transcludes, or is it name references too?

If the latter, then isn't this outcome correct?  watermelon is renamed to grapefruit, but watermelon+seed is still watermelon+seed, so the second link should still be 'L', not 'W'.

This is wrong only if the correct semantics is the former, that we do update the name of a plus cards that include this card.

Which is correct?
Thought so first too, but we return true, so the packet is considered valid
for the caller so it is its job to free it.
Shouldn't there be a 16 bit case as well?
Please remove this file from the pull request.
Varför ändra till `each_with_index` när indexet inte används?
I've created a new wiki page called [MongoDB-store](https://github.com/NAMD/pypln/wiki/MongoDB-store) with details about this specific store (that's the default for now).
Since HEAVY_UTIL is just a valid configuration, it is not a special case, so you shouldn't call it out in the docstring to avoid confusion
On a more serious note, maybe we can just use `-files` multiple times? Worth investigating.
It would be really helpful for debugging to see the chain of includes, e.g. A includes B includes C which tries to include B.
I'm really just getting all locks that are previous to mine. Since `#ordered_lock_children()` takes care of ordering them (ignoring whether it's a read or write lock), I'm just getting everything up until mine. The version you suggest redoes all of the logic that is done in `#ordered_lock_children()` with numeric comparison all over again.
Good point! Changing that now.
If the behavior here is exactly the same as `mouseDown:`, the implementations should be factored out into a common method.
`self.treeview.get_columns()[old_sort_id].get_widget().get_text()` instead of the badly-named `self.get_tree_columns`
Not sure if it matters or is the case. But it may be necessary to test both '>' and '<' for big-endian machines where I guess the singletons should be big endian? would be just a second net here and a second test below (unless there is a way to make net non-native).
No semi-colon
Apparently not but upon review of other functions in the openssl.c file there are resources that are similar, openssl_x509_parse() for example doesn't free a few items. Could you be specific in what the problem is?
Documentation to this class would be great
Ah, yeah, I read the diff wrong. We absolutely want the 1.9 syntax
I prefer:

```ruby
if found
  exec full_path_command, *args
else
  abort(...)
end
```

Is more readable.
Should be commented out?
I will.
`has_index?` and `attributes_with_index ` aren't need, but the setter - i'm not sure.  Only if not use `attributes.each` and use straightforward `first`, 'second' methods.
Please use `ActiveRecord::Migrator.needs_migrations?` to follow Rails code base conventions.
I'm not sure it matters, unless you are planning on a lot of 10+ group tests.  I've not seen more than 3, I don't think.
I agree with your assessment, the ticket is very solid and the link to what test need to be reenabled is very clear.
@mariusaeriksen That would indeed be even better!
I agree with @retronym that always requiring explicit selection makes a big difference. In particular, I think that the vast majority of users (> 99%) will want to use the latest fork-join pool anyway, since there is no faster alternative currently, in which case the default will have to be used. So, I also think that there should be either a default argument or an implicit default somewhere.
We can predict the MST exactly, since the other elements are >=3.
you are correct but i considered this way <code>ask(Q.real(x**(y/z)), Q.negative(x) & Q.odd(z)) == True</code> this holds only when y is even(ex: (-2)**(2/3) which can be expressed as ((-2)**(2))**(1/3) = 1.5874..  according to wolfram it might be wrong but according to any normal calculator it shows it as real) correct me if i am wrong
The older subclassing seems better. `ThinLens` does "contain" to `CurvedRefraction`s, however it inherits properties rather from `RayTransferMatrix`.
I misspoke, what I meant to say is you should call the doit on the args of whatever you return.
Hi Chris, 
Thnx for the correction... It looks proper now.
These are usable only for tests, this is why I have added them in the test files. They are basically crippling `dsolve` in a way that is easier to test.

The functions in the two test files are indeed similar, however I prefer to keep them separate as one of them is much simpler and raises errors on more occasions (and in tests we want stuff that easily breaks and raises error).

Thus, I do not think that these functions should be importable in sympy and I do not think that they should be merged (my opinion on the second one is not very strong, however for the moment I insist on the first one).
I saw elsewhere such private functions implemented within the function.
At least that's my understanding of functions defined within a function.
So... that's what I did.

See my other comment about potentially moving this to the glyph metrics table.
@Hoikas Do you agree with me that this one should be an `==`? (before I merge this)
`detirmine` -> `determine`
AWS passes back the error as a json string.  The core boto exception class is storing this in e.error_message as a string.  I'll take a look as to why I'm evaling instead of parsing it through a json loader.
null means missing data - neither using null nor other special values (in this case -1) are good options, but null feels worse as it is more ambiguous as to whether the data is missing or a special value.
Also, hard-coding to -2 is bad as it implies special knowledge of the implementation of UNLIMITED.
Since we probably need a way to encode 'no limit', I'd definitely prefer -1 to null.
Yup, pretty sure this has nothing to do with S3.  Should perhaps be EC2 instead?
So classes can use the configured adm relative path (nickvergessen's log class needs it)
send() exits, so you can actually do the above if/else block like so:
```php
if ($request->is_ajax())
{
    // $json_response ...
}

// trigger_error(...); 
```
This is consistent with usage elsewhere in the code; same applies to other usages in this PR.
In that case it should at least be documented for future reference.
as we need to change the value in the link aswell, it doesn't really matter, whether its fragile?
This message is now misleading. It should say that an old version of postgres is being used (pre-8.3). Including 8.3 in the message would probably be a good idea.
Should actually handle the error.
please use %1$d and %2$d here
What happens when the name requested is not in files? This should be documented.
what does this file do? it's just returning a nil it looks like.
is this correct? i usually use unpack({...})
I suggest using the tuple literal notation for this, it's easier to read (i.e. return (False, None, None)
Also, please add a comment on what these values mean.
Dafuq style?

delete is not a function
Seems like some virtual issue. So that GL_ARB_pixel_buffer_object feature isn't availiable
I'd actually say this should be start version 0.2? We should agree on some logic for what this means, but I think generally:

Major version - big milestone releases that can break API compatibility (doesn't matter yet since it's not 1.0)
Minor version - backwards compatible feature additions
Revision number - bugfixes
This is for v 0.9.0 so we don't forget (can ignore now). 

1 `s/lowerClr/setClearance/`
2. `s/setLabel.*//`
@isao @mojit0 I am afraid that this one introduces a problem. For example we have URLs in our config that we send to the client and they contain ampersands. Now instead of the string 'foo&bar' we get 'foo\u026bar' on the client. same would happen for > or other characters that are replaced with their unicode representation in this way.

/cc @caridy
The event name is the same name as the method.  I get your point about "on{eventName}".  I'm OK with that as the convention/style, if we agree to that as a team.
I agree, fixed with a new commit.
Yes, this sounds logical.

Do I need to fix this in this same pull request, or create a new one like I did for the self-hosting change?
all files below this one should be removed..
So add all of this into the toptoolbar.js file. We would ideally like to use this in other modules. Make the default value 'login' rather than 'screener', then inside this file, set the value to 'screener'.
I am fine with the normal instead of for...in. Infact I'd say its much more readable when requiring an index counter.
Although why don't you have j initialized inside the for() ??
And on lines 44-52. If you plan to change these later, still worth considering whether you can have a common method that you pass an argument.
best practice: please use ```;``` at end of lines in JavaScript
> There's exactly one or zero per data dimension.

Except, you can have ```DimCoord``` instances in the ```aux_coords``` list....
The "voltdb::" isn't required here right?
Of course, but then they are no longer private, no?
how to write tests is /topics/testing/overview
I know  it isn't equal to 'session' by default. For that i want to modify this...
I change this in my project because 'session' doesn't work but 'sessionid'.. 
And django/conf/global_settings.py:SESSION_COOKIE_NAME = 'sessionid'  # Cookie name. This can be whatever you want.
This is one the cases where eta-reducing makes thinks less readable in my opinion. Before it was visually clear where the ":" went, now you have to think twice to understand that.
white space is present after WEBVTT
It looks like SVG and their inner elements have a style object (separate from their style attribute) but it's not clear if that is due to the spec or just consistent implementations. http://jsfiddle.net/ZrKkc/
We have guaranteed that class >= 0.  However if class == 0 this will result in memory corruption.
tested. works.
Tu peux supprimé aussi je pense
Can you prefix your redis keys to prevent conflicts? We often write  an [r_key helper method](https://github.com/praekelt/vumi/blob/develop/vumi/dispatchers/base.py#L283) to do that for us.
Why do you initialize the store as `null`? What's wrong with `undefined`?
Yet another untraceable log message.
Ergänzung: Ich sehe diese Funktion auch in erster Linie aus der Sicht von Frontend-Entwicklern, und da sollte es aus meiner Sicht eine einfache Entsprechung für ehemals `$REX[‘CUR_CLANG']` geben.
Can you add a comment that quickly describes what this script does?
(generated or other) --> generated

(and above
I think you rewrite this function this way:
```js
function getOpacity(value) {
  return parseInt(value) / 10.0;
}
 ```
You're right. I'm giving it a go using pipe_term and pipe_term_ack, but it feels a bit sketchy at the moment - will add to the pull req once I get it working
hum someone needs to put indent on theses lines, last edit in eclipse wasn't such a good idea :D
Works great.

What do you think about removing Tab key handling? Are you waiting to see what NJ and Adam have to say in the e-mail thread?
If the results cannot be reliably verified, then we should get rid of the entire test. Otherwise, if you're hoping that something may change to fix this someday, then there should be a comment here.
Was the effect of the old code (not wrapped in any `describe()`) to permanently attach before/after handlers for _every_ unit test globally?  That's kinda awesome if so :-P
Would be helpful to have a comment explaining what case we're in now. I'm assuming that at this point, we think we're in between rules.
Relies on DirectoryEntry.toString(). Can we be more explicit here about the key? ``"files_" + projectRoot.fullPath``?
Nit: use ' inside the string instead of \"
The chances of the welcome project missing are pretty remote. I'm ok with this change. The only other option I could think of would be to open the "Open Folder..." dialog but the user still has the option to cancel out and then we're left with no current project.
Could we have `html_value` always return a SafeUnicode and an indication that escaping issues have been taken into consideration? i.e. is this

```python
    return mark_safe("%s" % value if value is not None else "--")
```

or

```python
    return escape("%s" % value if value is not None else "--")
```

or something else?
that's a bigger task than what i had anticipated ;)
simple question: is there any valid property in the ACL use case other than '{DAV:}property' ?? does it worth the trouble to implement all that ?
I will need to change this to check if the menu item has any child menu items... rather than checking by menu item ID's.
Unnecessary spaces around function arguments.
Will we allow posts to be edited?
Use util/uri.js to parse this, in case we add more things to the URL later.  Also, after you get the `savedDataUrl` query key, use a regex to clearly show what you're meaning to get (in case our URL structure changes, this will be more clear what you meant by the last /):

```
/\/api\/project\/(\d+)/
```
Missing apostrophe for "servers".
We can make this function faster and more compact: http://jsfiddle.net/75Dze/7/

Also, it looks like this happens every time `_setup` is called. Can we just draw this once and copy it a few times like this http://jsfiddle.net/75Dze/21/ ?
bloop space
You mean to load scripts more efficiently in parallel? Or to deal with dependencies easier? Either way, I feel like `require.js` just adds more baggage to the page than what it's worth. I would prefer to just keep it simple, but if you have a case to make for `require.js`, I'm all ears.
You can remove this and `this.pageId = pageId;` line above. JavaScript will use closure to store pageId for `event.renderingDone = pageId;`
unrelated to the problem fix
I dont see the need to return_request either.

The full_request is just putting the XML from two sub methods together. :meth:`self.access_request` and :meth:`shipment_confirm_request`.

If it is for debugging, the full request is available in the logs when the logger is in debug mode.
Just use Set()
By convention, types start with capital letters.
indeed. This is used to render some nice HTML (which can be changed by setting the ``compressed`` option to true, like other renderers)
defbufferop FTW!
Not configurable
I'd still like to see this change in a different patch as it's irrelevant to NUL support. You could submit this and the malloc/free #undefs in a separate "cleanup" patch.
@fabi1cazenave does this change makes sense on your side?
Ok, that makes sense - thanks!
I think there needs to be a space either side of the ':' for the Parameters and Returns sections, otherwise the docs don't build properly. (Annoying, I know.)
Any objections to renaming this to `are_pending_ownership_changes`?
Also, `riak_core_vnode_manager:xfer_complete/1` doesn't exist, only `/2`
Yeah it signifies which version the database is on (which migrations have been run). There is no reason this should have changed in this PR and means you probably have something funky going on with your database.
Don't you mean .clear()?
ipcThread() and ipcThread2() are now static functions :).
This replaces the usual...
Why do you put a newline at the beginning?
Missing a space between IF and open parenthesis
This should probably just be called `for` and be either `"personal"` or `"corporation"` (as returned by the API).
I think `document()` runs `load_all`, so this may not be needed. But the `reset` and `export_all` args should probably also be passed to `document`,
Fixed in 85e9d88, thanks.
My bad. I understand now why this is needed.
-ios-cap-insets
That's too much worrying. I'm going to restructure it so it does the right thing without having to.
Please indent the Q_OBJECT macro.
wow, nice sql-fu. Makes sense to me after your explanation. If the all the revisions have been reviewed and rejected, then it will appear as if this translated document didn't exist at all.
All lowercase would be better: `tabindex`.

http://www.whatwg.org/specs/web-apps/current-work/multipage/editing.html#attr-tabindex
Since we use `socorro.webapi.servers.CherryPy` we inherit the same base class as collector does. 
https://github.com/mozilla/socorro/blob/master/socorro/webapi/servers.py#L75-L79

Perhaps I can simply copy that stuff over to middleware_app.py. It's a bit impure but perhaps necessary. 

I'll see if it's possible.
you're creating a lot of complexity just to reuse the same connection for each run of the target app.  if you move your connection logic to the `run` function the code is simplified.  It eliminates the closure and the need for BaseBackfillCronApp.main to have the function passed in.    

```class BaseBackfillCronApp(BaseCronApp):
    def main(self):
        now = utc_now()
        if not self.job_information:
            self.run(now)
            yield now
        else:
            # figure out when it was last run
            last_success = self.job_information.get('last_success',
              self.job_information.get('first_run'))
            if not last_success:
                # either it has never run successfully or it was previously run
                # before the 'first_run' key was added (legacy).
                self.config.logger.warning(
                  'No previous last_success information available')
                self.run(now)
                yield now
            else:
                when = last_success
                seconds = convert_frequency(self.config.frequency)
                interval = datetime.timedelta(seconds=seconds)
                while (when + interval) < now:
                    when += interval
                    self.run(when)
                    yield when

class PostgresBackfillCronApp(PostgresCronApp):
    def run(self, date):
        database = self.config.database_class(self.config)
        with database() as connection:
            self._do_my_postgres_work(connection, date)


class MyPostgresBackfillCronApp(PostgresBackfillCronApp):
    def _do_my_postgres_work(self, connection, date):
        # whatever it takes
```
So since below we are going generic for invalid field, maybe we should just do the same here?
Why di that space get added? Looks odd.
Можна, але навіщо? Цей варіант лаконічний і правильний, навіть в документації TWIG такий приклад перевірки непустого масиву приведений.
Hi,

Oh.. i did read it twice, that just not enough... will just make a new
commit.. thanks for the correction :)

Best regards,
Michael Holm


2012/4/26 Kim Hemsø Rasmussen <
reply@reply.github.com
>

> >              </trans-unit>
> >              <trans-unit id="39">
> >                  <source>This value is not a valid locale.</source>
> > -                <target>Denne værdi er ikke en gyldig
> lokalitet.</target>
> > +                <target>Bærdien er ikke en gyldig lokalitet.</target>
>
> Hi, think you did a typo here :-P
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/symfony/symfony/pull/4123/files#r739143
>
`set` usually replaces all existing values (i.e. not passed items are deleted). But here you only replace existing and add non-existing values.
So I think the correct naming should be `addVars`, shoudn't it?
Shouldn't that be either handled in the constructor, like for Memcached or sent as a configuration dependency?
you should use a method of the extension instead, to be consistent with all other extensions provided by Symfony
you should move them at the end and make them optional arguments, to avoid breaking existing code calling the ``find`` method
why not just using a typehint ?
コメントありがとうございます。
確かにこの場合、Topic#owned_by(user)にした方が意味的に良いですね。。。
修正します！
Could you add some comments here to indicate what you're doing? The mocking is pretty inscrutable.