Skip to content
This repository has been archived by the owner on Dec 10, 2018. It is now read-only.

all: migrate quad to integrate/quad #20

Merged
merged 1 commit into from Jun 2, 2016
Merged

all: migrate quad to integrate/quad #20

merged 1 commit into from Jun 2, 2016

Conversation

sbinet
Copy link
Member

@sbinet sbinet commented Jun 2, 2016

This CL implements the migration of github.com/gonum/quad to
github.com/gonum/integrate/quad.

Updates #17.


// Package integrate provides functions to compute an integral given a
// specific list of evaluations.
package integrate // "github.com/gonum/integrate"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the vanity import necessary?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not necessary but it will make the error more obvious for people doing:

$ cd github.com/gonum/quad
$ go get -u 

I had the same issue with this package:
nanomsg/mangos-v1#184

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it'll all change anyway when we move to the new location :)

This CL implements the migration of github.com/gonum/quad to
github.com/gonum/integrate/quad.

Updates #17.
@btracey
Copy link
Member

btracey commented Jun 2, 2016

LGTM

@sbinet sbinet merged commit 3d99f16 into gonum:master Jun 2, 2016
@sbinet sbinet deleted the quad-integrate-migration branch June 2, 2016 14:34
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

2 participants