labofsahil/github-event-dataset-2012 huggingface dataset:
Could you show us the full trace for that please?



On 19/02/2012, at 7:01, Steve Root<reply@reply.github.com> wrote:

> So, I'm new to this, and my current problem may be unrelated, but yesterday I found that I could no longer add images to my products in my development environment.  I think it was working last week, I think I've run 'bundle update' and I think the thing that has changed is paperclip changing from 2.5.0 to 2.6.0.
> 
> I'm posting here just in case others are about to find the same thing.
> 
> The errors I get are:
> [1] with gem 'aws-sdk', attaching an image through spree admin:
> ...lots of lines of information, ending in....
> 
> ```
> 1664 /Users/steve/.rvm/gems/ruby-1.9.3-p0@rkbbspree/gems/aws-sdk-1.3.4/lib/aws/core/response.rb
> 1665 /Users/steve/.rvm/gems/ruby-1.9.3-p0@rkbbspree/gems/aws-sdk-1.3.4/lib/aws/core/http/response.rb
> 
> [NOTE]
> You may have encountered a bug in the Ruby interpreter or extension libraries.
> Bug reports are welcome.
> For details: http://www.ruby-lang.org/bugreport.html
> 
> Abort trap: 6
> ```
> 
> [2] If I remove aws-sdk from the gemfile, and I remove the s3.yml settings from the config directory, run bundle update, I get
> 
> ```
> LoadError in Spree::Admin::ImagesController#create
> 
> cannot load such file -- aws-sdk (You may need to install the aws-sdk gem)
> ```
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/spree/spree/commit/bb22be1e53cd9cc405ed4e0512b1fa3e68cd86f7#commitcomment-982423
I'm used to seeing ``type(self) == type(other)``, will that do exactly the same thing?
yes, it's been pushed to master repo
Oui, ça me paraissait bancale de stocker les arguments et le booléen memory dans la même variable.. En fait je suis en train de travailler sur une branche différente pour réécrire les RPC et le reactor. Je termine 2-3 trucs et je vais la publier pour te montrer ce que ça donne et que tu me dises ce que tu en penses, mais il restera encore un petit peu de boulot avant que ça soit complet et débugué.
mognod
Same as above, loop and wait for scan to complete and then mark as watched.
I'd add a hint here, which crond you tested that with / which one supports it. This is probably non-standard and dcron,fcron, vixie-cron and cronie/anacron may differ.
This wasn't called if the outer if succeeded and the inner if failed. As far as I see this shouldn't be a problem, right?
eilo's gunship is WiP
Sorry but I don't get where you are going  ? 

have you plan or a little explanation on how the mouse handling will be managed?

The MouseIn, MouseOut, MouseMove, MouseDown, MouseUp allow stuff like drag & drop in an easy way
You are referencing RouteNames within the code. Where does it came for? I suspect you forgot to add a class file to the project :)
Double date-format setting?
I think that we should put the GER/GEMM/GEMV op on the gpu in a not inplace version. Then make it inplace later. Having inplace on transfer to the gpu caused problem in the past. The current logic will cause the same problem in some cases.
Almost. Perl really. I tried to keep those ;'s out. As well as the random :w that show up sometimes too. :P
OR COULD I? yeah I could.
@dimitri, this extends the file-local variables that you set in `el-get.el` to the whole project. It also adds a few more variables, such as `sentence-end-double-space`, since I noticed that you use double spaces after sentences, and `indent-tabs-mode`, which when set to nil tells emacs not use use tabs to indent.

If I got anything wrong, feel free to edit. You can modify this file automatically using `add-dir-local-variable` and `delete-dir-local-variable`.
You should then set your own AdaptiveReceivedBufferSizePredictor via the  DefaultChannelSocketConfig.setReceiveBufferSizePredictor(..) method. Use the constructor of AdaptiveRecievedBufferSizePredictor to set the initial size zo 4096. 

See http://netty.io/docs/stable/api/org/jboss/netty/channel/AdaptiveReceiveBufferSizePredictor.html#AdaptiveReceiveBufferSizePredictor(int,%20int,%20int)

