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

Pull request/19845a6e #2537

Closed

Conversation

fairbrot
Copy link
Contributor

@fairbrot fairbrot commented May 2, 2015

No description provided.

@IainNZ
Copy link
Member

IainNZ commented May 2, 2015

Hi @fairbrot,
Looks like a neat package. Two things:

  • The name. There is a preference to err on the side of clarity, so GaussianProcesses.jl would be a really great choice for a name.
  • Tagging. You actually need to tag a version to be able to Pkg.add a package. This can be done after registration or at the same time.

@kmsquire
Copy link
Member

kmsquire commented May 3, 2015

Is this meant to be the canonical Gaussian Process package? If not necessarily, then it might not be good to take that name.

@kmsquire
Copy link
Member

kmsquire commented May 3, 2015

It might also be fine. There are many possible ways to implement decision trees, and one of them happens to be implemented in the DecisionTrees.jl package.

@ViralBShah
Copy link
Member

I think the generic name is ok here.

@ViralBShah
Copy link
Member

I mean GaussianProcesses is ok.

@johnmyleswhite
Copy link
Member

I think names like GaussianProcesses are fine if we develop some community standards that reflect who owns what. If we allow some kind of eminent domain rulings on names like DataFrames or Stats, then the names are safe to use.

@fairbrot
Copy link
Contributor Author

fairbrot commented May 4, 2015

Hi everyone, thanks for the feedback. We're happy to change the name of the package to GaussianProcesses. We'll update this over the next few days,

@fairbrot
Copy link
Contributor Author

fairbrot commented May 8, 2015

Pull request has been recreated at #2561

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.

None yet

5 participants