labofsahil/github-event-dataset-2012 huggingface dataset:
omnibox-17

Adds an 'omnibox' that is triggered when the user types `fi SPACE` into the location bar. If you're logged in, it will recognize if you're typing your username or @username and suggest that to you. Otherwise you can just jump directly to the object in Fluidinfo for whatever it is you're typing.

Note that the "Search" part of the text "Fluidinfo Search" that appears in the suggestions cannot be changed by us. Google apparently never heard of a Thing Engine.

Fixes #17
NXCM-3733: P2 event inspector fails to close JarFiles, causing locked files on Windows

at least until GC finalization occurs...
German translation

Hello i have made a german translation for you plugin. I hope you can use it.

Best regards Denny Schäfer
Redmine API changed:

to_utf8_for_attachments => Redmine::CodesetUtil.to_utf8_by_setting
Fixed alignment of dt elements

Sometimes these were misaligned in Chrome; think I've fixed it and have checked in Chrome, FF and IE.
fix typo

...es
Overhaul

Finished the first overhaul. 

Also, as its the first pull request, gogogo and fix it! Wheeeeeee :)
i-am-a-horse-in-the-land-of-booleans palautukset
some improvements to example code
fixed warning if no queries are added

```
ErrorException: Warning: Invalid argument supplied for foreach() in .../sensiolabs/doctrine-query-statistics-bundle/SensioLabs/DoctrineQueryStatisticsBundle/Analyzer/Query.php line 25
```
Update site/pl-PL.com_hwdmediashare.ini

Hi,
I'm trying this service in order to translate Polish language file.

Regards,
Krzysztof
Found a bug in the adding of roles to users
Add question about following bugs

There are always some long-living bugs in webkit/gecko/jquery/phonegap/etc that are haunting you from project to project. Of course you're subscribed to them in their bug tracker and shiver with excitement everytime an issue update notification hits your inbox. Full of hope, you rush to open that email just to read another cry from some outraged developer like yourself :)

Back to the real world, interesting to bring up some of that drama on a job interview and see the passion in candidate's eyes)
Condensing of sanitizing code as per @GaryJones

Initial testing is positive.
Added my name, yo!

Added my name. Pull that ish!
cleanup

You accidentally left your private key in the repo. I also added a gitignore for your sublime files and other sinatra files that should be ignored.

Nice project!
Shouldn't this be tsFlags.Type != cmTarget::SourceFileTypeNormal?
This is a check box to select a line and there is no explanatory text in the GUI. I don't know any GUI that has a text for this kind of check box. Other export plug-ins also do not have labels here.
a beautiful one now!
I can, however its just trimming trailing whitespace which my editor does
automatically.

If you still want it gone I will sort it so that's not touched.
On Feb 7, 2012 2:47 PM, "Gonzalo Paniagua Javier" <
reply@reply.github.com>
wrote:

> Lots of whitespace changes. Can you get rid of those?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ukplc/mono/commit/739bbcc3f94ffbe27dd0580024e7d048003b3417#commitcomment-941230
>
Unfortunately it's not our code that blocks the DOM from loading. It's the external stuff we have to add like, registration and analytics. They all insert sync script tags into the body. Getting them to change that stuff is like an act of congress... painful I know.

I don't mind using my own domReady plugin. Not really that big of a deal. I've been changing it since day one. I think I might eventually stop using it. I'm toying with the idea of modifying the dom plugin so it can resolve once a specific node is available. Either using DOMSubtreeModified or adding a `require(['wire/dom'], function(dom) {dom.ready('my-id')});` to the bottom of each of my container elements. This way all my widgets get instantiated before the DOM is completely loaded. Who knows if it's a good idea or not ;)

