Adding new projects to experimental
Purpose
The folder experimental
is for code that is candidate for being added to Oscar. In particular, this includes the following cases:
- Code from an external package that should be moved to Oscar
- Implementing a new feature from scratch
- In general: code whose API has not stabilized yet
The code in experimental
is supposed to be mathematically correct, experimental
is a staging area for new features whose interface is still to be stabilized. Also code is allowed to reside in experimental
while it is brought up to Oscar standard.
- Code from
src
must never use code fromexperimental
- Say there are two packages
A
andB
inexperimental
, andB
depends onA
. That means thatB
cannot be moved tosrc
beforeA
. Worse: IfA
gets abandoned,B
might share that fate. So please consider carefully in such situations.
Structure
For an example of the structure for a new project in experimental
have a look at project folders, i.e. experimental/PACKAGE_NAME
, that have subfolders docs
, src
, and test
(The first two examples are experimental/{FTheoryTools, PlaneCurve}
). The general structure is
experimental/PACKAGE_NAME/
├── README.md
├── docs
│ ├── doc.main
│ └── src
│ └── DOCUMENTATION.md
├── src
│ └── PACKAGE_NAME.jl
└── test
└── runtests.jl
The files src/PACKAGE_NAME.jl
and test/runtests.jl
are mandatory as they are used by Oscar.jl to find your code and tests. The file docs/doc.main
is used for integrating your documentation in the Oscar manual under the Experimental
section. Optionally please provide a README.md
describing your project and its goals, especially if you are starting from scratch and don't have any documentation yet.
There are still older projects in experimental
from before the introduction of this structure. Thus we mentioned FTheoryTools
and PlaneCurve
as projects having adopted to the new standard.
Useful functions for development
Apart from the hints in the Introduction for new developers, there are some more specialized functions for the structure of the experimental
folder.
test_experimental_module
— Functiontest_experimental_module(project::AbstractString; file::AbstractString="runtests", new::Bool = true)
Run the Oscar tests in the file experimental/<project>/test/<file>.jl
where file
may be a path. The default is to run the entire test suite of the module project
.
The optional parameter new
takes the values false
and true
(default). If true
, then the tests are run in a new session, otherwise the currently active session is used.
Procedure for adding a new feature
Ideally we envision the procedure to follow along the following lines.
- The new feature is implemented in the
experimental
folder. - For external authors, a maintainer is assigned to guide the authors such that the implementation adheres to the Developer Style Guide and the Design Decisions. Please get in touch with us as soon as possible, preferably on the OSCAR Slack.
- The new feature is tested thoroughly, other people are invited to test the new feature.
- In the end there are three possibilities:
- The feature is considered done and moved into
src
as is. - The feature is discarded, e.g., because it cannot be maintained.
- Parts of the feature are moved into
src
, others are discarded.
- The feature is considered done and moved into
Criteria for acceptance
The main criteria for acceptance are:
- The code adheres to the Developer Style Guide and the Design Decisions.
- The new code is well tested.
- It is clear who maintains the new code, i.e. the original authors commit to maintaining the code in the future.