Skip to content

Commit 4647c17

Browse files
authored
Fix 404 links, reorganize API docs (#100)
* Fix docs deployment Also adds codebase doc that was not included in reorg. * Add tiles to C API index page * Add sentence about each file in C API index cards.
1 parent 7622756 commit 4647c17

File tree

6 files changed

+36
-26
lines changed

6 files changed

+36
-26
lines changed

.github/workflows/deploy_docs.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,9 @@ jobs:
8181
8282
- name: Replace relative links in Doxygen include
8383
run: |
84-
sed -i 's@acquire_zarr_c_api/acquire_zarr_c_api#@#@g' acquire-docs/site/api_reference/c_api/index.html
84+
# Remove include statements from output files
85+
sed -i '/<ul>/{N;N;/<ul>\n<li><code>#include \"zarr.types.h\"/d;}' acquire-docs/site/acquire_zarr_c_api/acquire_8zarr_8h/index.html
86+
sed -i '/<ul>/{N;N;N;N;/<ul>\n<li><code>#include/d;}' acquire-docs/site/acquire_zarr_c_api/zarr_8types_8h/index.html
8587
8688
- name: Deploy docs
8789
uses: peaceiris/actions-gh-pages@v3

docs/api_reference/Doxyfile

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
DOXYFILE_ENCODING = UTF-8
22
PROJECT_NAME = "acquire_zarr_c_api"
3-
OUTPUT_DIRECTORY = acquire_zarr_c_api
3+
OUTPUT_DIRECTORY = docs/api_reference/acquire_zarr_c_api
44
CREATE_SUBDIRS = NO
55
BRIEF_MEMBER_DESC = YES
66
REPEAT_BRIEF = YES
@@ -10,7 +10,7 @@ FULL_PATH_NAMES = YES
1010
SHORT_NAMES = NO
1111
PYTHON_DOCSTRING = YES
1212
INHERIT_DOCS = YES
13-
SEPARATE_MEMBER_PAGES = NO
13+
SEPARATE_MEMBER_PAGES = YES
1414
MARKDOWN_SUPPORT = YES
1515
TOC_INCLUDE_HEADINGS = 6
1616
MARKDOWN_ID_STYLE = DOXYGEN
@@ -41,7 +41,7 @@ SHOW_NAMESPACES = YES
4141
QUIET = NO
4242
WARNINGS = YES
4343
WARN_AS_ERROR = NO
44-
INPUT = "../../acquire-zarr/include"
44+
INPUT = "../acquire-zarr/include"
4545
FILE_PATTERNS = *.h
4646
RECURSIVE = NO
4747
EXCLUDE_SYMLINKS = NO
@@ -50,7 +50,7 @@ FILTER_SOURCE_FILES = NO
5050
IMPLICIT_DIR_DOCS = YES
5151
INLINE_SOURCES = NO
5252
STRIP_CODE_COMMENTS = YES
53-
GENERATE_HTML = YES
53+
GENERATE_HTML = NO
5454
HTML_OUTPUT = html
5555
HTML_FILE_EXTENSION = .html
5656
HTML_DYNAMIC_MENUS = YES

docs/api_reference/c_api.md

Lines changed: 0 additions & 19 deletions
This file was deleted.

docs/api_reference/c_api/index.md

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
# Zarr Streaming C API Reference
2+
3+
Information on the `acquire_zarr` C implementation and how to use it. Check
4+
https://github.com/acquire-project/acquire-zarr for more information.
5+
6+
<div class="cards">
7+
<div class="card">
8+
<h4><pre>acquire.zarr.h</pre></h4>
9+
<p>The primary header file that provides the main streaming functionality and settings configuration.</p>
10+
<a href="../../acquire_zarr_c_api/acquire_8zarr_8h" class="button">acquire.zarr.h</a>
11+
</div>
12+
<div class="card">
13+
<h4><pre>zarr.types.h</pre></h4>
14+
<p>Contains all type definitions and enumerations used by the library.</p>
15+
<a href="../../acquire_zarr_c_api/zarr_8types_8h" class="button">zarr.types.h</a>
16+
</div>
17+
</div>

docs/for_contributors/codebase.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# Contribute to codebase
2+
3+
To contribute to the latest development version, build the libraries from source
4+
[following these instructions](https://github.com/acquire-project/acquire-zarr/blob/main/README.md#building) to enable
5+
adding and testing changes locally before making a
6+
[PR to the `acquire-zarr` repo](https://github.com/acquire-project/acquire-zarr). We especially welcome contributions of
7+
bug fixes, new features, and bindings in languages relevant to microscopy such as MATLAB and Julia.

mkdocs.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,10 @@ nav:
4242
- API Reference:
4343
- api_reference/index.md
4444
- api_reference/zarr_api.md
45-
- api_reference/c_api.md
45+
- Zarr Streaming C API Reference:
46+
- api_reference/c_api/index.md
47+
- acquire_zarr_c_api/acquire_8zarr_8h.md
48+
- acquire_zarr_c_api/zarr_8types_8h.md
4649
# - Acquire Imaging:
4750
# - acquire-imaging/index.md
4851
# - Get Started: acquire-imaging/get_started.md
@@ -141,7 +144,7 @@ plugins:
141144
projects:
142145
acquire_zarr_c_api: # name of project must be alphanumeric + numbers (without spaces)
143146
src-dirs: ../acquire-zarr/include # path to source code (support multiple paths separated by space) => INPUT
144-
full-doc: True # if you want to generate full documentation
147+
full-doc: True # if you want to generate full documentation; i.e. root pages with all objects for a .h file
145148
doxy-cfg-file: docs/api_reference/Doxyfile
146149
save-api: docs/api_reference
147150

0 commit comments

Comments
 (0)