When adjust the the DEFAULT_INITIAL you will have a impact on other implementations that don't care about HTTP.
It's indeed strange that pressing X again won't re-initiate the attack when you already have a target, and that looks like a correct fix to me.
Would be good to extract these checks in a separate method.
Why would you add this code again here when you perform the same logic check and variable assignment a few lines above for the redis connection? Wouldn't it be *cleaner* to piggyback the `config` object earlier in the code so you don't add so many hooks for Heroku?
It is non-Python to hide execution control inside of an expression — you really want `if` here. :)
I think that our Calculation in
time_t DungeonResetScheduler::CalculateNextResetTime(MapDifficultyEntry const* mapDiff, time_t prevResetTime)
is clearly wrong
I have a possible fix, take a look and comment
https://github.com/Reamer/mangos/commit/71490bd1a0ea78b18e9c334982dd3ebdcc5fd5d0
The "sLog.outError(...)" only for long term test.
I think that mistake in calculation is the reason for the daily freeze in InstanceReset.
When do you unhoock ? I think you'd better to use the Client.Load callback.
Bladestorm Immunities
Query : INSERT INTO `gameobject` (`guid`,`id`,`map`,`spawnMask`,`phaseMask`,`position_x`,`position_y`,`position_z`,`orientation`,`rotati...
:O I didn't realize there was a log implementation on the other clients.

Sent from my Windows Phone
From: David Fowler
Sent: 1/26/2012 1:00 AM
To: DamianEdwards
Subject: Re: [SignalR] Refactored log() in jquery.signalR.js (739151f)
Bad bad, you only changed the asp.net implementation.

---
Reply to this email directly or view it on GitHub:
https://github.com/SignalR/SignalR/commit/739151f803cffde848c5955a9f8380f4320f9ed3#commitcomment-903501
muss man dann halt nur noch irgendwann durchführen, die Tests ;-)
Oh wait, it does now!
I don't agree that this improves the readability. Please revert this.
ok I see your point.

The message doesn't say that the pointer has to be non-local. So a user might think that this check would complain about local pointers also. And therefore determine that it's probably noisy and not worth investigating too much.

now that I agree that the message is unclear, the problem will be to rephraze it...
you have two `class="button"`s
Is that comment true in Django 1.4? I thought they fixed the double loading of settings? (I know we're not on Django 1.4 after that was committed, however... just curious if we'll need this if we upgrade.)
Guys,
It's not so much a matter of who gets credit/how this gets integrated into the official code.  I just hope you agree with me that this improved readability of the code and will be more easily maintainable in the future for changes.

Brady,
As for which branch the commits are based on, I don't think it matters if when it comes time to put into master in this case if it's done by "fetch and cherry-pick" rather than "fetch and merge" or "pull".
25 is on reset not on respawn.
Instead of using get_user() here, we should have switched this to use model makers.  Ditto for most of the rest of the get_user() uses.
missing name after . operator in this line ;/
Вообще, в Spring MVC выставление статуса при исключении делается несколькими способами:

* Метод контроллера с аннотацией @ExceptionHandler - в нем можно выставить статус руками или использовать аннотацию @ResponseStatus
* Собственно аннотация @ResponseStatus на классе исключения
* Бин, имплементирующий HandlerExceptionResolver (например, SimpleMappingExceptionResolver), но там почти все руками делать придется
Given this is a utility function I'd rather not constrain it to use filenames, but it would be good to allow filenames too. 

Something like

img0 = images[0]
if not hasattr(img0, 'get_data'):
    img0 = load(img0)

and so on.

Also, it would be good to have a test for this variation in the behavior - would you consider writing one?
um, did you notice that it already had support for `crypto_scalarmult_curve25519` in the next section? :-). I like the new code better, though.. you might just want to remove the old copy (and the duplicate #include)
webfinger.fetch does check locally before queuing a remote job.  we webfinger here because we need to make sure we know who the author is, which only actually could be a problem if the object is a like or comment on a remote contacts post.Before, we were being really defensive about when do do this query, but I decided to do it earlier because 95% of the time, it is just a database lookup, and the other 5%, we were doing the webfinger anyways.
typo: 'fr' should probably be 'ft'?
1000 really looks like something special here. If we can't know the size ahead, we should use `init` instead `initWithCapacity:`
You should not need the `compdef` line anymore IIRC - now that "COMPLETE_ALIASES" is not set anymore.

Apart from that I have `alias grh='git reset HEAD'`, which should get included then also.
no, this line is fine. The subclass, TypeDeclarationDescr, should override the method, but it was lost (again) in some refactoring :)
i'll tell you what i think to be the best advice.  i believe that jquery and underscore could (and should) be anonymous modules.  it is much more of a "hazard" to use named modules rather than anonymous modules.  the "hazard" being talked about in the quote you referenced is when someone tries to include a module via a script tag rather than by using the loader to load the module.  ie this is the hazard:

```html
<script src="some/amd/loader.js"></script>
<script src="ko.js"></script>
```

the problem here is that ko.js will detect that `define` exists and will attempt to call `define` to define the ko module.  since the loader did not actively initiate the loading of ko.js, it has no way to imply an id for this anonymous module and so loading "without using an AMD loader" is the "hazard".  the fix is simple - remove the script tag that loads ko.js and let the loader dynamically/asynchronously load the module only when another module has declared a dependency on it.

for some reason, it was decided that adding ids for a couple of libraries was a better solution than having people avoid loading these modules "without using an AMD loader" even though an AMD loader was explicitly included in their page.  as you may have guessed, not everybody sees it the same way.

with ko as an anonymous module, someone using AMD can use ko via whatever id they want.  they can also do a build that will add the appropriate id for ko to work when included as part of a layer.  it also makes it possible to use 2 versions of ko in an application at the same time.  these things are not possible if you hardcode an id.  the id should be thought of as an internally private property that is a way for the builder and loader to work together but shouldn't be used apart from that case.

for more details on the debate around this, see the discussion at https://groups.google.com/forum/#!topic/amd-implement/VAtOIIyVrOs which also references:
 - http://dvdotsenko.blogspot.com/2011/12/amd-modules-with-named-defines-so-much.html
 - http://bdframework.org/bdLoad/docs/bdLoad-tutorial/bdLoad-tutorial.html#nameClashes
 - http://livedocs.dojotoolkit.org/loader/amd#the-amd-api

its probably a debate that's more than you might care to dig into but hopefully that's enough information to satisfy your curiosity.
Well it is simple javascript and has some simple documentation but if you could send the firebug console log with the errors and what the real issues are that you are experiencing then I could take a look at the code and solve the problems

Also it would be great to have some more developers helping (javascript / jquery)
I think there is probably a more general solution to this problem - ideally all possible items in the context would be checked for keys before calling an attribute. I'm wondering whether raw strings should even be considered valid context since they have no identifier.
certifi is needed only if you have ssl support I guess
gevent is needed only if you run async tests
envoy is not needed (the import in test_requests_async.py is a leftover and should to be removed)
gunicorn and httpbin are not needed if you run tests against public httpbin (httpbin.org)
Oops. OK, I removed that bad change I re-pushed.
Try with Rails 3.2
Fixed in 6c93dc14ce7bf0e66c48ce65a561dd4765a824f3 - cheers!
I miss working with you :-)
I thought you were switching to a more native approach?
also, install rspec as noted in the tutorial
Okay :) I have messed with the installer a bit so it works for per-user and per-machine - it probably won't merge so you may just want to re-do your changes there. As you are the only consumer of the registry location you change it to what you want, I can see you are aware of the 32\64 issue when dealing with HKLM (I spun some cycles on that); lose my name in the path and just have OpenCover though :)
Предназначение класса ужасно странное, убери его
Why was the dependency status removed?
wrong order - should be included *before* <stdio.h>
No entiendo que quieres decir. You can talk english to me if it is easier to you.
@scottjehl 

Looks like we lost the `bg` on the end there :(
Yeah.. any idea why?
Why 10? I've seen lots of 8.
let's keep the default hashCode() and equals() impl. and let the XAP platform handle that. from my experience it does good work there.
Instead of the lines about maven, with installing it locally and setting it up on the command line. I would recommend installing this maven plugin from the Eclipse Marketplace. The project in GIT has a M2E profile already so it will do the maven integration automatically.

http://www.eclipse.org/m2e/
Strange yahoo works correctly for me.
Tested but exception caught details in ticket #7733
Ah we removed that during review. No it hasn't been deprecated, I'll re-add that on Monday when I am working again if you can just disable the test for the moment. I also have a ticket to move download() into core but I suspect that'll be a while yet.
/*name*/ is most portable way avoid like warning for unused by reason arg.
Eso es para dos cosas:
1. Asegurarme que los atributos "expuestos" estén bien puestos
2. Evitar que exploten las pruebas (a partir de Rails 3.2 cualquier asignación masiva de un atributo "no permitido" tira una excepción)
Ich möchte nicht, dass die priv server zu sowas verkümmern wie gerade cata ist... SAI auf jedem Boss... lauter total unblizzlike gespawnte sachen... ich meine ich spawne nur unblizzlike aber bei mir sieht es wenigstens dann auch genauso aus wie aufm offi.
This is not a good idea.  eventually, newzbin will change their URL again, or rather, they will fix the DTD.   I submitted a patch, in the bug knowledge base with a dirty hack that at least examined the new an the old.   We can assume what the new URL will be, but might it be safe in the meanwhile either ignore the DTD NS or search both?
ok, so it's probably @d[0, start + 1]...
Why didn't I use it before? O_o
This is not working for me -> Breaks ChaosUA 1kWinter Script (wrong phases and spawn)
pff, I extracted and uploaded the maps yesterday, and now again... T.T

jeje i'm kidding, good job, thanks.
For DDAbstractDatabaseLogger && DDFileLogger:

Those properties absolutely ARE atomic. I do not believe they should be labeled as nonatomic in the header. If you take a look at the implementation files, you'll notice the accessor and setter all go through a private dispatch_queue, make them atomic.

For DispatchQueueLogFormatter:

The properties are supposed to be atomic. In the implementation file, the method "formatLogMessage:" may be invoked on a background queue. It depends on an atomic getter to maintain thread safety.
I want my moment enjoying reading this captured somehow :)

On Feb 8, 2012, at 9:03 AM, Simon Murtha-Smith<reply@reply.github.com> wrote:

> damn, jshint can go fuck itself.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/LockerProject/shared-ui/commit/aaecffb3b881d8ff0a685f725d923d1c7b8c577c#commitcomment-945953
fu wrong part :-/
UPDATE `game_event` SET `length`=20160 WHERE `eventEntry`=8;
Missing:
INSERT INTO `creature_text` (`entry`,`groupid`,`id`,`text`,`type`,`language`,`probability`,`emote`,`duration`,`sound`,`comment`) VALUES
Consider adding a DataSourceMetaItem::isXmlAttribute() can calling from here.
It should be `===` instead of `!==`. my bad
Hi,

Could you merge this commit to your 2.0 branch.

Thanks for all
Should you have cast it to bool * then ? You are assuming byte and bool are the same size (likely to be always true, I know...)
I just looked over http://phobos.java.net/overview.html, and it looks like Phobos 0.6.2 is similar to the Mozilla 1.6R7 script engine -- not sure of all the differences yet. 

Also, it looks like Oracle is working on a new JavaScript engine called Nashorn that will be released this year http://en.wikipedia.org/wiki/Nashorn_(JavaScript_engine).
dependent лучше всегда определять (nullify or destroy) - иначе будешь получать битые связи
используй context  а не describe
create... would be a better method name, since it is a static factory method.
Ah sorry I just redited the INSTALL files to match a 100 chars specification (I was only ever expecting the 96 character thing to be a private pecadillo!)... I hope you don't mind if I change this to match what we said in credo.txt, I really do think 100 chars is the best compromise...
I'm happy with that. I guess I will jump into the code way later than you, so keep the code as you think it's best and put some comments in (we can delete them later).
Already imported in a parent project.
Опечатка заралась.
Должно быть Osmarender
yes, this need to be reverted. I had hoped libuv had similiar mechanisms to hook into windows threading api's but you are right, this module should stay focused on unix and libeio.
:/ The problem is that for a Active Directory the flags are bits and it's not easy to test with a regular expression.

- AD:         flags & 2 != 0
- Samba: flags.include? 'D'
This did result in a compile error because workers had not been defined on PayrollCalculator.
forgot a success message
> [...] you had already defined a `Monoid` instance for `Color` in module `Shady.Color` with definition [...]. This means it is true that `toColor mempty == clear` and also that ``toColor (a `mappend` b) == toColor a `mappend` toColor b``.

I see how this conclusion follows in the case of `Color`, since there `toColor = id`, but then the TCM is trivial.

> Incidentally, why did you call it `preClear`?

Because it's a "pre-image" of `clear` under `toColor`, i.e., `toColor preClear == clear`.

> Can you expand on your remark "I don't think of `toColor` that way, because we can distinguish between values that `toColor` identifies". I'm not sure I fully understand.

I always want semantic functions to be TCMs, and I see failure of this principle to be abstraction leaks every time. Which raises the question of what is a "semantic function". I was trying to give one requirement that `toColor` violates--namely injectivity--but I failed, since `toColor` *is* injective for `Bool`.
looks good, just found 2 code style issues :)
Why You deleted method for SetArenaPoints and Honor?. And not removed declaration in player.h (+ using in other files)
cane, e il resto dov'e'?
this line is not sure. if date_up is None, it must not be passed when metric item is created (**date_up is not null**)
Something like this could solving:

    kwargs = dict(metric=metric, user=user, count=count)
    if date_up:
        kwargs['date_up'] = date_up
    MetricItem.object.create(**kwargs)
Yes and no.  Now I need to increase my lower bound dependency, however `persistent-postgresql`'s branch using your packages is not released yet so it didn't really break anything.

Could you please add a warning to this module's documentation stating that it may change even without a major version bump? =)
Ah, never mind. I renamed the project and Eclipse was confused, but it all got sorted out on a refresh and relaunch.
If this is to be implemented it should be:
```java             
                // call event for other plugins
                TravelAgent agent = new MVTravelAgent(this.plugin.getCore(), d, event.getPlayer());
                MVPortalEvent portalEvent = new MVPortalEvent(d, event.getPlayer(), agent);
                this.plugin.getServer().getPluginManager().callEvent(portalEvent);
                if (!portalEvent.isCancelled()) {
                    performTeleport(event, ps, d);
                }
```
Cool, please rebase, test locally and send a PR, i'll merge when travis will tell me it's fine to do so :)
Thanks. Looks great!
Method name says getAllFor**VdsGroup**WithStatus, but what you pass as argument is **getstorage_pool_id**
https://github.com/enjalot/adventures_in_opencl/pull/5 :)
I merged in your change --- did you have an issues with the change and south?
Can you send a pull request, so I can update the NuGet and you can get credit for the fix.
Looks fine to me.  Exactly what does 'database' mean here?  Is this Fennec's content provider, or something Android-y?

I verified that we don't put rec.lastModified into any other ContentValues objects, too.
so should drop extra print statements
+        foo = self.system.images("demoapps.png")
+        print foo
+        print "TESTING AGAIN"  

click("images/install_accept.png")
click("install_accept.png")
dude! Strava api :) nice
idem, variabiliser
Sory, but now in all instalation will be russian tab.

:caption => "Equipment", :after => :projects

Hear shuld be i18n...
I forgot to account for the date having padding when user doesn't want the date to show
dont know! just found that! Lol
the windowshade one im gonna have to compile and see but the statusbar unexpanded color value being default to ICS grey im quite confident is what were looking for...
You can clean this up by simply:
```javascript
return callback();
```
I want promise objects with more methods, extended through prototypes. Otherwise promises are just a bloated version of `Parallel`
IMHO it's very important in split read/write connections environments. Good catch.
Authentication itself should probably extend from Serializable
Shouldn't you use StringUtils.isNotBlank ?
This can be part of the $.when above
yes, and that has been fixed. I'm just compile-testing a change of all Duration settings I found which are in reality FiniteDurations. Let's start at the producer side, then we can with less pain tighten the consumers in 2.2 (or something).
Good point. Leave it as is and put a FIXME to change it to URLEncode when http://www.assembla.com/spaces/akka/tickets/2123 is fixed
You did agree to make it public a while back. ;-)

It's there so one can pass around compact ByteStrings explicitly and serialize them.
This facility is intended to be permanent, right? Then it should have very clear warnings that it is not suited for “serious” work, because you should properly design an actor system as a hierarchy, where top-level actors form the Error Kernel, i.e. creating them is not the rule but very much an exception.
I think we should only export akka*, not the other ones.
Why? In case that is actually relevant, we’re blown out of the water already anyway. This can only fail when interpreted and we hit a SOE for calling `suspendCount()` (in which case the actor will be defunct afterwards) or when facing a VME. I can of course change it, but please give me a reason for doing so.
any risk with that we can't change this one?
would it be possible to move to context?
= this ?
Is Exception the most specific thing here?
`decodePdu` should be lazy wrt. decoding the actual payload, since we might throw it away in untrustedMode (it might be a mem bomb …)
Note sure if this is the right way to test for strings across python 2 and 3.
On Thu, 30 Aug 2012, bthirion wrote:
>  > +print 'Fitting a General Linear Model'
>  > +fmri_glm = FMRILinearModel(data_path, design_matrix.matrix,
>  > +                           mask='compute')
>  > +fmri_glm.fit(do_scaling=True, model='ar1')
>  > +
>  > +#########################################
>  > +# Estimate the contrasts
>  > +#########################################
>  > +
>  > +
>  > +con = fmri_glm.glms[0].contrast(
>  > +    np.vstack((contrasts['left'], contrasts['right'])), contrast_type='tmin')
>  > +z_values = con.z_score()

>    You mean, the effect size of the contrast (c^T \hat{beta}), on top of the
>    statistic ? Sure.

yes (somewhat) ;)  GLM users often expect to obtain those 'beta's for
the original fit (i.e. 1 beta per each explanatory variable/observable),
without explicitly creating a separate contrasts per each
explanatory variable to get that value.

-- 
Yaroslav O. Halchenko
Postdoctoral Fellow,   Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834                       Fax: +1 (603) 646-1419
WWW:   http://www.linkedin.com/in/yarik
Alright, that's great :-)
Is the controller accessing the model here? Should members_of_type(organization_name, object_type, role) be a helper function for templates to call?
Mime would only allow to work with extensions, not folders, would it?
lets use `href="javascript:void(0)"` here -- it's a pain, but we may end up using the hash for routing on older browsers, in which case that could mess things up.
This is all simple renames, so let's just change it as soon as we know what we want for each one.  I'll do the updates indicated, and that should be that.
Ideally, you call this externally and prevent the mail sending from throwing an error, and this way they are using the same conditions.
put the explanation in the code as a comment
если это уже GNUmakefile для элемента, то цель можно назвать просто `update`/`up`
Mowisz o linii 62? No moze i mozna ja wywalic...
Please remove the brackets here.
`diff_expected` would be a better name (I think).
Instead of this intangled things I suggest we use addressPromise.tryCompleteWith(expression on listens that produces the result)
This needs to be prefixed with `tui_`.
If you quote the function name, as in `geiser-view-last-image', it will be a link in help buffers (those where an open quote and a closing quote).
Done @  b49f847
This seems fine, but doesn't `socket.destroy(er)` trigger `socket.emit('error', er)` which in turn triggers `req.emit('error', er)`?  (If not, then I think that's a bug.  What happens to socket errors?)
The callback must be deferred with `process.nextTick()`.
`isdefined(getattr(self.inputs, key))`
@cournape: What are these "ways to easily intercept function calls"? If you have dtrace then sure, I guess, but most of our platforms don't. A scriptable debugger? That somehow can somehow do automated analysis of breakpoint callsites and match those up with the Python-level information to produce a useful report?

However, writing this, I realized that I am actually -1 on the logging approach unless there is some fatal flaw in the callback approach. Because, with callbacks it's easy to capture the Python stack that resulted in each malloc, which is super-useful for profiling reports, and with logging, it's very difficult. The current interface seems like it pretty much *requires* you to install a trace function and poll the log after every line of Python code in all but the simplest cases.
That did cross my mind at one point in time. We would need to provide a mapping as well, and the values of the two don't really map well. (Also I don't recall any of our applications making use of it).
Yup! Just a typo
I don't agree with .idea being added to .gitignore. Rather, you should be careful with what you add to your commit, adding changes using preferably the -p option during git add, so that changes like debug statements or IDE pollutant directories don't get added. 

-- 
Ryan Bigg


On Tuesday, 12 June 2012 at 20:09, Philip Arndt wrote:

> > @@ -10,3 +10,4 @@ tmp
> > *.sw*
> > spec/dummy
> > *.rbc
> > +.idea
> > 
> 
> 
> Do you add it to every project you work on?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/radar/forem/pull/238/files#r965946
> 
>
1) It's easier to read. 2) Strings are correct - look at how Rails generates db/schema.rb and you'll see.
Does changing this from text to varchar have side effects on other db tests?
Can you add docs here?
What happens if you give it `false`, but has `X-UA-Compatible` set in the app? Will it override the users' value?
> As of jQuery 1.7, the .on() method is the preferred method for attaching event handlers to a document.