Raul
when I’m in that mood, strange things may happen: I created the ticket nonetheless ;-)
wrt every call: yes, I agree that it should be handled in the pool implementation, so the hook should stay as is here. Down in the default impl I think it should only resize on every Nth message, based on the elasticity parameters. Otherwise one flip of the AtomicBoolean per message send would be too costly. I propose an atomic counter with a threshold, and the reduction is done by the one doing the resizing; anybody else just always increments.
Dávat své třídy do namespace Nette\ je nežádoucí, takže bych to asi neupravoval (tj. nepodporoval).
@djones [Yes](https://github.com/rails/rails/commit/b83965785db1eec019edf1fc272b1aa393e6dc57#commitcomment-1040449)
Ah good to know this from github. I always have problems with its formatting. :D
Is this just to trigger the `clean`ing? Seems weird not to see this as a conditional.
Opps, I wrote "label" and meant "input". I was just pointing out the lack of triple-equals.
I'm not sure this is the best way to do this. I'm going to review this a little and see if I can start by adding some new event names. 

After that, we should probably setup some unit tests for `Hook.stop` and `Hook.kill` type events, they seems pretty important.
this is what I get under jruby 1.6.5.1

~/ #>jruby tmp.rb 
  
  [double, single]

  +--------+----------+
  | tach   | total    |
  +--------+----------+
  | single | 3.419000 |
  +--------+----------+
  | double | 4.314000 |
  +--------+----------+
The `(match? regex string)` regex used in this patch expands to `string.match(regex)`. Check out these lines in macros.sibilant:

```(defmacro match? (regexp string)
  (macros.send string 'match regexp))
```

Is there an advantage to using `/regex/.test('string')` over `'string'.match(/regex/)` as was done in this patch? I've been away from a
computer for a while now, but I seem to remember them being equivalent since we don't need the captures or matches.
I pushed the fix to all comments(this one and the one about assert). If there is no more new thing, I think you can merge it.
I have never pushed this patch to public, because it was unfinished at all. If Neo_mat took it and reworked it, how much I am still connected to it? If I wanted to make this patch available to public, I would post it on forums or elsewhere, but I didn't. Not to notice that I did that thing like 1 and half year ago on my local test server, pushing it to repo, where some people had access. So that's not my problem it appeared in ArcEmu, I seriously almost don't even remember that I pushed it to any repo and I was surpised that Neo_mat found it somewhere. So please ommit my nickname in all future commits about this, as I don't have anything to do with this.
Sorry about that ! Forget to remove it... :( 
Changes pushed in 90eab0ea435ed988bafb521bdf088632a2a630a9
Don't forget the case when a Jail is running on physical hardware and not under kvm/qemu. Something vaguely like this:

    If grains['virtual'] = 'physical' and jail:
        grains['virtual'] = 'jail'
for completeness, can you add tensor.transpose(x2, 0, 1) and tensor.transpose(x3, 0,2,1)?
Had to put it in a non-anonymous function for the ```remove``` to work. It was pointing to ```fn``` before.
Yeah, that accidental code reuse seems like the usual case for this method, we can leave it.
In the simulation, we call this probe ratio.  Fine to chance it here, as long as there's a compelling reason to do so...
The brackets in `X=[...]` are not necessary; they expand the range into a dense vector.
Ah, I didn't understand the question at first:
Sure, I'll remove the arrows \< \> gettext call.
Should this be get() or getLive()?

>oldChunk.observers.get().contains(p)
Hm, okay.
This is a read-only attribute. It's just the same as the last part of the address.
This code doesn't compile. The type of the first parameter is misspelled.
I think in order for this to work you also need to put the lengthened links at the end, eg:

.. _GH787: https://github.com/pydata/pandas/issues/787

Thanks for doing though!
This line should probably read `alias_method :to_s, :source`.
Why is this commented out?  And anyway it should use C89-style comments ;)
Same question... what is our rule to indent on various lines?
note also that this week nzmatrix added maxage as a required param now... you cant leave it unset.. use 0 for default. adding maxage resolved a lot of issues with sb on grabbing stuff from nzbmatrix's feeds.
http://nzbmatrix.com/forums.php?action=viewtopic&topicid=41452&page=4
-- 'i added maxage and no more issues (maxage cannot be null)'
Hi, no sql? I'm trying to apply these scripts to version 3.3.5, but I need the sql.
Some hard numbers for forking and execing the `true` program on my machine:

- shell: 7ms
- julia before this commit: 50-60ms
- julia after this commit: 20ms
- julia calling exec right after the fork: 16ms
- julia just calling exit right after the fork: 7ms
We didn't think that Wave or Buzz were going away either...
Im pretty sure that you have it broken since https://github.com/mangos/mangos/commit/bf7cdae6655d100bb479347b02dccc418893d231 , if you had dual spec before that.
I've just opened #1722 to clean this up for sliders.. But, note that code like this is pretty wide spread in Maqetta (which is sort of what I followed). So, perhaps you'll want other issues to clean it up elsewhere.
can you remove [this one too](https://github.com/mozilla/zamboni/blob/5ecfb6e/mkt/settings.py#L90)? and feel free to `rm` that ish!
If it wasn't a function-scoped static (which it sounds like, since you said
"static via anonymous namespace") then it sounds like it was this...?
http://www.parashift.com/c++-faq-lite/ctors.html#faq-10.14

The fix to that problem is to put it inside a function, so the initializer
doesn't run on module load but only the first time the function runs. Since
these are inside functions they ought to be OK.

Let me know if that makes you feel better...

On Fri, Feb 17, 2012 at 5:35 PM, JJ Allaire <
reply@reply.github.com
> wrote:

> You might think this is way too paranoid, but I once ran into trouble with
> statics that had constructor arguments (this was on Ubuntu). Here was the
> comment I made (this is over in the ServerGoogleDocs.cpp):
>
> // initialize server (create on the heap so it survives after this call)
>  // note that we attempted to make this static via anonymous namespace
>  // but on ubuntu server this resulted in the acceptor initialization
> failing
>  // (presumably because the presence of a constructor argument on an
>  // automatically initialized static gave the compiler problems?)
>
> As a result when I need to have a bunch of static regex's I use the
> pattern that you can see in RTokenizer.cpp.  I realize that this is verging
> on voodoo paranoia but (a) I spent a lot of time hunting down the cause of
> that first bug; and (b) this is C++ after all ;-)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rstudio/rstudio/commit/e87fe766cfb2f27c9849da9dcb67e63c5a526624#commitcomment-980487
>
The uncompressed version is in classes/Popup.js.
lets check for == 1, == 0, and < 0
закрузок -> загрузок
I think we need to keep the yaml ones backward around or we'll break backward compatibility with the old serialization formats.
I'd say that in to_yaml, we should make the change, but in from_yaml keep both around so the old format still works
do you want to check that `request.user` exists?
The name sucks I admit.
The existing code does that, I'm not sure what this is being used for so limiting query length seemed prudent. There's nothing to stop someone doing multiple queries, eg 2010, 2011, 2012
Is this complexity due to the current design, or is inherent? In any case, it should probably be explained (or a link added to where it is explained).
Nope, good catch! Thanks.
On Mon, Jan 30, 2012 at 8:49 PM, Gao,Yan
<reply@reply.github.com>
wrote:
> Thanks!
>
> The 60s was due to the change:
>
> https://github.com/ClusterLabs/pacemaker/commit/3f821aa7c2278596c152c0b23991e2551f66b068
>
> Anyway, it has been consistent with the meta-data in glue/lib/stonith/main.c , except the additional monitor "start-delay".
>
> Do you think the monitor "start-delay" is needed/appropriate?

