labofsahil/github-event-dataset-2012 huggingface dataset:
Added swedish translation for getting started guide

Swedish translation for Arquillian getting started guide.
now_playing updates
OAEWDGT-141 - Use History.js for internal links

https://jira.sakaiproject.org/browse/OAEWDGT-141
Fixes white space bug on responsive displays

Responsive displays are having an ugly 60px height white space. This fixes that :D
Also, __twitter/bootstrap.less__ is removed in favor of __twitter/bootstrap_base.less__.
Использовать конструктор Blob вместо MozBlobBuilder в Firefox > 12

https://bugzilla.mozilla.org/show_bug.cgi?id=736687
https://bugzilla.mozilla.org/show_bug.cgi?id=721569
Исправление http://iichan.hk/b/res/2496259.html#2562861
Minor copy-edits
New version of ZeusLoadBalancer 2.1.8
use SecurityContextInterface instead of SecurityContext

cross-posted issue, see https://github.com/symfony/symfony/pull/3522
Base64 in place of HTML character conversion
Reducing the height when the VC is added to a UITabbarController.
Adding Geolocation support for the onGeolocationRequested event in

WebWorks

Fixes issue blackberry/BB10-WebWorks-Framework#201 partially. Notifications will be in a separate pull request.
Fixes blackberry/BB10-WebWorks-Framework#350

Removes the distinction between XHR requests and other
networkResourceRequests.
Make blank line really blank to satisfy Test::EOL

that is all.
Added test for acal view and all required files
Implemented finding images.
Fixed a 404 error

It was caused by wrong link.
Master mistress

Add support for swapping master & mistress.
Fixed Rakefile so that bundle exec rake refinery:testing:dummy_app will launch.

To fix: config/resque.yml - it isn't being copied over so execution fails
[WIP] Update to Bootstrap 2.0

Just to let people know that I'm working on the MAJ of Bootstrap (i.e.: 2.0)

Change list:
- Bootstrap & Bootbox 2.0
- ability to add icon before each text in button (thanks to BT 2.0)
- help to display custom field in a form
- a quick fix about Doctrine integration

Todo:
- check `twipsy` integration since it change to `tooltip` in BT 2.0

Please do not merge this PR right now :-)
Got Zach's code running
Invert Translation
Text filter + test

TextFilter method, desc field, text filter test
Ics: updated system.prop

added reference to dynamically pulled in FP lib/jar
Fix mixin example

as discussed here: http://stackoverflow.com/questions/8743736/mixin-gives-error
Jason, moving the id from the &lt;li&gt; to the &lt;div&gt; breaks "Organize", i.e. you can't drag and drop books around after the change is made. Was this change intentional? It makes sense for me to have the ID on the containing element, not just the image wrapper...
This is a pretty pedantic micro-optimization, but wouldn't it be better to just move `var p = o.path || o.fullpath` up one line and change the if statement to `if (p)` to minimize the property lookup?
This section here isn't actually just formatting, but an actual fix. Though it seems I forgot to commit it.
k ... guess user is confused then ... thx :) (davilla stated the same...)
on osx, OnMove gets a notification when the window is moved or is created on some display.  One thing I'm wondering is perhaps we are mixing OnMove() with Move()

Move() would be a commanded move, OnMove() is a response to being Moved. Any possibility to change the windows code to generate a Move() instead of OnMove() ?
Yes … :-D
I think that it would be better to test this on Command level.
resource_ctrl.execute(SortKeywords())
and
resource_ctrl.execute(Undo())
and also
resource_ctrl.execute(Redo()) <-- is this currently working?
overkill :p if someone adds _queryEcho to Object.prototype they're messed up
Já bych radši, kdyby master byl celej fast forward. Tuhle mi přestal fungovat submodul, což mě nepotěšilo. Jestli tam hrály roli minuty nebo ne, to nevím.
Ops, my bad, didn't get the failing test. Thanks for fixing.
@abuddy I think that's because most people realize it's a stupid oversight on GitHub's part, but also not an entirely understandable one seeing as it's made way too easy by Rails to just define models and have the models be completely open for mass-assignment. GitHub fixed it, Rails's attribute protection defaults should be reevaluated and... anything else? 

Yes, I'm disappointed that GitHub made such an obvious mistake, but I'm more concerned over all the other sites that have this vulnerability, caused in the first place by Rails defaults, not stupid developers.

@Tie-fighter He posted an issue about Rails mass-assignment attribute protection defaults, which they ignored/dismissed, which they shouldn't have. The first thing he did when he found vulnerabilities in GitHub seems to have been exploit them, not report them.
I wish it was!  I think I copied the wrong part of my screen history. Just rerun the whole thing from scratch in a new terminal, pasted to a gist https://gist.github.com/1864074

Thanks for helping with this. I've been reading and trying other things too.  I thought perhaps I could fix my spree gem to just before that commit and found helpful info at http://stackoverflow.com/questions/6119946/how-to-get-a-specific-commit-of-a-gem-from-github  (see, you were helping even if you didn't know it!).  If I've done that right, I still have the same error.... I'm quite willing to think this is an error on my part.
This will fail with TypeError if `sys.stdin.encoding` is None (which it can be, our experience from IPython suggests).
I didn't even know you were allowed to use semicolons...

