labofsahil/github-event-dataset-2012 huggingface dataset:
Yep will do that. this commit looks fine otherwise. Will amend my commits on this then
Yes.  I use vim-powerline and love it.  It's referenced in the Statusline wiki page.
Would it not be better to use something like:

has_one :space, :class => 'Space'
should take $errmsg
btw, we can use https://github.com/errbit/errbit - The open source error catcher that's Airbrake API compliant
Ok, I'll change it to just start datetime and end datetime
Bitcoin developers can't shutdown DIANNA nodes now
This doesn't make sense.

You can clear a float afterwards with class="z-clearer" or (better) you clear the float before.
``<div class="z-clearfix indexBox">'.$indexBox.'</div></div>``

by the way: Why two closes DIVs and only one opened?
@Asyfupid 这里需要后台传一个评论人的id，这样回复传给后台的时候知道是在向谁回复。 data-xx-xx 是html5的新功能。
Should we switch this to make tenant_id the primary attribute?
@Reamer correct timezone
应该是$result吧
Interesting idiom for conditionally executing a JS function, haven't thought of doing it this way before
@kraven5 +1
Yeah, I suppose. That's a tough one, but I'll change it.
@kabum 

Ich glaube es ist enfacher die Properties hier erstmal redundant zu speichern, die Load()-Methode des Repositories müsste sonst rekursiv arbeiten. (kann man machen wenn man zeit hat :))
Look at the CursorLine of the 'murphy' colorscheme.
When loaded with CSApprox we get:
    :hi CursorLine
    CursorLine     xxx term=underline ctermbg=241 guibg=Grey40

This allows other highlights' foreground colors to show through the cursorline. Note that there is no 'ctermfg' entry. The above code (in red) causes the cursorline to always be the Normal fg color instead, which is quite worse.
Hi. FYI, your fork of salat produces build logs several dozens of megabytes in size (I see sbt debug messages all over). This slows down log collector (PG appends log chunks to very long logs significantly slower). If possible, please make sure log output does not exceed 1 MB. Thanks.
That's one to many ; there
Didn't know that, it's done.
what is the purpose of delay here, if ManaUseEvent does not use it?
when you use the alliance to horde changefaction, the horde get this spell http://www.wowhead.com/spell=40361 ...try to disable it.
wrong. unit always visible for  self - see begin of method.
alphabetize conf < core
That is exactly why I didn't revert it @narrowtux. This formatting style is exactly what our guidelines statee (as @Top-Cat mentioned).

If it is to be changed, then let me know and I can make the change.
Can you align the asterisks here so they are in one column. ie:

```
/**
 * ...
 */
```
Hi #sudoh this is excellent!  I don't love the use of data-dojo-type="dijit._WidgetBase" but it shows how it can be done.  I think we should create a "dojox.mvc.BindNode"? or something which would just extend _WidgetBase to indicate that it is used for binding, I would say just use dojox.mvc.Bind, but that is already used...
Can this loop not move to SDLVideo, and the switch into ProcessEvents?
I agreed that this is not the most efficient way to do this. As we are really focused on UX we try to set a sane default value. The way this loop is designed we'll only loop if user's calculated display_name matches an existing one, which should rarely be the case. If that happens we append _'s until we find a unique username. I predict that  should be on average just an extra hit on the database (i.e. we find a unique display_name on the first time) and max 14 hits on the db (extreme case, that 'll probably never happen) (note that we'll be expecting max 500 users in this website)

I understand that this is not optimal code / performance wise but it's optimal UX wise and I 'll like to keep it this way. Anyway since we are not expected to run into this situation a lot we can go either way. Let me know that you think it's best.
I don't think this needs a setting. I think we can figure this out from the data available/service catalogue/a quick query to the services...
person -> человек
for the lack of a better translation let's use a literal one
I think I've replaced most instances of the dash to use the pipe so we're consistent.  The homepage might require a different style to the rest of the site though I'm guessing.
@JMobius, @the-smith,
in the light of the current discussion on Infernals, is this correct? Shouldn't Infernals resonate with either nothing (sidebar p173: 'lose all magical material affinity') or with vitriol-tainted materials?
OK, that makes good sense. After the first load our copy of jquery is probably cached anyway.

hmmm maybe a new open source idea, tracking free content server to get the best of both worlds!
Hoo boy. I must have been very tired when I wrote this.
we should remember to enable this on `marketplace-dev`
Note that I changed the way that the repair of the inner array was done.  Initially, it was broken in that most any action would overwrite an explicitly set value.  However, this also meant that a default value was not needed.  So, rather than add an additional variable, I initialized the policy variable to null.  In the event of an explicit set by a function, the policy variable will no longer be null, and the passed value will be ignored.  Let me know if you prefer the original fix with an additional policy variable.
why comment on this line?
They have been removed in prod a few weeks ago.
Also, it seems that randommovementgenerator isn't working.
shouldn't you run the cron job with the ``--no-interaction`` option instead ?
Ja...aber mein Fragzeichen galt eigentlich genau der Zeile...aber nicht weiter schlimm ;)

Dir auch ein schönes Wochenende!
Better keep the dependencies listed on the README, as there are other *nix apart from debian/ubuntu/fedora.
But even if they were, it still makes sense to keep them, just for the overviews sake.
Should be StreetsOfMoscow
There are a lot of possible shortcuts, when creating this future we will have to try and use a general approach. I placed it on the wishlist.
Shouldn't change_block_button be is_sensitive?
Could you add one additional check for when they opt-in but don't pass one of the three known symbols?
throw Throwables.propagate(e);
I don't think this would really work, core, Browser, JSON are special, Array doesn't put anything in the global object, nor does Fx.Tween, etc..
Moved, just one question:

According to the instructions it should be alphabetical or am I wrong ?
Это лучше засунуть в циклы и прописывать лучше не :ad_type_id=>2 а создать ad_type_1 = AdType.new(...) и затем 
Ad.create(:desc=>'sdsadcx', :user_id => 1, :ad_type_id=>ad_type_1)

тоже самое и про User
Yes, should be fixed now.
All I did was work on setting up the arrays so that i can proceed with the rest of my code.  My issues are not only with learning how to code, working with git is not joke
This broke my project so that it will not run anymore.  

Bug below.  (Ignore the title of the bug.  I thought something else was causing my problems at first and Lighthouse frustratingly won't let me change the bug title.)
https://play.lighthouseapp.com/projects/82401-play-20/tickets/85-cannot-clone-project-into-directory-of-different-name#ticket-85-2
I don't really see when this append_diag function is useful in practice. Could the doc string suggest a real use case?
Ok but hinge loss + l1 will still be sparser than perceptron loss + no regularization. And as perceptron is traditionally used without regularization it would be misleading for the users to ignore the regularizer impact in that comment.
확실히 윤군이 오니까 봄싹 개발이 활기차지는군요… 캬.. 좋아~ 더 불살라~  

--  
KeesunBaik


On 2012년 2월 22일 Wednesday at 오후 5:21, Outsider wrote:

> 헉 복사하니까 괜찮아졌어.. 받침이 다 옆으로 떨어져 나왔었는데!!
>  
> ---  
> Reply to this email directly or view it on GitHub:
> https://github.com/whiteship/springsprout/commit/5a70cd5b61a6a8f4ca519b9ab03ff979bf07045b#commitcomment-994977
>  
>
K ich schau mal danach.
Haste die SelectRandom Methoden Convertiert? die gehen bei Skyfire nicht.
Sqls eingespielt? Scripts zum Scriploader hinzugefügt?
I actually like the declaration in the if statement.  It implies that the variables will be used inside the if block and nowhere else.  What benefits does the other method provide?
Hi Laura, there are two functions that generate recent activity statements. One for the website, and one for the email (they should really be refactored into one!).

Did you update both? or just one?
Thank you @timwienk. I appreciate your continued support and your trust in my abilities. I will ensure that mistakes such as this will not recur in the future development of this fine piece of JavaScript code.
I've put my changes back in with 5b771883ac85a54dcb2599002226feaaf822cc5d but it would be get to the cause of this. ianschneider reported a similar issue happened to him a while back. ahocevar suggests this could be an unlucky combination of stash, checkout and local file copying.

Any chance of finding out which shell commands let to this?
Should be checking $7 instead of $5
2) I still don't get why include to stdlib.h need to be moved out m68kops.h, this does not make any sense to me so can you at least give me the error or warning you are getting ?

It wasn't there to begin with (I think?), tried adding it, to fix this warning:
src/m68k/m68kops.h: In function 'UseDivsCycles':
src/m68k/m68kops.h:51:3: warning: implicit declaration of function 'abs' [-Wimplicit-function-declaration]

Adding stdlib.h however, I got:
In file included from /usr/include/stdlib.h:320:0,
                 from src/m68k/m68kops.h:5,
                 from src/m68k/m68kcpu.c:46:
/usr/include/sys/types.h:153:22: error: duplicate 'unsigned'
/usr/include/sys/types.h:153:22: error: two or more data types in declaration specifiers

so, I just reverted, and just pulled a "meh, I'll fix it later :P"


As for NTSC stuff, some guy specifically submitted a patch to being able to use NTSC directly from the core, so I merged it. However, when we pulled latest, the old NTSC fixes he did were overwritten, so had a look at it. Normally, I'd assume we could just set output format to 15-bit, but this didn't work, as the NTSC filter for some reason hardcoded stuff.

#define MD_NTSC_RGB_OUT_( rgb_out, bits, x ) {\
    rgb_out = (raw_>>(13-x)& 0xF800)|(raw_>>(8-x)&0x07E0)|(raw_>>(4-x)&0x001F);\
  }

Studying the code, this is for RGB565, so ifdeffed that for RGB555 with:
#define MD_NTSC_RGB_OUT_( rgb_out, bits, x ) {\
    rgb_out = (raw_>>(14-x)& 0x7C00)|(raw_>>(9-x)&0x03E0)|(raw_>>(4-x)&0x001F);\
  }

I know blargg SNES NTSC has proper defines for this, so maybe it's something that was pulled out?

At any rate, SSNES supports external plugin filters, and it'd be tons more clean to do it outside the code as a filter plugin, but haven't bothered making one yet.


One extra thing, which you might have an idea about. I had a strange bug report about libsnes port of genplus: 
https://github.com/Themaister/SSNES/issues/6

Reading this, SRAM is apparently always active and dumps an SRAM full of ones even if there's not supposed to be SRAM.
isnumber is not a implemented in my cstdlib, i [looked up][http://www.cplusplus.com/reference/clibrary/cctype/isdigit/ ], it seems isnumber as an alias of isdigit so changed it this way
Error when compiling, calendar related perhaps? I'm on revision bc151a0453a7
---------------------------------------
Compiled with MS Visual Studio 2008 Pro
Error	1	error C2039: 'SendCalendarRaidLockoutUpdated' : is not a member of 'WorldSession'	d:\Trinity Project\Custom with patches\src\server\game\Handlers\CalendarHandler.cpp	800	game
Error	2	error C2065: '_player' : undeclared identifier	d:\Trinity Project\Custom with patches\src\server\game\Handlers\CalendarHandler.cpp	805	game
Error	3	error C2227: left of '->GetGUID' must point to class/struct/union/generic type	d:\Trinity Project\Custom with patches\src\server\game\Handlers\CalendarHandler.cpp	805	game
Error	4	error C3861: 'SendPacket': identifier not found	d:\Trinity Project\Custom with patches\src\server\game\Handlers\CalendarHandler.cpp	817	game
Error	5	fatal error LNK1181: cannot open input file '..\game\Release\game.lib'	worldserver	worldserver
I don't understand this the way it's written. Maybe this:

    `delete` operator applied to bare identifier is prohibited
Could you add:

    link_directories(${LCMS_LIBRARY_DIRS})

..as I get linker errors on OS X when using an external LittleCMS (installed via Homebrew), because `${LCMS_LIBRARIES}` only contains `lcms` (unlike the bundled one which is a full path)

Nothing to do with this change,  just I was going to make a pull request to change this shortly!
В принципе это правильно. Но лучше абилити для сущностей прописывать руками и для админа. 
Причин как минимум две:
 - легче тестировать под админом, если есть ошибка в абилити она будет и под админом (не всегда конечно, но часто можн их увидеть). Дело в том что при разработке (добавлении новой сущности) не всегда проверяешь под user
 - может так получиться что админ может редактировать сущности которые относятся ТОЛЬКО  к нему, тогда придется писать что то подобное
can :manage, :all (это было)
cannot :manage, Image
can :manage, Image, :user_id => user.id 

Получается какой то костыль.
That smells possible. 
On second thought, wouldn't you need to change this line at top of routine also

    if crudful_request_methods.include?(meth) and args.size == 1 ##meth.to_sym?
That would work also, but we would probably want to reference that man page somehow or others might not know it exists.  This would create a separate page in the EditorConfig documentation, which may be a good idea.

If we wanted to duplicate this documentation we may also be able to use something like [@copydoc][], though I'm not sure how useful that would be.

[@copydoc]: http://www.stack.nl/~dimitri/doxygen/commands.html#cmdcopydoc
my text editor has free will.
use `?w=1`

(https://github.com/mate-desktop/mate-applets/commit/6458798c9ee00bcbb62e40c73075d831544f2839?w=1)
This API is still confusing. In some place we use `:pattern => true` and in another places `:readonly => :lookup`. I think that we should use `true` always.
or we could do 

```python
if isinstance(rsp, dict) and 'error' in rsp and 'code' in rsp:
```
@titu2012 I think your best bet for DB interaction would be <http://www.phpactiverecord.org/>. It's easy to implement and provides an awesome syntax when in an MVC environment.
Re-wrapping to fit in 80 columns, and to line up properly
This also includes the changes required to handle cases where redirected to a URI without any parameters (e.g. bitly on error).
err... Why's there a whitespace here. :(
What are the return values supposed to be? Can you document that. You don't seem to use the return value of these functions anywhere. Maybe on line 539, you can just call decrementRef and if it returns 0, you return -1 or something.
@derekhiggins  do you never need to create the bridge explicitly?
wow, 1 MB is still much if you only have a third of a GB of RAM ^^

it's working without any issues on Talon for some time afaik

Thx & Cheers
>post.Visib
>.Visib
Lolwut?
Why you reverted auth changes for 432 ? QQ
If layouts fit in grids, then grids would also fit in grids.

Is there an advantage to having a kind of layout which is not a grid?

(I have not looked at grid and have not attempted to think about css
layout concepts -- floats and so on -- though I am of the strong
opinion that css should be restricted to providing us with a
vocabulary for treating style issues.  I think we have to use
javascript as the primary styling tool.)

-- 
Raul

On Fri, Feb 3, 2012 at 10:22 PM, jhensley
<reply@reply.github.com>
wrote:
> Do we actually want layouts too? Or do grids replace those? I could actually see a layout fitting in a grid, so if a grid has 3 columns, a layout could be a collection of modules within that column... thoughts?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GannettDigital/django-change-o/commit/f658aa24e0519bf6487130ed7d6a700c37a02b18#commitcomment-932558
its not just me.  shaco  has done amazing work on private repo too :O
Yeah, I wasn't really thinking we could hide it, we've passed it around a
bunch already. i'd just wanted to encourage people to build it locally.

-Dan

On Tue, Jan 31, 2012 at 13:27, Aslak Knutsen <
reply@reply.github.com
> wrote:

> hehe yea i saw. only one tweet tho :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/arquillian/arquillian.github.com/commit/dd84352481eb3f927a89cbbdd2662bce72945cdb#commitcomment-919420
>



-- 
Dan Allen
Principal Software Engineer, Red Hat | Author of Seam in Action
Registered Linux User #231597

http://google.com/profiles/dan.j.allen
http://mojavelinux.com
http://mojavelinux.com/seaminaction
Yeah I'll take a look.

On 30 January 2012 09:30, Marko Tibold <
reply@reply.github.com
> wrote:

> Makes sense. I'll take care of it. Noticed the build errors on the Jenkins
> CI btw?
>
> On 30 January 2012 10:25, Tom Christie <
> reply@reply.github.com
> > wrote:
>
> > Okay yeah makes sense.
> > So long as that builds in the docs fine that sounds good.
> > Also re naming, I think it should probably move to CHANGELOG.rst.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/tomchristie/django-rest-framework/commit/f34ed6d1f3d8c1c4c5059beba944aa386e9a15ab#commitcomment-913588
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/tomchristie/django-rest-framework/commit/f34ed6d1f3d8c1c4c5059beba944aa386e9a15ab#commitcomment-913601
>
A couple of comments, Ola:
 * Would this be a good time to discuss https://trac.openmicroscopy.org.uk/ome/ticket/5953, which got closed after Jean-Marie updated?
 * I'm not currently getting anything under OMEROweb_request.log (debug=true and false)
 * The logger names from blitz_gateway are still short. Should we look to changing those to use __name__ as well?
everything english pls. as commented above; don't forget the IP; the validity could be added within the generated template; e.g. http://openengsb.org/registry.html?validTill=20120221120000&... 

this might not be perfect, but should be sufficient for now. Try to keep the bar low in a first shot. You're implementing quite a complex feature here. So try (at least for now) too keep the complexity low as far as possible at edges where it's not required for now.
because need modification Framework base, and i cant push it because my bandwidth is to slow :(
This commit includes a NEW button_group function, added MY_form_helper class that overrides the form_helper checkbox, radio button, and button functions.  The documentation examples was also updated to reflect the button additions.
Also, in the constructor, it would be an ideal place to provide a `timeout` value, too, alongside the `queue` and `connection` args.
ok, I guess you mean clear().  ok, I see that we turned that off.   might be an experiment down the road to see what that was about.
I think I added the `--promp`t option as a continence for those who didn't want it, but it now seems like prompting is necessary (not optional), at least to get the consumer key and secret. The option can probably just be removed.
Why do we skip this test for CUDA? Is that the indirect loop issue from #55?
removed the default value
added some docs
Fixed both. Thanks.
I'll bring back the old preRestart and add the postStop callback and mention it in the migration guide.
change to "will automatically be formed" - since a sysadmin can choose to form it even when this is off
Yes, but that is a very different thing.
I can believe that, but why don't any tests fail?

Oh, I think I get it.  It is sort of obscure to test.
really picky - we shouldn't have those spaces`{ 'offset': 0 }`  should be `{'offset': 0}` pep8
we patched it but were not able to submit it upstream (yet), so I assume we want to keep using our patch.
No e.g. after format. That one is less my OCD and more consistency with the rest of the CLI.
* Lets just support symbol option keys, not strings, consistent with other middleware options.
 * Maybe `min_size` or `min_length`? Just a little shorter?
If I remove that logic from CVideoDatabase and CMusicDatabase the filtering won't work anymore because it needs the actual videodb:// and musicdb:// path.
Oh, didn't look into the ProcessBuilder. I'll check :)
That's better. Will use it next time :)

Aynur Abdurazik, SAIC
Ocean Biology Processing Group
NASA/GSFC Code 616  (Mail Stop 616.2)
Greenbelt, MD 20771
Phone: (301) 286-4072







On Mar 12, 2012, at 10:49 AM, Norman Fomferra wrote:

        return image;
    }

+
+    /**
+     * logs each pixel value when a user chooses a log scale color map.
+     * @param sourceImage
+     * @return   returns a new image file with logged pixels.
+     */
+     //new method added by seadas team
+    private TiledImage logScalePixels(RenderedImage sourceImage){

In order to transform image data, please implement a special JAI OpImage. However, JAI already provides the "log" operation. See javax.media.jai.operator.LogDescriptor.

---
Reply to this email directly or view it on GitHub:
https://github.com/bcdev/beam/pull/2/files#r546056

--
To UNSUBSCRIBE: Send an e-mail to
seadas_dev-request@seawifs.gsfc.nasa.gov
with the subject, "unsubscribe"
Just some javadoc fixes
I know, I even noted that in my comment. :(
Insert a visual clue once a submenu that allows to change a value is entered. This is followed by the according delete command (via empty space).
actually I think that using a lock might be The Right Thing here, as FD creation is not a hot path and legibility trumps performance then
no need for the parens around localAddress
Capitalize 'if', space before paren.
I think we should keep C APIs simple. If we would support both strings and numbers, we can do that in lua modules.
Actually, it does right now with modeNum in fs.lua.
https://github.com/hnakamur/luvit/blob/master/lib/luvit/fs.lua#L92

And this is the same style as node.js.
The C++ api takes only number mode argument.
https://github.com/joyent/node/blob/master/src/node_file.cc#L575
modeNum function in fs.js converts strings to numbers.
https://github.com/joyent/node/blob/master/lib/fs.js#L313

However, as I said in my mail in the mailing list, I think modeNume is a waste
to check type of arguments at runtime. And it leads to various ways to write the
same mode as '0666' or tostring('666', 8).
If we would like string, just writing tonumber('666', 8) as arguments would suffice.
Call this something like filesAdded to make its meaning clearer
Discussed horrible alternative at Ben's desk.
The window of opportunity for getting to this point is incredibly small, has this been measured to be faster?
@njsmith: The complexity is in extracting the relevant parts of the stack trace, which, in the cases I've dealt with so far, is to to find the first line in the library you care about.  In the callback case, this complexity is higher, because you have to walk the stack.  In the log case, you only poll on the relevant lines.  This is the trace function I'm using for profiling CellProfiler, for instance:

    def trace_memory_usage(self, frame, event, arg):
        # We get line and return events just before they are executed.
        if self.path in frame.f_code.co_filename and \
                event in ('line', 'return'):
            new_state = (frame.f_code.co_filename, frame.f_lineno)
            if self.state != new_state:
                meminfo = get_memory()
                if meminfo['allocations'] + meminfo['frees'] > 16:
                    meminfo['lineinfo'] = self.state
                    meminfo['time'] = time.time()
                    self.trace.append(meminfo)
                self.state = new_state

        # continue tracing
        return self.trace_memory_usage

`self.path` is the path to the directory with the code being profiled.  `get_memory()` fetches and condenses the log to something useful.  The overhead is not particularly high, since it only triggers on lines within the relevant code, and if the log is empty, very little work is done.  I'd estimate running `np.test()` while profiling numpy takes only about 1.2 times as long.

I expect most people could make use of a simple profiling class, sidestepping most of this.  I have one based on https://github.com/fabianp/memory_profiler that I'll add to this PR in the tools directory when it's cleaned up.
Provide a `filterToParams(filter /*OpenLayers.Filter*/, params /*Object*/)` function with the protocol configuration instead of registering this listener.
Agreed that makes sense.
That's not valid syntax.
Maybe `remove`, not deprecate? Because deprecation it's only AS::Deprecation.warn message if someone send `key` argument to `run_callbacks`
Usability and clarity.

```
Rendering exception status: 500 with ActionDispatch::Routing::RouteSet
```

Was not intuitively obvious to some people I showed this to. This seemed to be better:

```
Rendering exception status: 500 with action specified in routes
```

The two most common cases would be the Rails routes and the PublicExceptions app, so I added this helper to clean up the names a bit. Would you prefer if I kept the original class name in the description?

```
Rendering exception status: 500 with action specified in routes (ActionDispatch::Routing::RouteSet)
```
Ah, I see, we aren't using sprockets-rails from git in Rails, so the change in Gemfile in this PR is fine
I don't see this being used anywhere, since it's apparently always setting `Parameters.permit_all_parameters`. Is this supposed to be a per-controller config? Also, there's probably no need for `||=`.
I don't think is a good idea assert the code using the same logic. I think is better to not use the `gsub` and do the assertion with a string.
Ok, so why don't you just return the 'tree' there?
We have a `SystemProperties` in the `sys` package, right? We could extend the functionality there.
Many of these tests don't require numpy.  Perhaps they should be split out into separate tests and only those skipped if numpy is not installed.
Oh, this should just be ```for elem in self``` you can drop the .elements. self is an iterable.
Actually, I think the original intent was just to show that if you pass a key that is not there that it will give [].
We already have https://github.com/sympy/sympy/blob/master/sympy/core/tests/test_arit.py#L127 so I'm convinced to remove it.
That looks crazy! What's wrong with `MatMul(*args)`?
We are going to do the representation in a separate pull request.
yes, that's why there where no check at all before.
The previous expectation was that `respond_to?(:binmode)` would return true because the object you were dealing with was an already opened file, ie the result of `file = File.open(filename)`.  Which is why the binmode check/setting was used (because I believe just calling size in that particular case will not be accurate without doing this).  Does that help clarify?
You can remove this, since it didn't actually help. Thanks for giving it a shot, though! I'll try to sort out the Travis issues later.
Well squeel has like 3 ways to refer to external variables, but the easiest one is to make them local.

Also we can still use the variables in non-squeel blocks (like Question.first("number" => number & "version" => version-1)) but since Squeel prevents us from using :symbols at all in where(), I just went and made everything use squeel syntax.
Just a positive note - this is an example where you wouldn't use the `@Test(expected=...)` syntax as you are verifying some state of the exception.
As with auth, I think the way to say these "when running with journaling"
Can/should we re-use LANGUAGE_DEKI_MAP here instead? (https://github.com/mozilla/kuma/blob/master/settings.py#L155)
So, this is returning true and continues iterating ?  In line 1191 you have an explicit true and that confused me a bit.
Then yes, just do `$result['avatar_type'] = $driver;` instead of redefining the whole array with the same key/value pairs.
Do we really need the leading underscore?
I had a feeling this might be the case, you can leave this alone for now.
Make "avatars" singular.
This should probably be a "move", so the previous assignment `$time = time();` should be removed.
This table should probably go into a fixtures module too.
lets kill references to the stateFile if we aren't using it.
For touch devices, audio simply cannot play without initially calling calling `prepareUserEvent`. I'm not sure of a case where it'll already be playing...
yes, just added a new commit with the code removed
if this is an openmrs Encounter, prefer not to name it *raxacore*Encounter. Else it appears to be something different. just "encounter"?
Further to an earlier refactor comment, these might become kwargs to *create_fake_message*.
For your knowledge, Py_XDECREF is equivalent of:
if(%(z)s) {Py_DECREF(%(z)s);}

So your if is useless here.
This function will be changed and cached. I think it should be amortized O(1).
I think we'll need to handle initial facilities differently, but that won't hold up this request
The other code in the platforms does not separate them. This is the only reason why I omitted the empty line
Why not just share the current `fxNow` and keep that logic around?
Please add a comment explaining WHY we need particular alignment.
Yes, we have the compatibility layer and use it.
Draft.
<pre>
Sort input strings based on the module name in them such that the dependency is resolved.
</pre>
Removing list_c in its entirety seems to be a rather substantial change here.  Can you provide additional explanation?
south is in here twice.
This idea came from a conversation I had with @jsocol when the Fennec team was trying to make requests against the old input site. The 200 codes it was returning were confusing, and I did not see a 4xx code more appropriate. The user should not see the 400, so it shouldn't be disruptive.
This can remain a single line ("postfix" if statement); there are a few others below as well.
`ENV.llvm` should be avoided and a `fails_with` block used instead; `ENV.x11` can be replaced by `depends_on :x11`.
Remove the parens for style, leave block indented
Won't that potentially interfere with SMSSync's own URL parameters?
Maybe: 

"Sometimes you need an object to be instantiated lazily.  You can achieve this by depending on a Lazy<T> rather than on T directly." etc.

This avoids the lazy-dependency-creation terminology a bit.
Yes. I agree. Olof just spent 15 minutes explaining it to me.

Please don't accept this pull-request. It is just stupid. :)

On Wed, Oct 31, 2012 at 9:40 AM, Joel Westberg <notifications@github.com>wrote:

> In
> stages/processing/basic/src/main/java/com/findwise/hydra/stage/CopyStage.java:
>
> > -		Logger.debug("Copying field " + fromField + " to field " + toField
> > -				+ " value: " + valString);
> > -	}
> > +    @Parameter(name = "prefix", description = "Input field prefix")
> > +    private String prefix = "";
> > +    @Parameter(name = "postfix", description = "Input field postfix")
> > +    private String postfix = "";
> > +
> > +    @Override
> > +    public void stageInit() throws RequiredArgumentMissingException {
> > +    }
> > +
> > +    @Override
> > +    public void processField(LocalDocument doc, String fromField, String toField)
> > +            throws ProcessException {
> > +        String fromFieldStr = prefix + fromField + postfix;
>
> I still don't see the difference. Suppose fromField is the sequence of ["x",
> "y", "z"] with prefix "pre_" and postfix "_post". You will invariably
> generate the fromFieldStr sequence of ["pre_x_post", "pre_y_post",
> "pre_z_post"] right?
>
> So why not just use that as the fromField if you always need it to be that
> way?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Findwise/Hydra/pull/107/files#r1989349>.
>
>
Thankfully the "pop" function takes no arguments otherwise this horrible error would trigger a failure :) You MIGHT have meant to write "k.toString()" here rather than referring to the toString function, but in fact you just need to write pop()
Why does every key in each transition need to point back to the transition name?
map this stuff into `/profile`? it has email address, which lots of ppl want.
since we're always using `dojo.io.script`, we could just hitch it here instead of creating `fn` above.
Agreed. will update soon.
I think it'd be better to put this in EditorCommandHandlers.  Most shortcuts related to editing text live there... select all (the only other one in this file) is sort of a special case since it's really just a patch to work around bugs in the core CodeMirror select-all functionality.
Would be nice to store `ExtensionLoader.getUserExtensionPath()` in a variable instead of calling it twice in a row.
Actually, we haven't created it yet--maybe you meant the comment to say `// Look for the selector we're about to create--we shouldn't find it yet`?
Here the LogProvider is setup. This is what a dev would need to do to capture log output of Raven.Client.Lightweight in their application.
This is going to need documentation. Also note that I have a related bug at http://code.google.com/p/dygraphs/issues/detail?id=401 -- is it possible to fix both of these at once?
I think in general it's impolite to reference ._ properties from outside
the class hierarchy since python doesn't have a notion of private fields,
so I actually prefer get_id everywhere.


On Fri, Nov 2, 2012 at 3:44 AM, Mike White <notifications@github.com> wrote:

> In corehq/apps/reports/views.py:
>
> >
> >      configs = ReportConfig.by_domain_and_owner(domain,
> > -        request.couch_user._id).all()
> > +        user.get_id).all()
>
> As far as I've been told, the reason .get_id exists on couch Document
> objects is because you can't access an underscored attribute in a django
> template. I've seen it elsewhere, but I don't know why one would use get_id
> instead of _id in Python code.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/core-hq/pull/174/files#r2013865>.
>
>
Fade visible in is legitimately a different desired effect. It's so that there isn't one object fading in while the other is fading out right below it.
This should be fixed before merging.
Some comments would be great for this method.
Cancel this function call if `self._endOnTick` is `false`.
User reST markup please!
Why you don't extend the one provided by mongoeninge?
this prints "Invalid..." when we run `jekyll generate site` but `site` is a valid option above in:

https://github.com/mojombo/jekyll/pull/567/files#L0R167

```ruby
if type.nil? || !['post', 'site'].include?(type)
```
you're mixing spaces and tabs in this file. Probably convert everything to 4 spaces for consistency.
If we REALLY want it better to put an empty div inside this one, and make that the target.
you can achieve this with a single regex.

`/^http(|s):\/\//`
Do you have max/min sizes you allow?  A font size of -6 will be `true` here.
You want to use hasOwnProperty here, since pluginOptions, or its prototype chain, will always have it:

`if ( pluginOptions.hasOwnProptery( key ) ) {`
One little nitpick: can you move these three event creation/setup lines down into the 'if' statement where you dispatch the event, so then we don't have an event created that may never be used if something goes wrong.
What?!? Why?
Was moving the config file necessary? I'm having a bad feeling about this :8ball:
I would move this method out of REST layer (into NexusEMailer?) to make it reusable.... it was totally wrong to have it here in 1st place
I had planned on using it for analysis of inference convergence.

I should likely have added this commit to #6. I'll close this one and add there.
It seems to make more sense to have `processKey` still take a `List[String]` and have the caller do the flattening
docs about what this does?
You're right.  Honestly, it was a bit of a quick-hack.
I'm not sure what you're all getting at. I tried the following, but it doesn't work like I'd hoped. I also may be missing your meaning completely.

```julia
function idump(fn::Function, io::IOStream, x, n::Int, indent)
    T = typeof(x)
    print(io, T, " ")
    if isa(T, CompositeKind)
        println(io)
        if n > 0 
            for field in T.names
                if field != symbol("")    # prevents segfault if symbol is blank
                    print(io, indent, "  ", field, ": ")
                    try
                        fn(io, getfield(x, field), n - 1, strcat(indent, "  "))
                    catch
                        println(io)
                    end
                end
            end
        end
    else
        println(io, x)
    end
end
function idump(fn::Function, io::IOStream, x::Array{Any}, n::Int, indent)
    println("Array($(eltype(x)),$(size(x)))")
    if n > 0 
        for i in 1:min(10, length(x))
            print(io, indent, "  ", i, ": ")
            fn(io, x[i], n - 1, strcat(indent, "  "))
        end
    end
end
idump(io::IOStream, x, n::Int, indent) = idump(idump, io, x, n, indent)
idump(io::IOStream, x::Array{Any}, n::Int, indent) = idump(idump, io, x, n, indent)
idump(io::IOStream, x::AbstractKind, n::Int, indent) = println(io, typeof(x), " ", x)
idump(io::IOStream, x::Symbol, n::Int, indent) = println(io, typeof(x), " ", x)
idump(io::IOStream, x::Function, n::Int, indent) = println(io, x)
idump(io::IOStream, x::Array, n::Int, indent) = println(io, "Array($(eltype(x)),$(size(x)))", " ", x[1:min(4,length(x))])
# defaults:
idump(io::IOStream, x) = idump(io, x, 5, "")  # default is 5 levels
idump(io::IOStream, x, n::Int) = idump(io, x, n, "")
idump(args...) = idump(OUTPUT_STREAM::IOStream, args...)

# Here are methods specifically for dump:
dump(io::IOStream, args...) = idump(io::IOStream, args...)

dump(io::IOStream, x) = dump(io, x, 5, "")  # default is 5 levels
dump(io::IOStream, x, n::Int) = dump(io, x, n, "")
dump(args...) = dump(OUTPUT_STREAM::IOStream, args...)

dump(io::IOStream, x::String, n::Int, indent) = println(io, typeof(x), " \"", x, "\"")
dump(io::IOStream, x::Type, n::Int, indent) = println(io, typeof(x), " ", x)
function dump(io::IOStream, x::Dict, n::Int, indent)
    println(typeof(x), " len ", length(x))
    if n > 0 
        i = 1
        for (k,v) in x
            print(io, indent, "  ", k, ": ")
            dump(io, v, n - 1, strcat(indent, "  "))
            if i > 10
                break
            end
            i += 1
        end
    end
end
dump(io::IOStream, x, n::Int, indent) = idump(dump, io, x, n, indent)
dump(io::IOStream, x::Array{Any}, n::Int, indent) = idump(dump, io, x, n, indent)
```
The following doesn't work as expected:
```julia
julia> dump([1,2])
Array{Int64,1} 
```

I think I like the macro form better.
You should use `||` here instead of `or`.
I would prefer this style:

        batch_add_columns(self.printer,
            'errata',
            'products',
            'packages',
            'repositories',
             ....

but its my personal view. Its easier to work with it. Not sure if pylint would like it.

ACK
            multiline=True, show_with=printer.VerboseStrategy)
This can't be right inside of the for loop. You're counting the number of total heartbeats not beat per connection.
`lodash: '../../../aura/lib/lodash',` ?
Can you not use the `onclick` here and bind the entire ConversationListView div to handle the click instead?
No problem.  I added other tests for that stuff [here](https://github.com/apache/libcloud/pull/72/files#L1R625)
Instead we should add each line to a list, and at the end write `'\n'.join(lines)`.  This isn't even so much about pedantry over the implementation of string concatenation so much as it is about simplicity and avoiding the need to use `os.linesep` (Python will convert `\n` to the appropriate separator when writing to a text file).
@demitri - I agree, but I'll also clarify what I originally meant is that if users type ``Coordinates(...)`` it will not work. They have to type ``from astropy.coordinates import Coordinates`` or something like that first, or the examples won't work.
I agree, the table of contents will look very busy if each line contains 'Documentation'
Consider using `Repairs = State#state.repairs` to match the syntax in the following handle_calls?
What happens here? Is this text to speech?
If you're going all the way to change it, please also make it a const instead of a magic value.
Why "pszBuffer"?
0.7.0 is signed by Gavin. Is that not a stable release?
I considered including installation of the key in the script, but I figured it was preferable to let the users install these by themselves. But now that you point it out, I'm not sure why the script shouldn't just install if it reaches the line above.

Where are those PGP keys in git that you mention? Also, I'd have to know who signs what with which keys in order to know how to use them.

It would be better if all the heavily involved developers sign the executables. That way it'd be even harder for an attacker to somehow get past this (by getting hold of Gavin's key, for example).
Why not make this a separate test and use `@expectedException`.
Previously if $cacade was not true lines 2444-2446 would still run, with these changes they won't.
To be consistent with the way we document i18n-related config option in other plugins:

```
The translated tool help html text (i18n).
```

This comment applies to the other i18n options.
please validate creator against LDAP
Please remove this. Files specific to your IDE should be ignored locally. See https://help.github.com/articles/ignoring-files for help
You're registering dummy twice
hmmmmm.. I don't think so, since both are strings.  I'll test and confirm.
We can get rid of `with_statement` here - we already dropped Python 2.5 support, and it's native in 2.6+.
lets not include dependencies on evaluate in mw.EmbedPlayer, instead just check this uiConf property in your liveStreamDVRPlugin ( like all the other uiConf plugins )
The function is `giterr_last()` nowadays.
Can you set this to 4096? I see no reason whatsoever to reuse `GIT_PATH_MAX` in this context.
Add example of use too please
why?

Also: inline JS --
Same down here, don't mix "import" and "from" lines
These are just titles for the KB articles that will get created to populate the page. This is a little screwy now that we have so many landing pages. Not sure what a good solution is though.
+1 I'll make the change.
Why is that appearing twice?
On Fri, Dec 21, 2012 at 04:31:10PM -0800, Rob Hudson wrote:
>do you need to worry about `'.'` or `'..'`?

If there's no / it shouldn't matter.

-- 
Kris Maglione

An organisation that treats its programmers as morons will soon have
programmers that are willing and able to act like morons only.
	--Bjarne Stroustrup
The presence of this implies that carrying the ConnectionID in the network stream is redundant in the first place! The originator is inferred here from the source, and connectionId is used only as a redundant check. To optimize the network byte stream, the whole connectionId can be removed, avoiding being hit by issue #471 as well.
Who converts uploads?
That's true, but the `FileResource` class use `<=`. 
So i would suggest, that we use `<=` here, too.
why commenting it ? you should remove old code instead (the versionning is already done by git)
Is there a reason for you to use `~` instead of `/` for delimiting your Regex ?
I understand you could use what you want, but `/` is a bit more standard no?

Also, why is there a `@` in front of it? I don’t recall [preg_replace](http://sk.php.net/preg_replace) can throw warnings. Invalid [flags](http://php.net/manual/en/reference.pcre.pattern.modifiers.php) can, but `u` is pretty standard.
I'm not sure who to make this as if you want the cache to be managed by the code then you need to get the cache instance but if you already know your instance name then there's no point in returning it, right? Or should I make it return it just for the sake of consistency?
little trick you'll find elsewhere in this codebase:

    if e.errno != errno.ENOENT:
        raise e
just do `[NSMutableArray array]` here so we don't have to change if the json sends back more pages
Sure, I'm just saying, call it MAX_LOOPS or something, so it's clear to the next person who works on the code what it's purpose is.
Die Signup-Seite aus dem UserBundle ist jetzt unsere neue Landingpage!
Is this right? What is `u`?
Maybe I misunderstood you. Do you suggest that it should be possible to relate more than one library?
please do NOT use inline styles, add it to the proper css
If you're referring to assigning `options_errors`, it does get used in the next line as well (I return the list of invalid keys as part of the error).  No reason we can't duplicate the calculation (validation failures are pretty unlikely), but it will raise the complexity on the #{} interpolation in the raise call even more.

If you're referring to using `unless` as flow-control instead of as a modifier, I wobbled back and forth.  Using it as a modifier made the `raise` line even longer (unless I wrapped it) and harder to understand.
Is the group properly serialized here?
Initially `isRouting` is true. We should probably state when the value is set, clearly.
result.get(FAILURE_DESCRIPTION) will never return null.
Would it be clearer and faster to just subtract out element paddingTop and paddingBottom?
We should never duplicate the logic of the base `_setOptions()`. This whole function should be simplified to something like:

```js
this._super( options );
if ( "value" in options || "max" in options ) {
    this.options.value = this._constrainedValue( value );
    this._refershValue();
```

And then `_setOption()` should only need to handle setting max.
It’s more a really small optimization. FXMLRPC reuses client instances. So you save a few cycles by only doing the general setopt calls once.
jquery has .is(":visible")
nit: last_hour -> lastHour
nit: add var frame = frames[origin] and use that
Guys, if you have another chance to fix relavant bug, please put all z-index rule in apps/system/style/zindex.css in suitable layer.
Put them everywhere makes z-index management hard, as it was before. Thanks.
yep, that has to be fixed
if (finalCandidates.length > _maxSuggestions) finalCandidates.length = _maxSuggestions 

to avoid creating a new array
OK I just saw that, sorry for the stupid question.
Because the screen flashing problem since M2 -- we want to update the screen before the screen is really turned off, so we won't see the previous screen when the screen is turned back on.
(Simple checkbox could be the same as the one in Settings)
Hrmm... I'm not sure about this part. It makes it inconsistent with handling of this._options.xoauth.
seeing which proctypes are available for an app ... just confused why it was removed? 
The new structure does actually make it easier to list I would say -- although it should be moved to `app list` then probably as proc types are not bound to apps anymore
put it here near where I use it. Will put it at the top if you insist.
Previous two lines are a bit long. Please break them up.
check for hasOwnProperty in all for in loops
open PR with `Darwin|*BSD)` - I will merge it for master, so it will be available in 1.17 - there is nothing holding us back to not support all the BSD ... maybe limited support - but still always better then nothing ;)
Private in the sense they should only be used internally. They are public for testing purposes which was the solution you proposed to me in a previous conversation.
Just remembered, `this.invoke('setZIndex', zIndex)` should do the same as this loop.
When do you get a `<center>` error?
This would proibably be cleaner if done using [text_field_tag](http://apidock.com/rails/v3.2.8/ActionView/Helpers/FormTagHelper/text_field_tag)
I think this needs to be wrapped in if remote.install_server_is_configured() ?
This looks like it will cause an infinite loop, since this will send SIGUSR1 to itself, which will call this function ...

There is already code to shutdown the master cleanly: `master.botmaster.cleanShutdown`. There is a but on the web status that does it (on the root page), so looking in `buildbot.status.web.root`, pointed me at `master.status.cleanShutdown`, which calls `master.botmaster.cleanShutdown`.
I'm very tempted to say that this function belongs in the web status. Particularly with this special case.
This is kinda yucky -- I don't like functions with class-like names.  I think just a rename would be an adequate fix, though -- there's no reason for a full subclass here.