Nope :-)
I just removed it here.  Not sure how that got in.

>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gao-yan/pacemaker/commit/12386648c3d194a0a90d9284aea56c6121dfb6b0#commitcomment-913642
@lloyd - addressCache is initialized before anything else in that file is run.  It is not even possible to run any functions that user.js exposes until addressCache is initialized because BrowserID.User will not yet be defined.
We should document this feature somewhere.
There are no hints in the pageaddvar-plugin nor in PageUtil.php nor in JCSSUtil.php.
@christophercoy thanks for the commit!

@hasghari can you review these changes?  @christophercoy can you do the same with @hasghari changes?

It'd also be good to do all work in separate branches for review before we merge into master

Thanks!
I don't have this `Developers` in my `groups`
why do you have to order by id?
was exactly what you mentioned with the wrapping - particularly for the media stuff.
Thanks for getting this feature in!

What would be involved in flagging things in such a way that the reporters could report details/metrics of "skipped" specs?
This quest still not working man !
FYI: We have a 'useravatar'-plugin in the core. This plugin is able to use gravatars if you activate this feature in the Users' module settings.

I think that the webmaster should decide if avatars or gravatars should be used. And not each module individually.
I prefer to be descriptive here. Anyone evaluating Walker to implement his custom Walker would benefit of having a more descriptive scenario.
Yeah, tests... :P

Your work is looking good by the way. When you feel it's up to scratch, squash your commits into another branch (add me as a remote, check out a new branch based off smileychris/master, merge --squash master) then pull request.
краша не будет в R2, так как в float Unit::GetCombatDistance( const Unit* target ) const стоит доп.проверка, но проверка в скриптах также должна быть
This is a very nasty time to merge. If you did in fact add streams to your local repo, then you may integrate them with this revision by creating a traits structure that specifies how to marshall the data from your stream to Lush. The other boilerplate are the Lush wrappers and adding various things to compile. However, this structure makes everything a lot simpler and prevents the RosClient class from bloating excessively.
It seems that the problem is with onCreateView's setup. I'm trying to reuse the fragment from the previous orientation but that doesn't seem to work out. Bummer.
you don't need to wrap appendView in a function call if you remove the args, since appendView can already access view and stateManager
mm, I tried pull but the sagepay files didn't come down...

On 23 January 2012 19:16, Michael McAndrew <
reply@reply.github.com
> wrote:

> those changes are independent so git will be able to handle the merging.
> you should be able to pull and the push and all should be ok - i'll get
> peter check and put those files back if necessary.
>
> On 23 January 2012 19:11, Ed Burton <
> reply@reply.github.com
> > wrote:
>
> > I think Peter (perhaps) added all the SagePay files to the repository, my
> > push promptly removed them all again, how embarrassing!
> >
> > (All I've been changing lately is LBChooks.module)
> >
> > On 23 January 2012 18:27, Michael McAndrew <
> > reply@reply.github.com
> > > wrote:
> >
> > > no probs - what did you mean/what to do? I'll fix it and let you know
> > what
> > > I did.
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> > >
> >
> https://github.com/michaelmcandrew/lbc/commit/0e3e895a413e76e47d0e623c1127e3e2592ab7f7#commitcomment-893117
> > >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/michaelmcandrew/lbc/commit/0e3e895a413e76e47d0e623c1127e3e2592ab7f7#commitcomment-893297
> >
>
>
>
> --
> Michael McAndrew
>
> tel: 020 7183 5828
> mob: 07817 802299
>
> Third Sector Design Ltd.
> http://thirdsectordesign.org
>
> *CiviCRM meet-ups, trainings, and code sprints coming up in London this
> February! http://bit.ly/CiviCRM-Feb-London*
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/michaelmcandrew/lbc/commit/0e3e895a413e76e47d0e623c1127e3e2592ab7f7#commitcomment-893313
>
Changing bad code instead of real refactoring is for :womens:.
The && should be &amp;&amp;
Thanks for tackling this. First of all: I think this code should go into a new module `Snap.Util.HttpAuthentication' rather than getting lumped into `Snap.Core`.

Secondly: I'm uncomfortable with providing basic authentication out of the box without any disclaimers/warning labels/restrictions on usage. Could you please change this API to:

    withBasicAuthentication :: MonadSnap m => ((ByteString, ByteString) -> m a) -> m a

and check that `rqIsSecure` is `True` within the handler. I really don't think we want to make it easy for people to send cleartext passwords out over the internet in our public API. A suitable disclaimer in the comment (that we will only authenticate secure connections using this method).

If you really feel that cleartext basic authentication is something that needs to be in the API, then add the function as `withUnsafeBasicAuthentication` with a big disclaimer in the docstring. Personally I think we're better off just omitting it.

I think the thing we all would really like here (and I think hvr mentioned it also) is RFC 2617 digest authentication. Would you be interested in tackling that?
Scala has the concept of implicit conversion.  The you can see the converters in converters.scala.  The converters come into play as long as they are imported into the code.
Is this a debugging thing? (just seems odd to see this)
Hi Tom,

I remember you were missing for some kind of overview at GIT. Was it 
something like this one: 
http://www.vogella.de/articles/EGit/article.html#using_history

Or was that a simply file comparison for different file versions?

Best regards,
Marcel

Am 26.02.2012 15:18, schrieb Thomas White:
> This isn't really a 75% memory saving.  Rather, it saves 75% of the memory coming from variables - so may approach 75% (probably never more than about 50%) in very long surveys, but may only be a few % in short surveys
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/LimeSurvey/LimeSurvey/commit/b93d63b9fd0db243a7b76ede9719b68f660c13d2#commitcomment-1012377
>
>
Will you put these docs somewhere like readthedocs website?
Список генерится довольно медленно, может стоит подумать о статической загрузке в innerHTML.
OK, I think this should test the 3 media query versions I found if devicePixelRatio isn't there. Not thrilled with it, but it seems like it works in my tests. Waiting on someone to try it for reals in FF.
When we are at it adding an option to omit a room, shouldn't we just enable the option to emit several rooms.

    talker.broadcast = function (content, excludes) {
        excludes = excludes || [];
I'm using it in a staging env to override the robots rules (env runs against a copy of production db)
It sounds great, then you need to call it correctly from the server!
well, here it is:
https://github.com/DerTeufel/samsung-kernel-aries/commit/69067d778a56a09760316767103c188020af8586

i suppose you already know what i was talking about ;)
We may need a separate coregen directory for UmTRX because it's a different FPGA family.  Will test...
Why aren't you implementing <=> instead of < > <= >= ?
Yeah, we should, but  I was going to wait until your changes got merged,
then update the uninstall command.

On Fri, Jan 13, 2012 at 12:00 PM, Matt Robinson <
reply@reply.github.com
> wrote:

> Kelsey, we mentioned earlier that we should check for local changes before
> deleting modules.  Is that still the case?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/kelseyhightower/puppet/commit/b350f87fac47db58096eae1c21da2825cab3f76b#commitcomment-865451
>



-- 
Kelsey Hightower
Solutions Engineer
Puppet Labs
1-678-471-9501
Hi

When I Initialise like 

```$(".hovercard").tipsyHoverCard({live:true , trigger : 'hover'});   ```

The hovercard doesnt visible when mouse is taken over the hovercard.
It's not documented to, but in some cases it does. This gets corrected
  original:http://www.karelia.com/res/images//stuff/../../img.png
  stdized:http://www.karelia.com/res/images/img.png
while this does not
  original:http://www.karelia.com/res/images//stuff/img.png
  stdized:http://www.karelia.com/res/images//stuff/img.png
Hmm could you describe how it is broken? I.e. what the desired behavior is vs. the current behavior?

That line is there to prevent JS in HTML files look the same as JS in `.js` files -- that is, unless there is a keyword or special syntax construct involved, Javascript should not be highlighted specially.
As an additional bonus, this automatically works with the new config layout by adding "theme_id" to your config, ala:

    [default]
    api_key="xxxxxxxxxxxxxxxxxxxxxx"
    password="yyyyyyyyyyyyyyyyyyyyyy"
    store="shopname.myshopify.com"
    theme_id="123455667"`
Could you please use the "Is it stable" thread on the forums? Thank you.
Incorrect usage of static variables.
hmm, again, without being overly picky or wanted to start a big discussion:  afaik the extra lines in `scale.py` (16–17) and `scaling.py` (121) are not required for pep8.  apart from that i agree, but have started to try to leave things (which don't "comply" with my personal taste) mostly as they are, i.e. keep the changes at a minimum.  at times i even mimic the existing style re white space, parentheses etc as i find consistency should win over personal taste…
Right. Thanks for the tip.
ok, i'll do the integration via email for now. you could use git to update the files after they have been published, if need be.
yes, you can fork the repository and add the strings files to the appropriate res/values-XX (where XX is the country code).
then push to your repository and submit a pull request.

http://help.github.com/send-pull-requests/
True. Just as precaution because the variable is used in a block .
RTFM: ...into a newly allocated memory... -- aka where is the deallocation?
Since there is no tag yet -- Oliver, everything is in your hands -- unveil 5 bugs and steam ahead ! ;-)

Just kidding -- great job all of you guys!
Missing 2 SQL:

old/4.0.6a/characters/2011_12_30_00_characters_guild_member.sql (Server startup error)
old/4.0.6a/characters/2011_12_31_03_guild_xp_for_level.sql

Except that, seem to work very fine, I'm gonna test it on a my big database ;)
I'm not sure I agree with this. If I diff a table with an empty one, I should get a diff saying all rows are missing. Try diffing a file to an empty file to see what I mean.
If this is accepted, recommend changing the comment to read (something like):

    Use PHP methods htmlspecialchars or htmlentities, instead.
just pour essai
epicness(dtdesign) = lim x -> ∞ 1/x
Ah yes, you were right on this on. In my sleepy stupor I misread how we were supposed to implement this. :)
kernel use 4MB~5 to operate, so i have lowered to 8, just in case.
Format:  Current code uses 4 spaces for tabbing.  Please be consistent with this convention
I think the right fix is to make sure that rest-framework serves up the css for the login form itself, without relying on the admin.
(Then uses get_static_prefix in the template.)
IE. Do the same as [renderer.html](https://github.com/tomchristie/django-rest-framework/blob/master/djangorestframework/templates/renderer.html)
Why did you removed that binding? Annotated source code is outdated -> it confuse me.
Typo! I switched it to a See
wowfurity read here http://www.strawberry-pr0jcts.com/index.php?/topic/137-armory-issues/
Couple of questions..
1) Can you tell me which ticket this change is in aid of?
2) What was failing on 1.9? We were passing all our tests, and I was under the impression it was working, after I patched up Dataflow to be encoding aware
3) If something was broken, we need to develop a test for it, especially if it changes the way deploys execute
class in p element and the one specified in 326 are different, is it intentional?
**First** one: [commit](https://github.com/CyanogenMod/android_device_commtiva_z71/commit/1af2afa6943c51f85e3d2560e9f9dc8720e37ae5#libaudio)

**Second** one: this value related only to TTY output devices ([link](https://github.com/CyanogenMod/android_device_lge_p500/blob/e831dddc70729c84f6ee935d48f20819cb0eae9d/libaudio/AudioHardware.cpp#L1327)) and nothing more.
Josh:
Re 1. let me know when we want to discuss unification.
Re 2. hmm need to test. works for me.
Re 3. Definitely! I will update in next commit.
@shucky Symfony 2.0.9 provides a class in the new location too. So simply upgrade to 2.0.9 and it will work
OK, not going to merge this for now - it needs to be solved in a larger context. The back links are all over the place, so its unlikely to fall through the cracks.
Is there a "delay" to update the files after commit?
You are forcing the pattern to be {application}, I think it would not be hard to make this an option because I can see some case where we'd want to integrate the plugin in an application that uses another pattern

otherwise LGTM
I don't know how familiar you are with GitHub but
if you go into your terminal (and in the folder that is for gamify) and
then type
 git pull origin master
it should download the latest version (including changes Gina's made).  If
you have any problems, Gina should be able to help.
M.E.

On Wed, Feb 1, 2012 at 10:45 AM, spirineta <
reply@reply.github.com
> wrote:

> TY M.E., where do I access this?
>
>
>
> On Wed, Feb 1, 2012 at 10:40 AM, mecathcart
> <reply@reply.github.com>
> wrote:
> > I think the little guys are cute! But if you have cuter guys send them
> > along! What do you think of the buttons and the menu? (the title
> placement
> > is bugging me but I'll figure it out). If you like it, I'll work on
> making
> > a similar structure for the other pages.
> > M.E.
> >
> > On Tue, Jan 31, 2012 at 12:31 PM, spirineta <
> > reply@reply.github.com
> >> wrote:
> >
> >> Thats' exactly the spirit. I also noticed that when the drawing is small
> >> (thumbnail size, which is what the logo should be anyway), the hearts
> and
> >> ears are not visible at all. Your ears and hearts are cute, I'm just not
> >> happy with the little guys to begin with, I'll redraw them tomorrow!
> >>
> >> ---
> >> Reply to this email directly or view it on GitHub:
> >>
> >>
> https://github.com/AccentsResearchLab/GamifyPsycholinguisticsExperiments/commit/13d7d4e0db216aa835309db65a4c12853b0b1339#commitcomment-919143
> >>
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/AccentsResearchLab/GamifyPsycholinguisticsExperiments/commit/13d7d4e0db216aa835309db65a4c12853b0b1339#commitcomment-922762
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/AccentsResearchLab/GamifyPsycholinguisticsExperiments/commit/13d7d4e0db216aa835309db65a4c12853b0b1339#commitcomment-922772
>
What if one or more of these fail?
Since you're calculating the 4 bytes by hand anyway, is it faster to just set them directly using `[offset]` access instead of `writeUInt32BE`?
I would be happy to see move_wheel_files() get its paths differently. There are a lot more possible paths but the basic ones can come from the old API.
After discuss we decide to use generateSourceCodes() to name it.
was there a reason to remove this version?
As of the recent merges, pycuda is also required.  But it needs to be quite recent, (at least for the linear algebra stuff to work).  So perhaps just recommend it, if it even exists as a package.

Some of these are not hard dependencies too right?

python-mako is only required for linear algebra operators with pyopencl and so forth.
? It still uses FrozenDicts
:-) thanks, at least you are making me think which is great.
This method only returns Unit, not Any.
cause I'm craaazy like that! ;-) I'll change it.
ah, did not realize. thanks!
If you scrap AMQPActor, fold class Settings into object Settings and rename it to AMQP you can hide the:

context.actorOf(Props(new FaultTolerantConnectionActor(cr.connectionParameters)), "amqp-" + UUID.randomUUID.toString)

Take a look at the TypedActor extension for more inspiration.
You don't need to do AcceptWithOptions, just instead of having the empty Seq as a default-parameter, create a new constructor that has 2 parameters and that delegates the empty seq to the 3-param constructor
Nice check. Should definitely been here all along.
expectMsgType is nice!

On Fri, Sep 14, 2012 at 5:14 PM, Roland Kuhn <notifications@github.com>wrote:

> In
> akka-actor-tests/src/test/scala/akka/routing/ConsistentHashingRouterSpec.scala:
>
> > +@org.junit.runner.RunWith(classOf[org.scalatest.junit.JUnitRunner])
> > +class ConsistentHashingRouterSpec extends AkkaSpec(ConsistentHashingRouterSpec.config) with DefaultTimeout with ImplicitSender {
> > +  import akka.routing.ConsistentHashingRouterSpec._
> > +  implicit val ec = system.dispatcher
> > +
> > +  val router1 = system.actorOf(Props[Echo].withRouter(ConsistentHashingRouter()), "router1")
> > +
> > +  "consistent hashing router" must {
> > +    "create routees from configuration" in {
> > +      val currentRoutees = Await.result(router1 ? CurrentRoutees, remaining).asInstanceOf[RouterRoutees]
> > +      currentRoutees.routees.size must be(3)
> > +    }
> > +
> > +    "select destination based on consistentHashKey of the message" in {
> > +      router1 ! Msg("a", "A")
> > +      val destinationA = expectMsgPF(remaining) { case ref: ActorRef => ref }
>
> this should be expectMsgType[ActorRef]
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/akka/akka/pull/698/files#r1608480>.
>
>
okay, will make this smarter and then merge it in.
Spent a little more time thinking about this.... want to hear your thoughts...

I think we *need* to synchronize (ie not buffer this write) on the fact that the finish loop above is exiting so we can be sure to eliminate any possibility of a race race condition between lines 437 to 441 (which pull a message off the channel and update counters, non-atomically) and the code below that fires the goroutine to purge the `finishedMessages` channel.

If it was the last message in flight and the goroutine below won the race, it would decrement first (before line 441), pass its logic statement, and never be able to read anything off of `finishedMessages`... essentially orphaning the goroutine.

Was there a specific reason that I'm not accounting for that required it to be buffered?
Again, this is not only allowing \r to precede \n, but it changes the semantics. E.g. [\r\n] accepts *one of* \r or \n, but I think the intent is actually `\r?\n`, put together the correct RE seems to me to be:

\r?\n|.+\r?\n?

which will perform as before, only allow \r where it was not supported. Please comment.
Revisit this issue on latest code.
nice strategy √
small style fix -> if (empty))
The rest seems to be correct
s/Than/Then/
Yes, so far clojure looks better to me if that's still ok...

On Jul 3, 2012, at 10:29 AM, Matthew Rocklin wrote:

>> +#installed postgres 9.0 on mac w/ macports
>> +#Following Aron Bedra's Heroku + postgresql note:
>> +#https://devcenter.heroku.com/articles/clojure-web-application
>> +
>> +initdb pg
>> +postgres -D pg &
>> +createdb hits
>> +
>> +export DATABASE_URL=postgresql://localhost:5432/hits
>> +
>> +# ... test w/ repl:
>> +lein repl
>> +# All his examples worked for me
>> +
>> +
>> +### OLD Django stuff, disregard:
> 
> Does this mean that we're likely switching to Clojure?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hits/hits/pull/4/files#r1093340
typo : bothe -> both

also : I think we should use "runs" rather than "session" (it seems that this is less ambiguous and hopefully adopted in Xcede)
A session would mean all the runs done until the subjects go out of the scanner
Hmm. The thing is, the helper is a wrapper for the `recently_changed_packages_activity_list_html` action function, so it's named to be consistent with that. Although they differ in ending with _stream vs _list which is a shame.
don't use `os.system`, use `subprocess.call` [shortcut] or `subprocess.Popen` [more control] as it works properly for all platforms.  read the args available to subprocessm and make sure you **DO NOT** set `shell=True` :-)
This seem like a remnant that should be removed.
[Does not like.](http://travis-ci.org/Sage/fudge/builds/2921332)
You should use "production_centre.name" (note the dot) since "production_centre_name" is only meant to be assigned through the REST API and does not exist on the private model.
Not important here, but use the comma form of log instead of %.

```
log.info("Container (%s) started, OK", self.id)
```

prevents unneeded string interpolation when log levels aren't set.
Yeah. Maybe we should never kill, but only never release the token so that no more processing should be affected.
They've been defined as private "static ExecutorService xxx" and initialized in setupOmid() just as the HBase initialization is done.
I've also created ExecutorServices for bk and tso in TSOTestBase.java following the same approach as in this class.
It's probably a good idea to note here that the passphrase ends up being stored world-readable in the Nix store.
If the second process is using the same port number it will die earlier (port binded already). Using a different port number it will for sure overwrite if they have the same control socket path.

I  did not want to deal with file locks if possible but I guess I'll add an exclusive lock.
is there a more elegant way to do something like this?
Do these three methods need to be public? They seem like an implementation detail.
Style: `window.frame.origin`
Heh. This is a nice temporary fix. We should probably rewrite d3_geo_albersUsa as a raw projection, and then wrap it in d3.geo.projection.
Is it the case that the size of  `ret` was set above, so this doesn't do anything new about shape/strides, just fills with zeros?
Thanks for point out. it works just because of macro expand.  will be update soon.  thanks :)
AC_CHECK_FUNCS() should take care of that. You either have it and it's in shadow.h, or you don't. I haven't seen it done in a different way. It's not POSIX, so we might run into other implementations.
The file where this and the following functions came from had a reference to :
Lladser, Gouet, and Reeder,  "Extrapolation of Urn Models via Poissonization: Accurate Measurements of the Microbial Unknown" PLoS 2011.

Since the doc string references Theorm 2 (i), we should have the reference in here as well for completeness.
ну да, а тут - не по имени свойства а по содержимому
Typo alert `/Image/Imagine` :)


btw, this is amazing. great work. thanks for doing this
@rafaelfranca I asked about the timestamps and datetimes because the current behavior seems inconsistent. a "timestamp without time zone" is converted to a datetime a "timestamp" on the other hand is converted to a "timestamp".
I'd initialize this to nil to avoid warnings
I think it's not necessary to use `[required_keys].flatten` here, probably just like `assert_valid_keys` does would be enough:

```ruby
required_keys.flatten!
# ...
```
Glad to see that one; I never understood why DefDef was being hijacked that way.
The only check I can imagine is the one that verifies that one of the base classes of linkedClassOfClass is, in fact, a JavaLangEnumClass. But as https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/1282/consoleText shows, this might lead to cyclic reference errors.

Therefore I suggest we don't add any assertions.
In light of your PSA, perhaps we should move this to `Type#dealiasWiden: Iterator[Type]`.
The n-dimensional versions don't even exist in scipy. No idea why. These things should really be kept consistent, it's easy to do. So never mind my initial comment.
It (and other custom Warning classes) are in `__all__` because people may want to silence them via `warnings.simplefilter("ignore", category=OptimizeWarning)`. Then, it is useful to be able to import it from the public namespace.
mmm, query methods
Agree. I've had this idea, but decided to wait for comments. Will fix it ASAP.
For what it's worth, this comment is pretty useless when 'expected' is your exception name.
Add a docstring here explaining `args`.
perhaps slip that colon in there, too.
I'm not sure if this really works, because the code will be run before knitr is installed, i.e. there is no `knitr.sty` at that time.
This is almost entirely unnecessary, as the page is already being pulled with a per-page count of 20.
Fixed, ['false', 'no', 'off', 'n', 'f', '0'] are all equal, so I leave 'False' as for now.
1) it still seems like an open question
https://issues.apache.org/jira/browse/KARAF-1735

2) validation happens at run time in
/org.apache.karaf.features.core/src/main/java/org/apache/karaf/features/internal/FeaturesServiceImpl.java
public function setParent(Group $parent)

So wird bereits eine exception geschmissen, wenn du eine andere Entität als parent angeben möchtest ;)
Use memset, bzero is deprecated.
IconUploader используется еще для загрузки ContactType иконки.Размер иконки 16x16 для contact type нормальный. А ты этим изменением его ломаешь. Следовательно, нужно что-то делать...
use case.
Meant to be commented out?
Since strings are immutable in Python, be wary of the + operator :) Likewise, list.pop() can (I think) result in Python shrinking your underlying Array data structure. The least impact you can achieve here is probably by just blanking out the last item in the list, then doing ``'/'.join(mylist)``

I might be overoptimizing this, but I thought I let you know some of the Python internals.
same as the flickr comment above
Looks good, but since it's now more complicated, can you provide a brief sentence describing what this algorithm is doing so it's easier to jump in?
We do not want to fork plupload, so we will use it as-is.
It seems to me that preg_replace_callback would make this code significantly easier to read, and potentially more robust as well.
You can/should just do p.user_from.count(), no need to call all().
I believe run has been replaced with `exec`
You just broke my setup ;)

Can't you override these values in some .local.php (which you subsequently do not commit)?
If these requires line comments, then they should probably be broken out into methods
Would rather use `string.Empty`
LAB_HOME, LAB_USERNAME & LAB_PASSWORD are temp till we don't migrate to 1.9 (probably within a week). I didn't store it in util.js (in common resource folder) because it was Module specific variable. 
And thanks for pointing out the error in LAB_USERNAME, I'll update them soon.
...	 ...	
@@ -0,0 +1,114 @@
 	 1	
++
 	 2	
+/**
 	 3	
+ * Copyright 2012, Raxa
 	 4	
+ *
 	 5	
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
 	 6	
+ * use this file except in compliance with the License. You may obtain a copy of
 	 7	
+ * the License at
 	 8	
+ *
 	 9	
+ *     http://www.apache.org/licenses/LICENSE-2.0
 	 10	
+ *
 	 11	
+ * Unless required by applicable law or agreed to in writing, software
 	 12	
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 	 13	
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 	 14	
+ * License for the specific language governing permissions and limitations under
 	 15	
+ * the License.
 	 16	
+ * 
 	 17	
+ *  This is the controller of Laboratory Module which contains Lab order creation 
 	 18	
+ */
 	 19	
+
 	 20	
+Ext.define('Laboratory.controller.LabOrderCreation', {
 	 21	
+    extend: 'Ext.app.Controller',
 	 22	
+    id: 'Main',
 	 23	
+    views: ['Viewport', 'Home', 'PaperEntry1', 'PaperEntry2', 'PaperEntry3', 'PaperEntry4', 'BatchApproval', 'QueueStatus', 'ReportDelivery1', 'ReportDelivery1', 'ReportDelivery2', 'ReportDelivery3', 'ReportDelivery4', 'ReportDelivery5', 'SpecimenCollection1', 'SpecimenCollection2', 'SpecimenCollection3', 'SpecimenCollection4', 'SpecimenCollection5', 'SpecimenCollection6', 'SpecimenCollection7', 'SpecimenCollection8', 'SpecimenCollection9', 'SpecimenCollection10', 'SpecimenCollection11', 'SpecimenCollection12', 'SpecimenCollection13', 'LabOrderCreation1', 'LabOrderCreation2', 'LabOrderCreation3', 'LabOrderCreation4', 'LabOrderCreation5', 'LabOrderCreation6', 'LabOrderCreation7', 'LabOrderCreation8', 'LabOrderCreation9', 'LabOrderCreation10', 'LabOrderCreation11', 'LabOrderCreation12', 'SpecimenRegistration1', 'SpecimenRegistration2', 'SpecimenRegistration3', 'SpecimenRegistration4', 'SpecimenRegistration5', 'SpecimenRegistration6', 'SpecimenRegistration7', 'SpecimenRegistration8', 'SpecimenRegistration9', 'SpecimenRegistration10', 'SpecimenRegistration11', 'SpecimenRegistration12', 'SpecimenRegistration13', 'ResultEntry1', 'ResultEntry2', 'ResultEntry3', 'ResultEntry4', 'ResultEntry5', 'ReportApproval1', 'ReportApproval2', 'ReportApproval3', 'ReportApproval4', 'LabOrderList'],
 	 24	
+
 	 25	
+    init: function () {
 	 26	
+        LAB_HOME = 'http://openmrs.gielow.me/openmrs-1.8.4';
 	 27	
+        LAB_USERNAME = 'Admin';
 	 28	
+        LAB_PASSWORD = 'Admin123';
1	
 nathanleiby repo collab 5 minutes ago
These vars should only be defined once and referred to from a common location (e.g. utils.js). Currently, they are also defined in app/controller/PaperEntry.js. Note that in one the username is 'admin' while in the other it is 'Admin'! If this is a mistake, all the more reason to combine them in one p
Changed in a new commit.
do ->  does
return -> returns
``computing``
PS, for the release, should I put warn_type back in, have it default to None, and ignore its value except to issue a deprecation warning if it ends up being anything other than None?
поля оставляю, модели. киляю. 
окай.
This TODO seems to be still TODO. We shouldn't expose new public methods on interface objects by accident.
so, what is wrong?
Use `require('path').basename(relPath, '.' + suffix)` to strip extension
Good point.   I've pushed down the raven.contrib.transports.zeromq module into it's own package in 6cfecd327af and removed the entry in raven.contrib.transport.__init__  

I'm indifferent to where the sample client and server should stay in the source tree.  The sample server should really be fleshed out into a proper Sentry service class and removed entirely.  If that goes away, I'm slightly partial to keeping the sample client located in the same source directory as the zeromq implementation.

I generally think of 'contrib' stuff as "will work, but less documented, please read source" quality code.  Keeping the source files together helps with that (at least for me).  If the zeromq transport proves to be something people really want, then I'd say we should remove the client entirely and just include it in the documentation.

Anyway - changes are included in : 6cfecd327af
There is a lot more code duplication in that TestCase then that ;) 

The isolation is also poor, because it modifies Django source tree, so this won't work in parallel and won't cleanup if ``startproject`` actually makes a directory, but with malformed name. I started refactoring it, but that would make the patch quite a huge change for this ticket. I can still do it, though.
Should there really only be one status server?
Wether to use `{}` or repeated `_` is a matter of style, so please don't change them all.
Good call, but I'm not sure IE6,7 ever need to go into this function
This function seems to compute exp_Mi instead of log_Mi which is the name of this function. A name that describes what this function is actually doing would be helpful.
And this block is commented out, not only negator.
Note that the other events (like namiEvent) should be changed to something like `event:new` or these new ones changed to `connectionCreated` for consistency
Destroying doesn't work on Windows, works fine without it.
I would really use WS for all communication. It's better to focus on one protocols and do that very well than to divide resources over two implementations. And REST is slower. I am really amazed at the speed of WS. My webclient performs almost at the same speed as a native MPD-one. Me like!
did you mean `casper`? ;)
As discussed, can remove
So the issue is that ruby will return the last thing it evaluated so were were throwing error such as foo can not be converted to int.  Worse is if they did return the wrong value.  Perhaps we should just return 0 on return from command and errorcode on rescue.
Little bug here.using rsplit can give incorrect results with submissions whose titles contain the string '.json'. Replacing `rsplit` with `rpartition` fixes these problems.
Shouldn't we just make Binding.UNRESOLVED implement getDependencies?
This seems an odd change. Going to switch back to being separated below.
Please replace `@param` tags with `@synopsis` tag: https://github.com/wp-cli/wp-cli/wiki/Creating-Commands

See other internal commands for examples.
Please simplify and or comment the following 4 lines.
Might give this argument a more descriptive name. Boolean blindness can getcha
Just loading the chunk winds up resulting in it being marked as dirty (artifact of the way the data map works - loading in the data array trips off all the dirty block logic), which would result in incorrect reporting of updates that either didn't happen, or are implicit in the fact that the chunk is newly generated (and which is implicit in the isNewBlock=true in the ChunkLoadEvent).
Extraneous line?
This makes the local var ```i``` unneeded.
I thought we used ellipses when we require more input.
Ditto, maybe we should remove or comment out since it won't be supported for a while.
Change: ``Path to a directory. This directory will serve as the root of the FileSystem instance.``
Would be good to document the result in various cases: offline, we've already checked recently, we haven't checked recently / first launch, etc.  In particular it wasn't clear to me at first whether this returns null vs. cached data in the case where the last check was < 24 hrs ago.
Parse JSON files once only and assign the loaded objects in local variables in the module.
Agree with you. So change the logic to just compare start.ch and end.ch since both have the same line.
XML escape, may contains apostrophes.
strict mode doesn't like this, not future-proof.

Please name the function.
Yes, I will pass the runner
I agree on the callback instead of the `dbg-attached` event
What do you mean that the user is "updated"?
I just made sure it is able to find the user correctly, because it is then just set as the current session user.
I don't think anything is updated.
The reason it's there is because if I move it up, it breaks thanks to import trouble.
Do we really need to return `-1` instead of throwing an exception? If yes, the commented code should go.
thought 'index.html' was presumptive, and 'file_default' is pretty self explanatory
Tried removing various combinations of apps in INSTALLED_APPS to no success.
*form_element* -> *field*? *form_field*?
It is [here](http://dev.wlan-si.net/browser/nodewatcher/nodewatcher/web/account/regbackend.py#L57).
There are `no-js` fallbacks for the carousel so that the page still looks reasonable without it.
This comment isn't quite correct. It should be
```javascript
// Use the default order of preferred renderers
```
or similar.
This test is to check we get an exception if the PIR does not exist; so to set up those circumstances I would want to stub out whatever methods (preferably outside the Deployment class) go and fetch the PIR and make them act as if the PIR is missing.
Where does this magic value of `80` come from?
For style consistency, could you put a space after # and capitalize the first word? Thanks! :)
Put the "Connecting to IRC" "Connected" the "MOTD for irc...." and the actual MOTD into a different constants.   Basically they are functionally separate strings.

You also may want to make the MOTD be an array where each item in the array is a separate line, rather than including newlines directly into the string.
May actually want to make this only run on grid things and for releases, not always run.
*p++='/';
added them.