Anyway 94-99 use a triple list comprehension to build a bunch of straight lines... setting visible/invisible would be a bit annoying; have to loop through whole thing.
Could just not draw it in the first place.
Stupid Tamir is stupid. :D
Could you add an item in the TODO file to add support for a new multimedia key library?
Rather than the magic number 0, better to use the stage ID constants. (Maybe flag this with a warning about bug #6463 when you do so.)
It's required that a stream is initialized (with uv_tcp_init or uv_pipe_init, etc) before uv_import is called.
@fixxolas: This is a bug in the [Mutagen library](http://code.google.com/p/mutagen/). The released version of Mutagen has [this bug](http://code.google.com/p/mutagen/issues/detail?id=52), which renders some FLAC files unreadable. Fortunately, the developers have [committed a fix](http://code.google.com/p/mutagen/source/detail?r=108). I've tested your file with the svn version of Mutagen and it seems to work fine. So you might consider installing Mutagen from svn trunk (uninstall your current mutagen with `pip uninstall mutagen`; check out the source with svn; `python setup.py install`).
I prefer `: Monad` here. It guarantees that you get one, consistent, law-abiding instance.
how about doing something like do_choice = mana.npc_choice
if it is only a wrapper anyway?

This applies to the following functions as well of course
Well, *actually*, the Jaguars definitely don't oscillate to zero. When I was tuning the Jaguars for the first time third period, the flywheel would oscillate violently all over the place given a nonzero setpoint, and immediately stop when given a zero setpoint.
The fact is that I wanted to also add support to non contiguous hair styles and color ids.
What if I'm declaring ids 1, 2 and 4, for instance? I then can't call the number 3 as it would return a default value at best, right?
Use `row['onCreate']` to be consistent.
I will put the _xsd_ file for the instances format in the _*.knowing.presenter_ bundle.
ID: _knowing.instances.xsd_ 
Make sure to pull the current master of [Knowing](https://github.com/knowing/Knowing)
A very good idea!
It is allowed to be exposed, it just cannot be used by anyone put us. We don't want two different clients for these differences.
bonjour florian
$object est déjà passé en paramètre dans la fonction executeHooks
The quotes are optional.

"^\"?([0-9]+-)?(?<opaque>.*?)\"?$"

* \"? appears before and after the old expression and will match a double quote if present.
* The match for <opaque> is now non-greedy (otherwise the trailing " is included in opaque rather than being discarded).
test ha ha
I know but if you read my page you'll see my drive crashed I just fixed it and now I'm uploading it slowly
I test and I am 90 functional %, or it had not put it
Well, seeing that Apple are moving towards OS X only, maybe reverting this would ease the balance in the force? :)
this URL is now 404 - it has been moved to http://www.pip-installer.org/en/latest/contributing.html
See inline comments. BR, Ulf
I have not checked the usages but if `null` is never used then the best solution is to remove the default value together with the cast (or test)
To do so you can implement the widget in the same as vicious.

See the following link (lines 194-200):
http://git.sysphere.org/vicious/tree/README
on peut liker les commit ? ça serait chouette.
cool, let me know how you go with it and any feedback you might have.
Should this still be = array()?
Thanks, Teoman... This has been fixed now...
This javadoc should be removed. 

This is what we got SCM for.
this has a bug, you need to get the deck with the unnormalized name, target_player_cur_name

fix is here:

https://github.com/rrenaud/dominionstats/commit/70d940cdadd4fa6a45bb73587e09bb8842ed26e6

and here (cause i was too brazen)

https://github.com/rrenaud/dominionstats/commit/d482bd6fabf05eada263b8f2bbe30d51259f53d5
Эот человек н является пользователем.
Let me know if/when it's committed to CMake master.
this is obsolete now.  we should focus on teaching mysql_wrapper about pooling, and break it into its own project with tons of tests.
Shouldn't we check both input and output streams? In this case it doesn't probably matter since they're stdin/stdout.
script/subliminal bot working anymore. 
```AttributeError: 'module' object has no attribute 'Subliminal'```
Only way I would see that would work is if they bought a domain name and pointed as subdomain to the IP address of the WAN?  And their subdomain was the same name as the host name.  Then I don't see why it should not work but if you wanted to make it work with lets say with dynamic dns hostname then I would have to add another else statement before the else statement I made as a check. Then that should work too and that would be pretty nice too.
Updated here but not in the cwVer declaration, where it really matters? Tsktsk.
This can't work, surely. "newState" is map of job name -> color. newState.name will be the color of the job called "name". 

The way to exclude disabled jobs would be to omit them in the "massage" function.
Some of our clients used it to create internal links as it was easy to copy paste. But now that the "Current Filename" is there (with path), they can use that.
Instead of reloading the whole page (with assets and all) with window.location.href = window.location.href; you could put all the initializing stuff in setup() .. and just call that.
Could you explain the need for these changes please? I have no problem accepting the -drain changes, but they follow on from this, and I'm not entirely sure we want it.
OK.
btw: U can disable e-mail notification...

On Sun, Jan 8, 2012 at 1:48 PM, mfs6174 <
reply@reply.github.com
> wrote:

> i say you two do not use this to chat!!
>  2012-1-8 1:44"bigeagle" <
> reply@reply.github.com
> >
>
> > Do not send Chinese characters via email!
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/xdlinux/xdproject/commit/d967864e37647aea72f333bccb30c191c360d124#commitcomment-847164
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/xdlinux/xdproject/commit/d967864e37647aea72f333bccb30c191c360d124#commitcomment-847170
>



-- 
Open Source,Open Mind

Blog:    http://bigeagle.me/
E-mail:  bigeagle@xdlinux.info
client *
Certainly!

And since this struct is already called `AttributeInfo`, you could probably shorten `attributeInfoTypes` to just `types`? I'm still not sure what this actually means, though.
imo make a seperate branch if your not planning to complete it soon, since it blocks the rest of the bosses  in icc ...
The two repository were totally out of sync. I don't think it is a good idea to keep those two repository separated, and allow commits, and modification on both. I suggest to set up a proper documentation system, using sphinx or another tool and drop the github wiki for anything related to the project itself.
Hi Venugh,i have tryed your code to see how exactly mmaps are looking ingame but i get a strange error x.X
src\server\game\maps\Map.h:508 in Map::getNGrid ASSERTION FAILED:
  x < MAX_NUMBER_OF_GRIDS && y < MAX_NUMBER_OF_GRIDS
[0x1d20c9]
[0x1d25da]
[0x1d2637]
[0x77624]
[0x2df442]
[0x2df5c5]
[0x22cd92]
[0x1bfef0]
[0x1c9810]
[0x2f4a12]
[0x2f4d24]
[0x2f5374]
[0x2f54d8]
[0x1d2458]
[0x20547c]
[0x2058eb]
[0x205b56]
[0x1d3424]
[0x1d5ea9]
[0x2cbe16]
[0x341b09]
ACE_Task_Base::svc_run()+0x41 [0x8a461]
ACE_Thread_Adapter::invoke_i()+0x45 [0x8aa05]
ACE_Thread_Adapter::invoke()+0x71 [0x8ab31]
endthreadex()+0x43 [0x21d9f]
endthreadex()+0xdf [0x21e3b]
BaseThreadInitThunk()+0xd [0x1652d]
RtlUserThreadStart()+0x21 [0x2c521]


any ideas what might cause this?
looks like the file spotyXBMC/thumb/SxFanart.cpp is missing
You're right Andreas. I'll update the pull request.
Each author does on their own line with a contact email address and initials
Because it is immediately obvious that it is a silent class. If referred within the declaration  it is hard to find and be known if it is so (especially if you have many declarations within).
fail, precis den tråden jag hittade och läste igenom lite men missade helt att det var du som frågat
@macfanatic I want to add the possibility to wrap index view into a form, to edit quickly collection.

But the way "insert_tag" in "render_index" works, crush form in view.

Is it possible to wrap index view with a form ?
The lrelease part could also be built into the cockatrice target, but it may not be trivial as we need to dynamically generate a second .qrc file if we don't want to compile the .qm files into the source dir.
I agree about the double negative. However, most people only want to build the client, and I believe they should be the ones who can stick to default values. Not sure though. If I call the options WITH_CLIENT and WITH_SERVER and a user calls cmake just like that, nothing happens. That might be even more confusing.
it should indeed
LGTM. I'll merge.
Всякие bodyPart можно было и как enum вынести. а вообще не вижу смысла усложнять себе задачу. можно сделать просто 1 параметр - id айтема. и всё, метод обо всём позаботится (и даже вернет прошлый в рюкзак). нэ?
Replace:
```<form method="get" id="logForm" onsubmit="return top.restoreSession()">```

Need to deal with sessions whenever open a new script or refresh the current script: http://open-emr.org/wiki/index.php/OpenEMR_System_Architecture#PHP_Sessions_and_Browser_Windows
I just tried, but `global` does not have a `body` method, `compiler` does. But `GlobalIndex` expects a `global.Tree` type, not a `compiler.Tree`. I don't see a way to make it work...
You can remove the code changes and leave only systemd stuff and please use the format "utils: add init scripts" for the message because we have two components in the same repository.
As I understood, this is what You meant.
Wouldn’t Sass complain about the braces here?
Then please remove also the debug-code in ```CacheDetailActivity``` and related layouts.
已合并，参见：提交 b6349d30da97e6fbe4d2a4698105565ad64bc333
If my phone was being cooperative, I would just make this a line comment. Line 123 of bitmap.cpp -- there shouldn't be a function call within an assert as asserts may not be compiled (ie. MSVC release mode).
Yeah that was a self comment 

Sent from my iPhone

On Feb 10, 2012, at 7:03 PM, anand kishore<reply@reply.github.com> wrote:

> i modified it to camelCase
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/semanticvoid/reDygest/commit/0ef58ebf67252501e604706e6df1b7fa65e68a91#commitcomment-955915
but i liked that one.
I appreciate the effort to seperate logic and UI, but my experience in the last two years with ActivitiyInfo and StatCloud9 is that the Presenter/View pattern often makes things worse, or at least doesn't help with complexity.

The problem that I see with this class is that division between logic and UI is entirely arbitrary: the presenter is adding UI components to a GXT layout, some events are being handled in the View, some in the presenter, it just ends up generating a lot of excess code for the interfaces and doesn't make the implementation any more clear.

In the future, let's try to avoid Presenter/View pattern *unless the logic is so complex that is necessary to write unit tests for verification* OR we need to reuse logic against several different kinds of views.  An example fullfilling both criteria is AdminFieldSetPresenter.

By contrast, what would improve this code considerably is better decomposition. Recall the Single Responsability Principle: here you have a class that is doing ten different things: fetching a report preview, switch UI views, saving/loading models etc. Better to break up into several classes that have responsiblity for one of these things.
I'm not capistrano expert but as far as I can remember, when you do "cap
setup", it should create the shared path... it's a special folder provided
by capistrano.

Pascal

On Thu, Jan 5, 2012 at 11:10 PM, David Y. Ross <
reply@reply.github.com
> wrote:

> I do have a :shared_path defined, it's just not mkdir'ed by the scripts
> nor is it used by play for logging (unless I have made some mistake...). I
> think this change is more simple than changing the other recipes to create
> the directory and send logging to it, but it's your call.
>
> Unless I have missed something...
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/dyross/play-capistrano/commit/97154f1c9ea0d7d276875b2f0c92fd6ac40ecdc8#commitcomment-841846
>
For some reason (white space) I couldn't keep the indentation as it was.
Besides that I just added a missing dependency.
How do I get the input when it’s piped? Is there a variable for it? (The fact that I didn’t know is what made me use `while read x`.)
I am not too clear about this name. Network has a very specific meaning in graph theory http://en.wikipedia.org/wiki/Network_(mathematics) - since we don't have capacities on the edges, I wonder if there would be a better name? Is this a directed acyclic graph?
What about if you want to change the size of the youtube video?
Hi there,

We're getting this error now and I'm wondering if anyone else might be seeing the same thing:


    raw_response: |
      --- !ruby/object:ActiveMerchant::Billing::Response 
      authorization: 
      avs_result: 
        code: 
        message: 
        street_match: 
        postal_match: 
      cvv_result: 
        code: 
        message: 
      fraud_review: 
      message: "Received unknown parameter: email"
      params: 
        error: 
          message: "Received unknown parameter: email"
          param: email
          type: invalid_request_error
      success: false
      test: true

Thanks!
Fixed in 1479a3e51809b62795c364a2ed4bb6f46e4deb73
With `npc_spellclick_spells` data I mean the `quest_start`,`quest_start_active`,`quest_end`,`aura_required`,`aura_forbidden` conditonal columns only, naturally.
Why move general property validation into the password validation plugin?  That doesn't seem right to me?

It does raise another interesting question, however, that maybe this validation plugin should be more general than just the password and should be a general property validation plugin.
https://github.com/symfony-cmf/symfony-cmf/commit/e4f7a86e34a9aedbf0ff8ed427d93df82e41be79#L0L12 this is no longer a valid email adress like this :-)
Here is an example that has my data removed: mongodb://heroku_app9999999:abcdabcdabcdabcdabcdabcd@ds999999.mongolab.com:99999/heroku_app9999999
Send event for the status bar change.
Now we create our list of rules in the calling method. Because all of the classes implements the Rule interface we're able to hold them all in a single rules list.
*(Same as above)*

1. You document two parameters but the method only has one.
2. The return type is `bool`.
chloe, esther
What were you saying to Pranav about commenting out code? :-)
I may not understand dependencies right, but does this not depend on express (gathering from #46)?
Ah, I should've mentioned I'm working on a new Readme with the API documentation that it better formatted for display on the homepage for Github. Should've at least committed the WIP, sorry.
Are there any actual changes here or are these whitespace changes only? Please try to avoid whitespace-only changes!
It did in my case.

My client and backend are properly configured for chunked encoding, but the proxy was dropping the header.

I had to look very closely at the headers the proxy was getting, then setting for the request to the backend to figure this out.
Just FYI, this patch breaks the Android build, there is no GL_DEPTH_COMPONENT32 in this target.
If you remove the ```{rear = []} -> None``` and change the end of the function (starting with Some (List.hd front ...)) by:

     match front with
     | [] -> None
     | hd :: front -> Some (hd, {front = front; ...})

It should work just fine. In my opinion, by adding a match case for the rear of length one (and of length zero) you speed them up, but you slow every other operations because there is one more match to do. And the other operations happen much more often.

For now, I won't do benchmarks, I'll concentrate on fingertrees.
I think power collapse state saves more energy than acpu in sleep.
I am wondering about the consequences of executing this method at runtime??? Can we add another constructor and make it final?
Assuming the user know what he is doing isn't necessary the best idea
also work for #6
^ and the Customizer recognizes the above method and is able to live with it very well.
Can you explain the problem this solves? It looks to me like this will have no effect on the actual display, because any LI will be inside an UL.
Currently when the fail-over kicks in and the retry is successful, subsequent calls to getResultCode return the initial failure result code as i_obj->rescode is not reset to RES_SUCCESS (or similar) in the code executed after the goto.

The simplest solution as I see it would probably be to handle doing so in php_memc_handle_error unless there is a specific reason this is currently skipped for the success case that I've not spotted?
What about using a Guava Splitter to split on " " ?
It didnt feel right on the tablet builds. Reverted some of it for now, but still going to go back and see if things can be organized better for both types of devices.
Weird, does this 'if' not have an 'end' somehwere??
our documentation standard typically does not make use of @param and @return. Instead we try to describe everything in one neat paragraph
Must be runRequest(request, counter+1);
Not sure if I agree with this one. By upping the required version of springSecurityCore to 1.2.7.1, we are forcing *everyone* who wants to use the SAML extension to upgrade to this version. What about others who are still using older versions of spring security core?

I would suggest leaving it as it was.
The `L.isFunction()` check here is unnecessary overhead. The parameter is documented as a function, so we can safely assume that's what it is. If the user passes in something else and things break, it's their fault for not reading the docs.
"didn't existing" should be "doesn't exist"
Is this a placeholder for future implementation, or was it no longer necessary? I see a little further down the MPI case for Global reductions is handled.
Can we make the "ring" a "nodeRing" then?
now that the bug is nailed, we might want to factor this code out, probably having Recreation and Creation extend a helper trait with `enqueue()` and `dequeueAll()` methods and then reducing this match to two cases again (and the others below).
That is true, no doubt about it. I wonder how that is part of my pull request, though … (will fix)
what if buffering is already true?
No, that will be just fine. ClusterView subscribes to all events (`ClusterDomainEvent`), so it will get ClusterMetricsChanged anyway. All I want is that you remove the trait `MetricsEvent` and change `ClusterMetricsChanged` to:

    case class ClusterMetricsChanged(nodes: Set[NodeMetrics]) extends ClusterDomainEvent
Ewww. That was even worse. Also, Symbols aren't typesafe, so a slight misspelling can trip things up good.
Just use case objects if you need states. KTHXBAI
You're right. It is unnecessary check.
Yes, I know and fully support that idea, and come to think of it. I should probably revert some of this, and make the other nodes remove themselves before exiting.

Why didn't I do that before?
IF this is renamed in config it needs renamin' here as well
Ok, I see the `after {}` statement now, but won't that just work for the first entry?

What will the test conductor think about you reusing barrier names?
DAmn these non-complied documentations
Let's call it clock so as to not confuse the readers ;-)
Yeah, I suppose I should've said it *tends* to crash my machine... I gave it a try with all my work saved, and it ended up working. So I know it def works with IE8, not to say anything of the others, because [this](http://jsbin.com/ikavex/1/edit) prints `falsetrue`.

I'll work on the test suite some time soon, though I may (or may not) need some pointers when I do. :)
Ah, I moved to a new install and forgot to "make hook"
Sorry, a silly question. What is the indentation error here?
this change seems pointless
I've add documentation about installing this
Alias would break the expected Ruby behavior. Also, I need to extract spec
to another file and add support for sorting functions.

A have a little problem with the latter, probably will work on this
tomorrow.
On Dec 13, 2012 7:32 PM, "Elia Schito" <notifications@github.com> wrote:

> In core/array.rb:
>
> > @@ -911,4 +911,16 @@ def zip(*others, &block)
> >        return result;
> >      }
> >    end
> > +
> > +  def sort
> > +    %x{
> > +      return #{self}.slice().sort();
> > +    }
> > +  end
> > +
> > +  def sort!
> > +    %x{
> > +      return #{self}.sort();
> > +    }
> > +  end
>
> return and semicolon are implicit, also I suggest to use ` in a onliner,
> like:
>
> `#{self}.sort()`
>
> or alias_native
>
> alias_native :sort!, :sort
>
>  —
> Reply to this email directly or view it on GitHub<https://github.com/opal/opal/pull/164/files#r2410316>.
>
>
do we have any default values for others that we use if not specified below? Also maybe worth mentioning what does higher concurrency weight mean.. Like users might have questions like - does more weight mean it get's more threads, more cpu cycles and hence finishes faster? does it get priority over others? Is it guaranteed to finish before lower weight tasks?
It may be best to leave the word "plugin" out of this, to avoid the confusion that led us to settle on "bundle" in the first place. Maybe s/plugins/components/ ?
I mean ACTION_SELECT to switch between the nibs in the slider.
#else (like in the android implementation) because otherwise you'll probably get a compiler warning about unreachable code if _FBDEV_WINDOW_H_ is defined.
What would you like it to return?

ElementHtmlException just returns a single line for me, which I could implement, but I think the extra information is useful (if not essential). Or I could pipe string the existing exception message if that would be better. Something like:

	return sprintf("Exception thrown by %s\n%s",
	                $this->element->getXpath(),
	                $this->pipeString($exception->getMessage()));

There is no response info available as I'm not passing in the session (at the moment, could change that).
Will do.

Aynur Abdurazik, SAIC
Ocean Biology Processing Group
NASA/GSFC Code 616  (Mail Stop 616.2)
Greenbelt, MD 20771
Phone: (301) 286-4072







On Mar 12, 2012, at 10:43 AM, Norman Fomferra wrote:

@@ -283,6 +283,50 @@ public static ColorPaletteDef loadColorPaletteDef(File file) throws IOException
    }

    /**
+     * Recomputes sample values for a color palette

This API doc is not JavaDoc conformant, please use:
@param <param-name>  <description-text>

---
Reply to this email directly or view it on GitHub:
https://github.com/bcdev/beam/pull/2/files#r546025

--
To UNSUBSCRIBE: Send an e-mail to
seadas_dev-request@seawifs.gsfc.nasa.gov
with the subject, "unsubscribe"
This method is about _display_. Scaling is a part of the _data model_. Display and model shall be decoupled, simply because there may exist multiple ways to display data.
That WMS looks really neat in `EPSG:900913`. so if you'd find the time to switch the projection, that'd be extra cool.
Not sure I understand how the comma is different that the %
Very good solution, kudos.
I agree, ill change it.
On Aug 3, 2012 4:39 PM, "Katy Huff" <
reply@reply.github.com>
wrote:

> > @@ -30,6 +31,15 @@ void FacilityModel::init(xmlNodePtr cur) {
> >      this->setInstName(inst_name_);
> >      LOG(LEV_DEBUG2, "none!") << "Facility " << ID() << " has just set
> its inst to " << inst_name_;
> >    }
> > +
> > +  // get lifetime and set decommission date
> > +  try {
> > +    fac_lifetime_ = atoi(XMLinput->get_xpath_content(cur, "lifetime"));
> > +  }
> > +  catch (CycNullXPathException e) {
> > +    fac_lifetime_ = TI->simDur();
> > +  }
> > +  setDecommissionDate(TI->time());
>
> oh, I see how this works.
> By reading how setDecommissionDate works I see that it accepts the current
> time.
>
> I would highly recommend not implementing setDecommissionDate in this way.
>
> Code should read like plain english. setDecommisionDate() should probably
> take the date of decommission as its parameter.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cyclus/core/pull/282/files#r1305881
>
Oooooh! COOKIES!!!
Because the `$filter` must be a `Twig_SimpleFilter` instance and not `$name`.  `$name` must be the name of the `Twig_SimpleFilter` instance. 
With your solution `$name` is the filter name and `$filter` is `null`.
Why not: `'test' === pathinfo($info->getFilename(), PATHINFO_EXTENSION)` ?
osName is still null at this point.

Also, throwing an exception here seems premature since we don't need the architecture at all if System.loadLibrary succeeds.
What's this about? It isn't in the OAuth 2 spec. Is it application specific to you?
less indent here
I think this breaks a lot of useful searches, for example foo::bar, foo.bar, ~foo, etc.
Space before `++`
In this case, you could. The code is pretty much taken from the get_prefix() to make it easier to implement the same structure as the prefix in a later stage.
не обязательно писать ок :)
I don't like the fact that it changes the arity of this method. Maybe something like that:

```ruby
def exists?(id = :default)
  return false if !id && id != :default
  # ...
end
```
Regular expression could be replaced with a string:

```ruby
path = controller.url_for(options).split('://', 2).last
```
@josevalim  I didn't check `html_safe_translation_key?` because translate helper do 3 different things (rescue format, scoping by partial, marking html safe). I added a test for scoping by partial `test_default_lookup_scoped_by_partial`.
why are you using stderr?
Oh, is that you you add multiple throws clauses? Has this been verified?
Ok, I see your reasoning. But then how would one match on both? Seems like it might get a bit weird/ugly/confusing...

    catch {
      case NonFatal(NonInternal(e)) => //...
    }

So that ^ is what typical user code would have to do, right?
Maybe add a doc comment?
These changes are OK. I think these routines aren't used, though.
Interesting discussion: here I just duplicated the previous behavior.  Any change would be a potential compatibility issue: before making that decision we should probably discuss it on the mailing list.
On 23.05.2012 00:30, Aaron Meurer wrote:
>> +        elif iterable(elem):
>> +            tpl = tuple(self.ring.convert(x) for x in elem)
>> +            if len(tpl) != self.rank:
>> +                raise CoercionFailed
>> +            return FreeModuleElement(self, tpl)
>> +        elif elem == 0:
>> +            return FreeModuleElement(self, (self.ring.convert(0),)*self.rank)
>> +        else:
>> +            raise CoercionFailed
>> +
>> +    def is_zero(self):
>> +        """
>> +        Returns True if ``self`` is a zero module.
>> +
>> +        (If the coefficient ring is not the zero ring, as this implementation
>> +        assumes, then this is equivalent to the rank being zero.)
>
> This is ambiguous. When I first read it, I thought you meant that it assumes that the coefficient ring is the zero ring.
>

Changed the wording.
This is the "warning" I am referring to.
Anyway, in the future it would be great to extend the potential implementation to treat two body potentials :)
Use XFAIL.
How about just use the `else` clause to the for loop.
Its convenient for the RGS algorithms. I'll rebase my branch on top of your 1164 pull request.
@cebe Okay, got it.
why not `return $this->_items[$this->_currentIndex];` ?
Nevermind, I misread the sed command.
You should use the Padrino route helpers for intra-application links.

For example, replace `/acm/conference` with `#{url_for(:acm_conference)}` and adjust the ACM controller file to recognize that route with

    get :acm_conference, :map => 'acm/conference' do
        ...
    end
Belay that for now, we can address it in a later commit.
this is really a note.
Doing what exactly?  This should work for template creation and WYSIWYG usage.
Il y a un probleme avec ca, relie au script error que t'as trouve aussi. L'adresse "root" d'une page correspond toujours a la page html  en question et toute inclusion de script est cherchée par rapport a cette page. L'adresse de require.js cherche par la page de service tel qu'ecrit a la ligne 11 serait:```service/static/js/other/require.js```. Ce n'est pas la bonne adresse pourtant. On voudrait plutot ```/static/js/...```.

Dans ma dernière version, j'ai change l'adresse des scriptes a ```../static/js/other/require.js```, ce qui regle le probleme pour toutes les pages a part la page d'accueil qui est ``` / ```. Pour la page d'accueil, l'addresse des scriptes doit rester telle qu'elle est.

Cette solution implique que la ligne ```<script data-main="../static/js/main" src="../static/js/other/require.js"></script>``` doit apparaître dans toutes les pages qui ne sont pas la page d'accueil; a moins de trouver une solution plus élégante bien sur.
> EDIT: In fact, maybe also last sentence should be rewritten, because "should" is not proper verb. You cannot use trailing comma, because of IE.

Correct.
Missing `()` here.
Is this supposed to return "test"? Looks like some development code that isn't removed yet.
You don't need to traverse the filesystem here, you can use `$files` and `file_exists()` instead now.
nice, this is super sweet
Is closured. How are you going to test that function?
Why not just requires['handlebars'] and then pull var HB from Y instance?
Minor: magic numbers, consider moving to init()
The class and it's members are missing XML comments
I forget to add that commit to https://github.com/NixOS/nixpkgs/pull/92
Want to abstract this out to a constant
It's a required field for POSTing. If I take it out, then javascript REST throws an exception, which is probably better. Taking the check out.
I didnt removed it...
may be some error, I check it again.
Your validation can be done in a more condensed fashion by putting them into the model http://senchaexamples.com/2012/03/22/validating-an-ext-data-model-object-using-sencha-touch-2/ . (I'm also against pop-ups, so prefer to avoid alert boxes if possible! :)

We are a bit inconsistent on this currently, e.g. see how allowBlank parameter is used in Registration https://github.com/Raxa/Raxa-JSS/blob/master/src/registrationextjs4/app/view/RegistrationPart1.js . My impression is that model based validation is the most rigorous, organized approach. http://docs.sencha.com/touch/2-0/#!/api/Ext.data.Model (http://docs.sencha.com/touch/2-0/#!/api/Ext.data.Model-cfg-validations)
Now that you've rebased, this CML file is out of date.
The only file in the current sample data set which already contains bounds is `hybrid_height.nc`. This is 3-dimensional, so we'd have to take a single slice.
```
air_potential_temperature           (atmosphere_hybrid_height_coordinate: 15; grid_latitude: 100; grid_longitude: 100)
     Dimension coordinates:
          atmosphere_hybrid_height_coordinate                           x                  -                    -
          grid_latitude                                                 -                  x                    -
          grid_longitude                                                -                  -                    x
     Auxiliary coordinates:
          model_level_number                                            x                  -                    -
          sigma                                                         x                  -                    -
          surface_altitude                                              -                  x                    x
     Derived coordinates:
          altitude                                                      x                  x                    x
     Scalar coordinates:
          forecast_period: 0.0 hours
          time: 347921.166667 hours since 1970-01-01 00:00:00
     Attributes:
          Conventions: CF-1.5
          STASH: m01s00i004
          source: Data from Met Office Unified Model 7.04
```
*"...from it's coordinate system..."* **as long as it's in latlon or rotated latlon**.
importing traverse is not needed.
Yes, it will -- it says "axes should be smaller than the dimension of x and y". If we were writing tensordot from scratch, I'd argue for no default, because with the exception of 0 there is no default argument that is guaranteed not to raise an error -- and 0 is an unlikely use case. However, 2 is the default argument in numpy and I think that wherever possible, numpy and theano should have similar signatures -- that certainly helped me when I was learning theano. I think it would be confusing to someone if, when testing some x and y, np.tensordot raised an error with its default axes argument but tt.tensordot did not, even if that was because theano was doing some helpful magic behind the scenes.
Spelling/meaning: "totald"? Also, cut and paste error -- many options described identically as" the number of totald hosts used in this test".

(End of Sep 1 2012 review comments -- I don't consider any of the above points "show-stoppers").
I think that as parameter you must put "id" instead of 1l
This is okay, but generally I think being more explicit is a bit better for non-bool types - e.g. `if(file_stream != NULL)`
+1 @jacobian
Should be with locking, since otherwise the row could be deleted in that timeframe.
you should use ``&&`` here instead of nesting 2 ``if``
[SemVer](http://semver.org/) names '-' as separator for prerelease- and '+' for build- identifier. 
[gem versionomy](https://github.com/dazuma/versionomy) handels many different formats. 
Hard to say what's the "right way" to do it.
This is going to break on Ruby 1.8.7:

```ruby
"44"[0] #=> 52
```

This should work on both:

```ruby
"44"[0].chr #=> "4"
```
I have no idea why we were testing an undocumented interface.
It might be safer for future maintenance to keep the early return even though using `notxml` in two if statements may compress better.
From the PR description: "I did not include unit tests, however, because those would require us to be testing SVG and MathML, and is not something I'm interested in. I'm interested in fixing this regression and including an explanatory comment. If this means it doesn't get merged, so be it."
@Alamoz You must create a pull request to rails/rails for this kind of changes.
fixed. Using _ as prefix to indicate internal function.
Maybe it's better to do this in the caller, if possible?
Should this be an elif?
Hmm. Good point. I think we can push this off until later though, when we actually start generating these reports. There might also be some privacy concerns about email and elasticsearch.
I had to check the Scrapy docs to figure out what the heck the values in this dict are, and why we'd have both Offsite Middlewares listed here. Might be worth a one-line comment to explain that the `None` disables the built-in version.
At this point things without a versioned download aren't really acceptable for core; they would be better off in a tap.
Traditionally we have defined these kind of methods outside the class definition; avoiding `self.class.method` calls.
Please fix the indentation here:

```rb
args = %W[./Configure
          --prefix=#{prefix}
          --openssldir=#{etc}/openssl
          zlib-dynamic
          shared]
```
Don't stick a temporary directory in `prefix`. The entire build is already staged in a temporary directory.
I don't think there's a requirement, but I feel it is better to "isolate" non-lisp files from lisp files. It seems to be more clean to me
Probably not. I'll fix that.
Is there a way you can make this class implement all of the other codegen-time interfaces? I don't think we need to divide it up by feature: Codegen linker can do codegen-static injections!
accountPushName is not necessary to construct the UI, so I'll set it from a separate function
変数名形容詞単体で変な感じがします。下の関数だとisVisibleなので、そちらに合わせたほうが良い気がします。
Updating RequireJS
The question at the beginning is a bit awkward. Let's incorporate it into the sentence and make the button "Clear Base Path", e.g.:

"Clearing the base path will cause new attachments to be stored using absolute paths."

"Clearing the base path will convert %S attachments to absolute paths. New attachments will be stored using absolute paths."

(For the first one, if we think it's important, we can add "No existing attachments were found beneath the existing base path.".)
style-nit: For multi-line Javadoc comments, the closing */ should be on its own line.

See http://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#format
yeah it work, but it's not usual in maven central.
the companies use the range of version but i haven't seen any maven central artifact like this.
imho with strick version we are on safe side if hamcrest makes any incompatible changes
the user of junit may still exclude or override junit's dependency of hamcrest-core:1.3 to other version

are you facing a need for, that it would help you in your use of junit?
maybe [1.3, 2.0) is more safe
The program will not run properly if you remove this line. When it finds that curEval is less than bestEval, it has to set bestEval to curEval because curEval represents the new best.
Because that's not what he was testing?
the spacing here is weird now that we don't have a ternary.
Not sure if we need this? Couldn't we just called dojo.xhr in _getRemoteData anyway?
fixed

-----Original Message-----
From: Randy Edmunds [mailto:reply@reply.github.com] 
Sent: Thursday, July 12, 2012 2:28 PM
To: Ty Voliter
Subject: Re: [brackets] HTML Tag Code Hinting (#1236)

> @@ -160,11 +160,13 @@
>  
>  
>  /* Menu-related styles */
> -.toolbar .nav, .context-menu {
> -    @menubar-top-padding: 8px;
> -    @menubar-bottom-padding: 6px;
> -    @menubar-h-padding: 9px;
> -    @item-highlight-color: #e0f0fa;
> +
> +@menubar-top-padding: 8px;
> +@menubar-bottom-padding: 6px;
> +@menubar-h-padding: 9px;
> +@item-highlight-color: #e0f0fa;
> +
> +.toolbar .nav, .context-menu, .codehint-menu {
>      
>      // The 1px adjustments here are to account for the border around the top-level menu items.
>      margin: (@toolbar-top-gap-px - @menubar-top-padding - 1) 0 (-@menubar-bottom-padding + 1) (-@menubar-h-padding + 4);

You can use "padding: 0;" as a shortcut.


---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/1236/files#r1154323
var ``i`` shadowing, see prior comments
Indeed. And now I'm doubting my sanity because I **did** have the '\' in there. I must have inadvertently deleted it before pushing. Thanks for catching that!
I was using it in a similar way that the other command in this file is using. So removing now.
maybe like this ```"0%s" % Response::CRLF * 2```
note to self: changed
Style note, I would use "See `package-unpack-single'". Do you know if that works with package.el for Emacs23?
Shouldn't this be `#reject:target`?
.pick 其實在這裡沒有用到的話   就可以拿掉了
*python* -> *Python*
Don't use `*` in field labels for required fields!
Might as well stay consistent here and add spacing.
You should use `track.getTrackEventById( trackEvent.id )` here!
Can you fix this white space?
time = -time is actually not holding state, but making the negative time positive so we can do all our time converting against that, and convert it back if needs be.

If not, we end up with cases like this.

-120 convert.

Expected: "-2:00"

Actual:  "-1:0-2:00"
I checked out the behaviour here, it looks like we can just leave out `editorElement.type = "text";` fallback.

IE9 does default to text, but it also throws that exception, so we can just suppress it. just add a comment that says: `// suppress type errors from IE9` or something.
This syntax, while compact is hard to read at a glance.
yes, much nicer!
I've come across this problem too. Definitely your solution is a wise one.
I'd use BufferedOutputStream here.
Yeah, there's no reason to use it: https://www.owasp.org/index.php/HttpOnly
small  whitespace issue here
Added a pull request for this to you. I'd like to get your comments on the above though.
change `d` to `hub`.
I would like that. The simple solution is to save that into an indexedDB backend. The real fix is probably to alter the photo file.
It's not necessarily bad to have this section, but it's probably safer to document the methods only in their own docstrings.  Sphinx will generate the method documentation on the same page, and this way, if someone later on adds a new method, you don't have to worry that they'll forget to add it to this list.
Another 5 second default cross-node call. Opinions on this one?
Should this be non_neg_integer() too, like health_failures?
In case there is not enough disk space to do pre-allocation, we probably want to fail gracefully, instead of not just preallocating.
No worries, it's removed.
Most of these elements that you have changed to classes should remain ID's (if there is only ever going to be one of them).
you can type these all out if you want, but convention is to put a `a = row.attrib` beforehand and then use that shorthand
params =
Break these across multiple lines please, don't want to have to scroll to view a diff
Passing simulation twice looks weird -> fishy
I'm not really familiar with the components used here, but I find it a bit confusing that for a class that has to do with doctrinePhpCr, an exception class from symfony is imported. Especially since in the code below, two time PHP's vanilla \RunTimeException is used, and then one time the RuntimException from symfony. 

Like I said, I don't really know how this all fits together, but I was always under the impression that doctrine and symfony were two sepearate entities, i.e. that you could use doctrine (and therefore potentially this class) without symfony. If that's the case, then creating an interdependency with the exception class seems unnecessary
A description of what this function does would be helpful.
This wouldn't be needed with two paths.
Looks ok at my end, will have a look again later,
yup, makes sense. I put in a detection so it won't show fallback unless the mp4 URL is on our CDN.
Doh, I submitted my comment after you updated the code but before Github updated my view, this is the `those` that needs to be a `these`. :P
A pull request comment?
There is no need for a `$this->here` for the last crumb since it is the curretn page. Update the others occurrences too.
Since the return values are essentially the same, I would factor out a `principal_type` function and put the ejson literals in one body.