[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [octave forge] new external package
From: |
Julien Bect |
Subject: |
Re: [octave forge] new external package |
Date: |
Tue, 29 Aug 2017 19:32:51 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
Le 20/08/2017 à 23:21, JuanPi a écrit :
I want to contribute the gpml (Gaussian Processes for Machine
Learning) package to the octave ecosystem.
I keep a repository here[1], which is a snapshot of the released version [2].
Since the original developer contributed (and might do it in the
future), I keep the structure of the original code, but provide an
extra folder 'pkg_octave' with files used by 'Makefile' to generate
the package, i.e. only the result of 'make release' has the octave
forge package structure.
I am still struggling to converge to a method to include test and
demos and to improve the documentation. Oliver has gave me some nice
clues, but any suggestion is welcome. See [3] for the mailing list
thread.
In [4] you will find a file that should be installable using 'pkg
install gpml-4.0.0'
Regards,
JPi
[1]: https://bitbucket.org/KaKiLa/gpml
[2]: http://gaussianprocess.org/gpml/
[3]:
http://octave.1599824.n4.nabble.com/OF-Docs-Demos-and-Test-for-External-package-td4681888.html
[4]:https://bitbucket.org/KaKiLa/gpml/downloads/gpml-4.0.0.tar.gz
Hi JPi,
Cool. Having GPML readily available in Octave would be nice. A few
ideas / remarks :
* I wouldn't block the release for problems related to tests / demos /
etc. You can improve these aspects later.
* Is startup.m (at the root) needed in the octave package ? If not, I
would remove it. If yes, perhaps rename it to a less generic name ?
* The developper manual should be in /doc, not inst/doc, I think.
* The m-files that only provide documentation (such as meanFunctions.m,
covFunctions.m...) could perhaps be renamed with a "gpml_" prefix. This
would make them much easier to find using completion.
* Same remark for demoClassificaton, usagePrior, etc.
* inst/doc/changelog -> ChangeLog (at the root) ?
@++
Julien