Contributing to Manopt.jl
First, thanks for taking the time to contribute. Any contribution is appreciated and welcome.
The following is a set of guidelines to ManoptExamples.jl
.
Table of Contents
I just have a question
The developer can most easily be reached in the Julia Slack channel #manifolds. You can apply for the Julia Slack workspace here if you haven't joined yet. You can also ask your question on our GitHub discussion.
How can I file an issue?
If you found a bug or want to propose a feature, we track our issues within the GitHub repository.
How can I contribute?
Add an objective
The objective in Manopt.jl
represents the task to be optimised, usually phrased on an arbitrary manifold. The manifold is later specified when wrapping the objective inside a Problem.
If you have a specific objective you would like to provide here, feel free to start a new file in the src/objectives/
folder in your own fork and propose it later as a Pull Request.
If you objective works without reusing any other objective functions, then they can all just be placed in this one file. If you notice, that you are reusing for example another objectives gradient as part of your objective, please refactor the code, such that the gradient, or other function is in the corresponding file in src/functions/
and follows the naming scheme:
- cost functions are always of the form
cost_
and a fitting name - gradient functions are always of the
gradient_
and a fitting name, followed by an!
for in-place gradients and by !!
if it is a struct
that can provide both.
It would be great if you could also add a small test for the functions and the problem you defined in the test/
section.
Add an example
If you have used one of the problems from here in an example or you are providing a problem together with an example, please add a corresponding Quarto Markdown file to the examples/
folder. The Markdown file should provide a short introduction to the problem and provide links to further details, maybe a paper or a preprint. Use the bib/literature.yaml
file to add references (in CSL_YAML
, which can for example be exported e.g. from Zotero).
Add any packages you need to the examples/
environment (see the containting Project.toml
). The examples will not be run on CI, but their rendered CommonMark
outpout should be included in the list of examples in the documentation of this package.
Code style
We try to follow the documentation guidelines from the Julia documentation as well as Blue Style. We run JuliaFormatter.jl
on the repo in the way set in the .JuliaFormatter.toml
file, which enforces a number of conventions consistent with the Blue Style.
We also follow a few internal conventions:
- Any implemented function should be accompanied by its mathematical formulae if a closed form exists.
- within a file the structs should come first and functions second. The only exception are constructors for the structs
- within both blocks an alphabetical order is preferable.
- The above implies that the mutating variant of a function follows the non-mutating variant.
- There should be no dangling
=
signs. - Always add a newline between things of different types (struct/method/const).
- Always add a newline between methods for different functions (including in-place/non-mutating variants).
- Prefer to have no newline between methods for the same function; when reasonable, merge the docstrings into a generic function signature.
- All
import
/using
/include
should be in the main module file. - There should only be a minimum of
export
s within this file, all problems should usually be later addressed asManoptExamples.[...]
- the Quarto Markdown files are excluded from this formatting.