Extracted from http://api.jquery.com/bind/
Please move this entry to the top.
Could also throw something else (failure to produce a result)
Isn't that method added using the ask support? If so, we should remove the method here.
Indeed, it turns out that my worries about the constants turning public were unfounded, and the overhead of one `lneg` in each case surely gets lost in the noise. I’m running the tests again and will push a fix shortly.
Doesn't this change the shape of the result?
Ah, scratch that.  I forgot we are woking with three-valued logic.
I read this as "... and raise NotImplemented".  Feel free to correct.
Wait, isn't that an implicit import? Isn't it bad or am I getting it wrong.
I'll move the `Factory(:instructor)` into task_test so that it at least moves it out of the base class.
did you add these lines by hand? usually you do not need take care of this namespace file, because finally I will run roxygen to automatically update it
Well, actually, that wouldn't be wrong here, because `SphericalCoordinates` *includes* a cartesian representation - you can do ``coord.x``, ``coord.y``, and ``coord.z`` and you will get out the cartesian coordinates, as well as `coord.cartesian` and get a `CartesianPoint` object (this isn't implemented yet, but it's in the API doc). But your example would hold if you want to use e.g. cylindrical coordinates. 

But I definitely see your point that `SphericalCoordinates` is a more descriptive and accurate term here - @demitri , do you have any objection to that?
Should be addressed in commit db4a473
maybe we can more this logic into the holder instead?
Should we log this?
I might put this in as a note block or something... I think running the whole instance with the profiler on is probably not something that we want to suggest, and rather just include it as a footnote rather than a section.
coordinates need to be in-line literals
[09:04:31.381] ReferenceError: getUnitsCSS is not defined @ http://developer-local.mozilla.org/media/syntaxhighlighter/scripts/shBrushCss.js:78
Probably adding something simple like this would help:

/*
* Options array: [key] = array(lang, auth)
* key: string
* lang: language array key.
* auth: true if current user has permission to use this option, false if not. Only options with auth != false will be shown
*/
We still might want to consider keeping that functionality.
This then becomes unnecessary.
Again, an unrelated change.
Rank and Color are totally independent
This is shown to the Administrator in the ACP, so the extra text is not needed.
check the error code
why get again destination, since it is the same Player object as the one returned by get_profile.
Since some of my changes seem to be mixed in with yours I'm just checking to see if you intended for the Static Arrow discussion to be deleted.
what's the reason for this line? is that 1000 an arbitrary number?
Why are we removing `scripts.start`? Is that not used by any hosting environment?
с прошлого апдейта осталось похоже)
Change the url here as well.
Yes - it'd be much nicer if `coord_dims()` returned something that wasn't a reference to mutable internal state of the Cube.

But instead of modifying `coord_dims()`, how about converting the internal Cube state to use immutable tuples? Then they can't accidentally leak out via some other route.
Can we go with the latest 1.9.16?
如果words是一个paper.set()的话，直接words.show()就可以了吧。无需遍历
On Tue, Jun 26, 2012 at 10:48 AM, nouiz
<reply@reply.github.com>
wrote:
>> + * dB0, the source leading dimensions size
>> + */
>> +template <int operator_num>
>> +__global__ void k_take_3(const int d0, const int d1, const int d2,
>> +                         const float* indices,
>> +                         float* a,
>> +                         const int sA0, const int sA1, const int sA2,
>> +                         const float* b, const int dB0,
>> +                         const int sB0, const int sB1, const int sB2,
>> +                         int* err){
>> +    for (int i0 = blockIdx.x; i0 < d0; i0 += gridDim.x){
>> +        int idx = (int)indices[i0];
>> +        if (idx<0)
>> +            idx += dB0; // To allow negative indexing.
>> +        if ((idx < 0) || (idx >= dB0))
>> +            *err = 0xFFFF;
>
> I think this is more safe, as concurrent write don't have all the behavior well defined I think. I didn't wanted to look into the defail, but I think only 1 of the write is correctly writed, but only if the write don't cross a cache line. I think that when you alloc memory, you receive memory that will have each element fix in one cache line, but I prefer to be safer in case this change in the future.

But ... you're still just writing a single 32-bit number. How could it
cross a cache line? Maybe you prefer to allocate one byte?

Anyway, you should definitely write your thoughts on the matter as a comment.
Replace this function with a new method "void AbstractPlanNode.reattachFragment(SendPlanNode child)" and don't use combinePlanNodes or PlanNodeTree for this at all.
We will squash migrations anyway but it seems this is a pretty unhelpful file name.
no, we shouldn't. node.js compiles this as `exports, require, module` too.
Why is this checking by name?
If I am right, it can be a DOM *node* (can not be a window) or a jQuery *instance* - but which jQuery?
@gibson042 Thanks, fixed in my upcoming PR. Made sure there are no other ones we missed:

`dev/jquery (master)$ ack 'service.*echo[^\/]' --js`

(0 matches)
Support for command prompt!
In that case it should probably be ``backend.library.refresh(uri).get()\nreturn`` to emphasize that we don't have a return.
None of this code belongs in this method.  It needs to be much more cleanly separated from this method (which is already too long).

Handling help here isn't the right place - help is a special command and it belongs to the run! method on Commander::Runner, not to this class.

This also isn't the right place to rescue exceptions - that's again something that run! should be doing.
Yes, for that reason we also store the md5 hash of the path in the filecache, since that is small enough to index
Oh you're right, it doesn't. Maybe it should. Anyway, something for a different branch some other time. :)
You might want to describe what you mean by "use". It's technically clear since you said consumers, but I'm not confident people will be that up on our terminology. Consider adding something about accessing the repositories from yum, since that's the only time the protection kicks in (as compared to, say, binding to a repo which arguably matches "use" as well).
@rmocanu: you're missing the license header
i don't understand....
You mean to have auto_set_breadcrumb called automatically? Not sure - I was hoping the examples in https://github.com/58bits/ariane-sample would be okay - showing how before_filter could be used to control this.

Things suddenly got busy at this end - at least for the next week or two. I think the code here is looking pretty good thanks to your suggestions. Also the dynamic breadcrumb feature does not effect the existing codebase (or gem), since we are now using LevelCrumb and not Crumb. How would you feel about accepting this pull request - for now? And then later we can add the i18n support for crumb labels based on keys. 

Thoughts?
@solnic ahh yeah, after I mentioned it, I realized that the object needs the ivar set internally. I think `#instance_eval` the way we're doing it is probably nicer then.
I hadn't realized (until I just looked) that new RandomAccessFile() creates the file if it's not there.  Now I know.
Ah, in the case of removing itself (or any listener)?
uh no, this will be logged every packet.
On second thought, it's probably better to use your own icon so there's no dependency.
One thing I was considering doing (since this is kind of "parameter hell") was changing these args to be an object (with fields named after the args), so the calls would be easier to read. The downside is that it makes the calls a lot more verbose, but maybe that's worth it for legibility. What do you think?
I thought this was handled in `AssocOp.__new__`.
Extra cruft for an option we're not going to use. I've already cannibalized the entire gem; may as well remove irrelevant features.
had to map every datestring we have in our database. we've changed our datetime strftime and i think couchdbkit and java's representation is subtly different in how they default the ISO8601 timestamp. this is pretty comprehensive - i think :)
when making new form widgets, it's best to add them as hqstyle plugins....here this is especially necessary since it's dependent on the bootstrap typeahead plugin.
It's just in the latest update. I would never edit a bootstrap plugin
locally lol.


2012/12/12 Biyeun <notifications@github.com>

> In corehq/apps/app_manager/templates/app_manager/releases.html:
>
> > -                            "Once you've fully tested an application, you can star it to mark it as released. " +
> > -                                    "When a CommCare application searches for updates, only starred builds will be considered."
> > -                            )
> > -                });
> > -                row.removeClass('build-released');
> > -                row.addClass('build-unreleased');
> > -            }
> > -            var LATEST = 'build-latest-release';
> > -            $('.' + LATEST).removeClass(LATEST);
> > -            $('.build-is-released [data-is-released="true"]').first().each(function () {
> > -                var row = $(this).parents('tr');
> > -                row.addClass(LATEST);
> > +        var el = $('#releases-table');
> > +        ko.applyBindings(new ReleasesMain(o), el.get(0));
> > +        el.show();
> > +        {# This currently doesn't work because of a bootstrap update, #}
>
> ah, did you modify a bootstrap plugin directly at some point? The best
> practice is for any new functionality to create a custom bootstrap plugin
> that builds on an existing one... anyway, no biggie
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/core-hq/pull/373/files#r2391803>.
>
>
This would be slightly cleaner if you put `scale_shape_identity` in the specification of `s`
Valid point. This is consistent with the other API.
Hardcoded URL. `setLanguage` -> `set_language_`
Ok i will check this later.
No. As you maybe remember, everybody will get an account on our portal. "Registration" is in our case just configuring the username and password so that you can have the same username on multiple computers. So immediately when somebody will visit our portal, an account (user object) will be created for her and language selection will be set based on her browser settings. And then language of the portal itself will be using this information from the user object for language selection.

We could still have small language selection in the footer for easy access, for all users (similar to how Facebook has it, if you check).
Something with that, yes.
Are any of these classes necessary?
`if ( templateConfig.plugin.plugins ) {` => `if ( templateConfig.plugin && templateConfig.plugin.plugins ) {`
How about `classList.contains( "handle" )`?
What if the user decides to drag this beyond the video? I'm fairly certain it's something we don't want to happen. Should consider stealing the normalize function we use in other plugins like popup.
Yea that's pretty awesome
Mind making this `<a href=\"http://www.twitter.com/" + twitterHandle + "\" target=\"_blank\">` so it matches everything else we have?
[14:46:45.720] ReferenceError: assert is not defined @ http://107.22.172.223:8877/1dea92c0c1e8ac5/web/viewer.js:439
Otherwise you would have lots of copy'n'pase code, see `case TextRenderingMode.ADD_TO_PATH:` present in both switches
Could this maybe be inserted via JS as the filter code is being kicked off. Then we don't need to hide it with CSS.
This might be wrong. This line increase tries counter.
一模一样的
Wouldn't it make more sense to split out the client and the email into two classes? Create an instance of the client and send multiple emails to it.
Seems like this stuff shouldnt be in-line here, but use the existing work in utils.py to do this. https://github.com/GeoNode/geonode/blob/dev/geonode/utils.py
Can you move <streamname(comma separated list)> to the end of the argument list and make it optional?
Either way works, but these two lines have two spaces while the following ones have four spaces, unless Github is messing it up.
This change cause an compiler error ( acctually this is because the change of treating warnings as error now)
Since "pos" is unsigned "i" also should be NSUintegher.
I understand why you change this( the condition is i>=0 and this is always true, right?).
I have already added condition check at the end of this block so this loop is not infinite.
Keep it NSUinteger is better now I think.

Any opinion?
Or maybe `start::Any`
`allow much more powerful options to deal...`
Don't need this anymore either.
Added the declaration of the files variable to avoid the compilation problem. @jacalvo commented the other declaration in April of 2012... check it please to see if could be uncommented or fixed.
Is there a reason this is commented? Does it fail?
Well it actually did work when I tested it, but I put it in an if block
anyways for clarity

*Nikhil Goel* | Vanderbilt University, 2013 | Computer Engineering,
Economics**

LinkedIn <http://www.linkedin.com/in/ngoel1> |* ngoel36@gmail.com | ( (256)
503-9183 | NikhilGoel.me <http://nikhilgoel.me/>



On Sat, Dec 1, 2012 at 5:49 PM, Luke Steensen <notifications@github.com>wrote:

> In app/helpers/application_helper.rb:
>
> > @@ -1,2 +1,12 @@
> >  module ApplicationHelper
> > +  def avatar_url(user)
> > +    # if we have an image from OmniAuth, return that
> > +    user.avatar_url if user.avatar_url
> > +
> > +    # otherwise, try to get a Gravatar using the user's email address
> > +    # if no Gravatar (or email) exists, use the default image
> > +    default_url = "http://i.imgur.com/NgOB1.png"
> > +    gravatar_id = Digest::MD5::hexdigest(user.email).downcase
> > +    "http://gravatar.com/avatar/#{gravatar_id}.png?s=48&d=#{CGI.escape(default_url)}"
> > +  end
>
> This code will not do what you want it to. Line 4 won't just automatically
> return; you need to put this all in an if else block. Ruby only
> automatically returns the final evaluated expression in a function.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bitcast/bitcast/pull/23/files#r2286519>.
>
>
Yes, I sent a transaction (to myself in that test-case) and got asked the "do you want to pay ... bla bla" message, clicked Yes and got a tray-notification of that Tx. Can you try that code on Linux? Or do you want me to leave it a pointer?
Tool.js should be `@require`'d.
incorrect notation
You're right. I have just commited a change. Thanks!
That is a great tip.  I reworked the user mailer to use a shared example.  Pretty slick.
I think we want to allow touchmove and only preventDefault of we detect that the user is swiping to open/close the menu. So this would require grabbing the Y value as well and detecting whether it changes
Where does `_exec_filename` come from? I don't think I've come across this before.
>In Microsoft SDK 7.1 Command Prompt:

The first thing the wiki tells you to do is to "Open the **VS2010 Command Prompt** ". This is different from the SDK command prompt.
Heuh is this needed?
Yeah, that was leftover cruft.
Why not use `target="_blank"`?
Do you still need this sorting now that it's done in the template?
Does this mean that you can't pass `lang_files` into `_lazy` if you have extra lang files in the calling module?
I added the thunderbird logo
In other tests we always use the tmp folder. We shouldn't behave differently here.
aren't these supposed to be agnostic to the resource type?  why is it directly creating suites here?
On 7/31/2012 11:05 AM, Chris Van wrote:
>> @@ -426,3 +426,7 @@
>>   
>>   # Percentage of what developers earn after Marketplace's cut.
>>   MKT_CUT = .70
>> +
>> +# Login credential source
>> +LOGIN_SOURCE_UNKNOWN = 'u'
> could you change this? (to answer your question: yes. how? that's still being figured out by Identity.)
>
yep, talked to rtilder about it and realized that zamboni is the 
authoritative source of credentials. I'm concerned that there may be 
other issues, but those are not related to this.
Couldn't there be an issue here: An uri would be like `path;param1=value2,param2=value2` and if I get your proposal right, `;`, `,` and `=` would not be encoded in **both** the path and path parameters so the solution is wrong if the path contains any of these 3 chars.

If this is the case - and provided that parameter path is seldom used - could we pass the path parameters  in the `$parameters` instead (i.e. `_path_parameters`) ?
This will trigger a notice if the left array is empty for the first iteration as ``$shiftLeft`` will be undefined. And if it is empty for a later iteration, the variable will contain the value set at the previous iteration.

```php
if (empty($left) {
    $item = array_splice($right, 0, 1);
} elseif (empty($right)) {
    $item = array_splice($left, 0, 1);
} else {
    // ...
}
```
Can you revert to everything on one line? (same below)
supersede has got nothing to do with seeds ;)

http://www.collinsdictionary.com/dictionary/english/supersede
show an error message using `SVProgressHUD` in `else` block
Due to space requirements, Endorsements was purposely spelled that way (see 18546).
use `[a postsCount]` instead - it will not raise any warings
same for `b`
What's the reason for this change? Just curious and wasn't documented in the PR message.
Actually, this is used only in line 17, so you are currently doing `Event.find` twice.
add_indes :user ?
(deviation > NumberTraits<Integer>::ONE) ?
Please, do not use that strategy to add the info:

- in the template, add the HTML and a placeholder for the rudderId, something like <pre><div class="rudderID"> <b class="threeCol">Rudder ID :</b> <span id="rudderId"/> </div></pre>
- in the Scala code, use a css selector to change that place holder with the actual value, i.e: "#rudderId" #> the_actual_id &
- and that, for all IDs.
Perhaps just leave this out. Size will be provided by the geoaxes.
It should still work if it's None, when used as a standalone function.
зачем тут это?
It certainly would be nice to rewrite this, I think it seems more complex than it needs to be (but maybe it really does need to be so complex, I'm not sure).

But I do think that this addresses the problem, and will do for now. Without this, it's not even possible to test easy publishing properly, and that shouldn't be held up just for the sake of making something more elegant if it works.
it seems rather monolithic to me. There's basically one huge menu that gets chopped up on each request.
```
jonas:~ python2.5
Python 2.5.6 (r256:88840, Dec 16 2011, 14:22:12) 
[GCC 4.6.1] on linux3
Type "help", "copyright", "credits" or "license" for more information.
>>> from dj_database_url import parse
>>> parse('postgres://user:password@host:1191/dbname')
{'ENGINE': 'django.db.backends.postgresql_psycopg2', 'NAME': 'dbname', 'HOST': 'host', 'USER': 'user', 'PASSWORD': 'password', 'PORT': 1191}
```

where's the problem?
We should include an example IPv6 address so the user knows those are supported.
Also missing semi-colon here.
the above for loop goes through the matrix to turn it into a csv style shape rather than a matrix of different length rows. 

for each row in the matrix, it builds an array of cells using the header that was extracted from the matrix, so for example if there is an entry in the matrix that is "1sgImper(+)" it puts that into the column that corresponds to the "1sgImper(+)" 's position in the header
Can this module and the related maven dependency now be removed from AS 7?
And if you are making packages, you've got it installed.