-
Notifications
You must be signed in to change notification settings - Fork 8
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #237 from nfdi4plants/developer_Docs
Update README.md 📚
- Loading branch information
Showing
6 changed files
with
686 additions
and
617 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,7 +3,7 @@ | |
| "isRoot": true, | ||
| "tools": { | ||
| "fable": { | ||
| "version": "4.1.4", | ||
| "version": "4.4.1", | ||
| "commands": [ | ||
| "fable" | ||
| ] | ||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,280 +1,53 @@ | ||
| # ARCtrl | ||
| Top level ARC DataModel and API function descriptions. | ||
|
|
||
|  | ||
|  | ||
| > **ARCtrl** the easy way to read, manipulate and write ARCs in __.NET__ | ||
| and __JavaScript__! ❤️ | ||
|
|
||
| - [ARCtrl](#arctrl) | ||
| - [Jargon/Nomenclature](#jargonnomenclature) | ||
| - [API Design](#api-design) | ||
| - [Top level overview](#top-level-overview) | ||
| - [Stack](#stack) | ||
| - [Libraries](#libraries) | ||
| - [Design choices](#design-choices) | ||
| - [Fable compatibility as top priority](#fable-compatibility-as-top-priority) | ||
| | Version | Downloads | | ||
| | :--------|-----------:| | ||
| ||| | ||
| ||| | ||
|
|
||
| ## Sub-Libraries | ||
| ## Install | ||
|
|
||
| [ISA README.md](src/ISA/README.md) :books: | ||
| (currently only prereleases available, check the [nuget page](https://www.nuget.org/packages/ARCtrl) or [npm page](https://www.npmjs.com/package/@nfdi4plants/arctrl) respectively) | ||
|
|
||
| [Contracts README.md](src/Contract/README.md) :books: | ||
| ### .NET | ||
|
|
||
| # Jargon/Nomenclature | ||
|
|
||
| In general, a distinction is made between `DataModel`s, `API`s, and `Tools`: | ||
| - `DataModel`s are the data structures which represent the ARC or it's respective parts in memory. They are serializable and can be used as data exchange format between tool implementations: | ||
| - `FileSystem`: Represents the file system structure of an ARC. All files and their path relative to the arc root folder are contained here. | ||
| - `ISA`: Represents the experimental metadata of the ARC that is stored in the ISA-XLSX format (investigation, studies and assays). | ||
| - `CWL`: Represents the workflow definitions of the ARC in the CWL format. | ||
| - `API`s are static methods on the `DataModel` types that perfrom operations on the `DataModel`s. Often, these are CRUD-like operations, and are aimed to be be composable. | ||
|
|
||
| **Example**: A `ARC.addAssay` function has to do several things: | ||
| - Add a new assay to the `ISA` | ||
| - Add a new assay to the `FileSystem` | ||
| it should therefore combine the respective functions of `ISA API` and `FileSystem API` to achieve this. | ||
|
|
||
| - `Tools` or `Clients` are user-facing software such as Swate, ARCitect, or the ARCCommander. They should ideally compose their functionality from the `API`s and work with an in-memory representation of the ARC via `DataModel`s. There are operations such as IO for reading/writing actual files to the file system, which are not part of the `API`s, but rather part of the `Tools`. | ||
|
|
||
| ## Local Setup | ||
|
|
||
| 1. `dotnet tool restore` | ||
| 2. `npm install` in *root* (this will also run `npm install` in `src/ISA`). | ||
|
|
||
| ## API Design | ||
|
|
||
| Command syntax should be inspired by ArcCommander commands, as they are already well established and known to the power user base. | ||
| See syntax : https://nfdi4plants.github.io/arcCommander-docs/docs/01GeneralCLIStructure.html | ||
| See selection: https://nfdi4plants.github.io/arcCommander-docs/docs/02SubcommandVerbs.html | ||
|
|
||
| ## Top level overview | ||
|
|
||
| In the following, the dependency graph of the proposed arcAPI toolstack can be seen: | ||
|
|
||
| ```mermaid | ||
| flowchart TD | ||
| %% ----- Nodes ------ | ||
| subgraph ARCtrl | ||
| arc[ARC] | ||
| subgraph ISA | ||
| isaj[ISA.Json] | ||
| isax[ISA.Spreadsheet] | ||
| isaa[ISA] | ||
| end | ||
| subgraph FileSystem | ||
| fs[FileSystem] | ||
| end | ||
| subgraph CWL | ||
| cwl[CWL] | ||
| end | ||
| end | ||
| subgraph DataTransferObject | ||
| fsspread[FsSpreadsheet] | ||
| thoth["Thoth.Json (string)"] | ||
| end | ||
| subgraph IO | ||
| node[node.fs] | ||
| fsspreadx[FsSpreadsheet.ExcelIO] | ||
| excelJS[exceljs] | ||
| systemio[System.IO] | ||
| end | ||
| %% ----- Edges ------ | ||
| %% Arc | ||
| arc --> ISA | ||
| arc --> FileSystem | ||
| arc --> CWL | ||
| %% ISA | ||
| isaj --> isaa | ||
| isax --> isaa | ||
| isaj --> thoth | ||
| isax --> fsspread | ||
| %% ------ Tools --------- | ||
| subgraph Tools | ||
| arcdotnet[/arcIO.NET\] | ||
| arcitect[[ARCitect]] | ||
| arccom[[arcCommander]] | ||
| swate[[Swate]] | ||
| end | ||
| %% ----- Edges ------ | ||
| Tools --> ARCtrl | ||
| arcdotnet --> fsspreadx | ||
| arcdotnet --> systemio | ||
| arcitect --> excelJS | ||
| arcitect --> node | ||
| arccom --> arcdotnet | ||
| fsspreadx -.-> fsspread | ||
| systemio -.-> thoth | ||
| excelJS -.-> fsspread | ||
| node -.-> thoth | ||
| %% ----- Styling ------ | ||
| %% dotnet | ||
| style Tools stroke:#3A0EC6,stroke-width:2px | ||
| style fsspreadx fill:#8C71E3,stroke:#3A0EC6,stroke-width:2px,color: black | ||
| style arcdotnet fill:#8C71E3,stroke:#3A0EC6,stroke-width:2px,color: black | ||
| style arccom fill:#8C71E3,stroke:#3A0EC6,stroke-width:2px,color: black | ||
| style systemio fill:#8C71E3,stroke:#3A0EC6,stroke-width:2px,color: black | ||
| %% javascript | ||
| style excelJS fill:#C1AD09,stroke:#EFD81D,stroke-width:2px,color: black | ||
| style arcitect fill:#C1AD09,stroke:#EFD81D,stroke-width:2px,color: black | ||
| style swate fill:#C1AD09,stroke:#EFD81D,stroke-width:2px,color: black | ||
| style node fill:#C1AD09,stroke:#EFD81D,stroke-width:2px,color: black | ||
| %% fable | ||
| style ARCtrl stroke:#007B00,stroke-width:2px | ||
| style ISA stroke:#007B00,stroke-width:2px | ||
| style FileSystem stroke:#007B00,stroke-width:2px | ||
| style CWL stroke:#007B00,stroke-width:2px | ||
| style DataTransferObject stroke:#007B00,stroke-width:2px | ||
| ```fsharp | ||
| #r "nuget: ARCtrl" | ||
| ``` | ||
|
|
||
| style fsspread fill:#39B539,stroke:#007B00,stroke-width:2px,color: black | ||
| style isax fill:#39B539,stroke:#007B00,stroke-width:2px,color: black | ||
| style isaa fill:#39B539,stroke:#007B00,stroke-width:2px,color: black | ||
| style isaj fill:#39B539,stroke:#007B00,stroke-width:2px,color: black | ||
| style arc fill:#39B539,stroke:#007B00,stroke-width:2px,color: black | ||
| style fs fill:#39B539,stroke:#007B00,stroke-width:2px,color: black | ||
| style cwl fill:#39B539,stroke:#007B00,stroke-width:2px,color: black | ||
| style thoth fill:#39B539,stroke:#007B00,stroke-width:2px,color: black | ||
| ```bash | ||
| <PackageReference Include="ARCtrl" Version="1.0.0-beta.1" /> | ||
| ``` | ||
|
|
||
| ## Stack | ||
|
|
||
| ```mermaid | ||
| classDiagram | ||
| class arc["ARCtrl"] { | ||
| ISA | ||
| CWL | ||
| FileSystem | ||
| } | ||
| class isa["ISA"] { | ||
| Investigation | ||
| Studies | ||
| Assays | ||
| ArcTable | ||
| CompositeColumn | ||
| CompositeHeader | ||
| CompositeCell | ||
| } | ||
| class file ["FileSystem"] { | ||
| - FileSystemTree | ||
| - History | ||
| } | ||
| class cwl["CWL"] { | ||
| CommandlineTool? | ||
| Workflows | ||
| } | ||
| class io["IO"] { | ||
| FsSpreadsheet.ExcelIO | ||
| System.IO | ||
| exceljs | ||
| node fs | ||
| } | ||
| io <|--|> arc : Contracts | ||
| arc <|--|> isa | ||
| arc <|--|> cwl | ||
| arc <|--|> file | ||
| ### JavaScript | ||
|
|
||
| ```bash | ||
| npm i @nfdi4plants/arctrl | ||
| ``` | ||
|
|
||
| # Libraries | ||
| ## Docs | ||
|
|
||
| Mockups for each library are in the `ARC` folder. They contain API/Domain stubs to draft and test the new library designs. | ||
| Currently we provide some documentation in form of markdown files in the `/docs` folder of this repository! | ||
|
|
||
| ## Design choices | ||
| [Check it out!](/docs) | ||
|
|
||
| ### Fable compatibility as top priority | ||
|
|
||
| All libraries should be Fable compatible, and produce javascript/typescript code that is ergonomic to use in a js/ts environment, therefore: | ||
| - we use classes with static members over nested modules | ||
| - we use the `[<AttachMembers>]` fable attribute for each class | ||
| - Using overloads with the `[<AttachMembers>]` attribute will make js functions shadow itself. Never use this! | ||
| - we use the `[<NamedParams(n)>]` fable attribute for all optional parameters in static methods that use tupled, named params. | ||
| - we use `Array<'T>` for all collections in F#, since they get transpiled to native js arrays. | ||
| - we use the `[<Erase>]` fable attribute for union cases that contain data | ||
| ```fsharp | ||
| [<Erase>] type X = | Y of string | Z of int | ||
| ``` | ||
| - we use the `[<StringEnum>]` for unions that contain no data (e.g.) | ||
| ```fsharp | ||
| [<StringEnum>] type YesOrNo = | Yes | No | ||
| ``` | ||
| **Example:** | ||
| ```fsharp | ||
| [<AttachMembers>] | ||
| type Study = | ||
| { | ||
| Identifier : string option | ||
| Assays : Assay array option | ||
| } | ||
| [<NamedParams>] | ||
| static member create (?Identifier, ?Assays : Assay array) = | ||
| { | ||
| Identifier = Identifier | ||
| Assays = Assays | ||
| } | ||
| ``` | ||
|
|
||
| will become the following javascript code: | ||
|
|
||
| ```javascript | ||
| export class Study extends Record { | ||
| constructor(Identifier, Assays) { | ||
| super(); | ||
| this.Identifier = Identifier; | ||
| this.Assays = Assays; | ||
| } | ||
| static create({ Identifier, Assays }) { | ||
| return new Study(Identifier, Assays); | ||
| } | ||
| ``` | ||
| and the following typescript code: | ||
| ## Development | ||
|
|
||
| ```typescript | ||
| export class Study extends Record implements IEquatable<Study>, IComparable<Study> { | ||
| readonly Identifier: Option<string>; | ||
| readonly Assays: Option<Assay[]>; | ||
| constructor(Identifier: Option<string>, Assays: Option<Assay[]>) { | ||
| super(); | ||
| this.Identifier = Identifier; | ||
| this.Assays = Assays; | ||
| } | ||
| static create({ Identifier, Assays }: {Identifier?: string, Assays?: Assay[] }): Study { | ||
| return new Study(Identifier, Assays); | ||
| } | ||
| ``` | ||
| ### Requirements | ||
|
|
||
| ## Development | ||
| - [nodejs and npm](https://nodejs.org/en/download) | ||
| - verify with `node --version` (Tested with v18.16.1) | ||
| - verify with `npm --version` (Tested with v9.2.0) | ||
| - [.NET SDK](https://dotnet.microsoft.com/en-us/download) | ||
| - verify with `dotnet --version` (Tested with 7.0.306) | ||
|
|
||
| ### Setup | ||
| ### Local Setup | ||
|
|
||
| 1. `dotnet tool restore` | ||
| 2. `dotnet paket install` | ||
| 3. `npm install` | ||
|
|
||
| Verify correct setup with `./build.cmd runtests` ✨ | ||
| Verify correct setup with `./build.cmd runtests` ✨ |
Oops, something went wrong.