Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

Adding Funs to METADATA #315

Closed
wants to merge 1 commit into from
Closed

Adding Funs to METADATA #315

wants to merge 1 commit into from

Conversation

dlfivefifty
Copy link
Contributor

Adding the Funs package, for Chebyshev approximation

@StefanKarpinski
Copy link
Member

This definitely needs a better name. Maybe ChebyshevApprox or ChebFun?

@StefanKarpinski
Copy link
Member

Also, the repo name should match the package name with .jl appended.

@dlfivefifty
Copy link
Contributor Author

It will include Fourier Fun eventually, so neither name is appropriate. How about

FunctionApproximation?

Sent from my iPad

On 07/07/2013, at 0:57, Stefan Karpinski [email protected] wrote:

This definitely needs a better name. Maybe ChebyshevApprox or ChebFun?


Reply to this email directly or view it on GitHub.

@StefanKarpinski
Copy link
Member

Let's call it something appropriate to the current functionality. If and when it takes on a broader scope, we can consider renaming it or splitting it into several pieces.

@dlfivefifty
Copy link
Contributor Author

I just added the Fourier and Laurent series functionality

Sheehan

On 06/07/2013, at 3:06 PM, Stefan Karpinski [email protected] wrote:

Let's call it something appropriate to the current functionality. If and when it takes on a broader scope, we can consider renaming it or splitting it into several pieces.


Reply to this email directly or view it on GitHub.

@StefanKarpinski
Copy link
Member

Ok, let's call it FunctionApproximation.jl then. The name is long, but that's ok since it only needs to be used on import.

@kmsquire
Copy link
Member

kmsquire commented Jul 8, 2013

I have no particular problem with the name, but, e.g., when using many packages, I tend to use import rather than using (especially while debugging)

import GZip
import Stats
import FuncationApproximation
# other package imports here

...
f = FunctionApproximation.Fun(exp,[-1,1])
...

This gets a little long. Of course, one can do

import FunctionApproximation

FA = FunctionApproximation

f = FA.Fun(exp,[-1,1])

I often do this, though it seems a little kludgy.

See also: JuliaLang/julia#1255

@JeffBezanson
Copy link
Member

That works well; you will want to make FA const though.

@StefanKarpinski
Copy link
Member

We might want to make import FunctionApproximation as FA a syntax for

import FunctionApproximation
const FA = FunctionApproximation

except without binding FunctionApproximation to anything. I know it's not strictly necessary, but it would be rather convenient. I'm open to some reasonable abbreviation for this package name, but Funs is way too short. I feel that FunctionApprox would be good.

@dlfivefifty
Copy link
Contributor Author

How about ApproxFun?

Sent from my iPhone

On 08/07/2013, at 3:56 PM, Stefan Karpinski [email protected] wrote:

We might want to make import FunctionApproximation as FA a syntax for

import FunctionApproximation
const FA = FunctionApproximation
except without binding FunctionApproximation to anything. I know it's not strictly necessary, but it would be rather convenient. I'm open to some reasonable abbreviation for this package name, but Funs is way too short. I feel that FunctionApprox would be good.


Reply to this email directly or view it on GitHub.

@StefanKarpinski
Copy link
Member

I like it – especially since it's reminiscent of chebfun, which is what people will probably be most familiar with.

@toivoh
Copy link
Contributor

toivoh commented Jul 9, 2013

+1 for import as

@ViralBShah
Copy link
Member

+1 for new name and import as.

@dlfivefifty
Copy link
Contributor Author

OK Changed to ApproxFun. What do I do now?

Sheehan

On 08/07/2013, at 11:20 PM, Viral B. Shah [email protected] wrote:

+1 for new name and import as.


Reply to this email directly or view it on GitHub.

@StefanKarpinski
Copy link
Member

You can either amend this commit and force push to this same pull request or you can open a new pull request.

@Keno
Copy link
Member

Keno commented Aug 1, 2013

Merged as ApproxFun

@Keno Keno closed this Aug 1, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants