Skip to content

Commit 6ea1c7b

Browse files
authored
docs: fix outdated links and various typos (#105)
This commits fixes several broken/outdated links and typos throughout the repository. ### Link Updates - Updated outdated links pointing to the original `BlueBrain` repository to their new location in the `highfive-devs` organization. - Fixed a broken link to the github pages migration guide in `CHANGELOG.md`. - Note: Links to specific historical items (e.g., discussions or issues) in the original repository are untouched to preserve context. ### Typo Fixes - Corrected a number of typos and grammatical errors in the documentation and github/CI config files.
1 parent 7c105ea commit 6ea1c7b

File tree

11 files changed

+42
-42
lines changed

11 files changed

+42
-42
lines changed

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ is much higher.
2222
- [optional] HDF5:
2323

2424
**Style Guide**
25-
1. Please paste text as text and not as a screen shot.
25+
1. Please paste text as text and not as a screenshot.
2626
2. If in doubt paste too much output rather than too little, i.e. don't be too
2727
scared of a large wall of text. Especially, if it's a compiler error.
2828
(Anything past the first error is largely uninformative and can be safely

.github/ISSUE_TEMPLATE/build_failure.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ assignees: ''
1111
Unfortunately, build failures tend to be highly specific and there's many
1212
things that could go wrong on both our and your side. Therefore, a reproducer
1313
is essential. You could try the minimal setup found here:
14-
https://github.com/BlueBrain/HighFive/blob/master/doc/installation.md#manually-install-highfive
14+
https://github.com/highfive-devs/HighFive/blob/main/doc/installation.md#manually-install-highfive
1515

1616
from there you can work upwards by adding complexity until you reproduce the
1717
issue.
@@ -35,7 +35,7 @@ than less output.)
3535
- HDF5:
3636

3737
**Style Guide**
38-
1. Please paste text as text and not as a screen shot.
38+
1. Please paste text as text and not as a screenshot.
3939
2. If in doubt paste too much output rather than too little, i.e. don't be too
4040
scared of a large wall of text. Especially, if it's a compiler error.
4141
(Anything past the first error is largely uninformative and can be safely

.github/ISSUE_TEMPLATE/config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
blank_issues_enabled: true
22
contact_links:
33
- name: HighFive Community Support
4-
url: https://github.com/BlueBrain/HighFive/discussions
4+
url: https://github.com/highfive-devs/HighFive/discussions
55
about: Please ask and answer questions here.

.github/workflows/ci.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ jobs:
8686
run: |
8787
ctest -j2 --output-on-failure -C $BUILD_TYPE
8888
89-
- name: Test No HDF5 Diagnositics
89+
- name: Test No HDF5 Diagnostics
9090
working-directory: ${{github.workspace}}/build
9191
run: |
9292
! ctest --verbose -C $BUILD_TYPE | grep HDF5-DIAG
@@ -133,7 +133,7 @@ jobs:
133133
run: |
134134
ctest -j2 --output-on-failure -C $BUILD_TYPE
135135
136-
- name: Test No HDF5 Diagnositics
136+
- name: Test No HDF5 Diagnostics
137137
working-directory: ${{github.workspace}}/build
138138
run: |
139139
! ctest --verbose -C $BUILD_TYPE | grep HDF5-DIAG
@@ -185,7 +185,7 @@ jobs:
185185
run: |
186186
ctest -j2 --output-on-failure -C $BUILD_TYPE
187187
188-
- name: Test No HDF5 Diagnositics
188+
- name: Test No HDF5 Diagnostics
189189
working-directory: ${{github.workspace}}/build
190190
run: |
191191
! ctest --verbose -C $BUILD_TYPE | grep HDF5-DIAG
@@ -232,7 +232,7 @@ jobs:
232232
run: |
233233
ctest -j2 --output-on-failure -C $BUILD_TYPE
234234
235-
- name: Test No HDF5 Diagnositics
235+
- name: Test No HDF5 Diagnostics
236236
working-directory: ${{github.workspace}}/build
237237
run: |
238238
! ctest --verbose -C $BUILD_TYPE | grep HDF5-DIAG
@@ -281,7 +281,7 @@ jobs:
281281
run: |
282282
ctest -j2 --output-on-failure -C $BUILD_TYPE
283283
284-
- name: Test No HDF5 Diagnositics
284+
- name: Test No HDF5 Diagnostics
285285
working-directory: ${{github.workspace}}/build
286286
run: |
287287
! ctest --verbose -C $BUILD_TYPE | grep HDF5-DIAG
@@ -372,7 +372,7 @@ jobs:
372372
run: |
373373
ctest -j2 --output-on-failure -C $BUILD_TYPE
374374
375-
- name: Test No HDF5 Diagnositics
375+
- name: Test No HDF5 Diagnostics
376376
working-directory: ${{github.workspace}}/build
377377
run: |
378378
! ctest --verbose -C $BUILD_TYPE | grep HDF5-DIAG
@@ -428,7 +428,7 @@ jobs:
428428
run: |
429429
ctest -j2 --output-on-failure -C $BUILD_TYPE
430430
431-
- name: Test No HDF5 Diagnositics
431+
- name: Test No HDF5 Diagnostics
432432
working-directory: ${{github.workspace}}/build
433433
shell: bash -l {0}
434434
run: |

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ Same as 3.0.0-beta3.
4949

5050
## Version 3.0.0-beta1 - 2024-07-16
5151
This version is a major one and is breaking some usage compare to v2.
52-
Read the migration guide from the documentation: https://highfive-devs.github.io/highfive/md__2home_2runner_2work_2_high_five_2_high_five_2doc_2migration__guide.html
52+
Read the migration guide from the documentation: https://highfive-devs.github.io/highfive/md__2home_2runner_2work_2highfive_2highfive_2doc_2migration__guide.html
5353

5454
The minimum version for C++ has been moved to `C++14`.
5555

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ option(HIGHFIVE_VERBOSE "Set logging level to verbose." OFF)
1919
# protected constructor, e.g., `HighFive::Object::Object`.
2020
#
2121
# Starting with `2.7.0` these friend declarations don't matter
22-
# anymore. It's mearly a means of appeasing a compiler.
22+
# anymore. It's merely a means of appeasing a compiler.
2323
#
2424
# The values of `HIGHFIVE_HAS_FRIEND_DECLARATIONS` are:
2525
# - that the macro is undefined.

README.md

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
> [!NOTE]
2-
> HighFive was orignally developed and maintained at
2+
> HighFive was originally developed and maintained at
33
> https://github.com/BlueBrain/HighFive. To continue maintenance of HighFive as
44
> an independent open-source code without support from BBP or EPFL, some (one)
55
> of the developers decided to create this repository.
@@ -37,7 +37,7 @@ target_link_libraries(foo HighFive::HighFive)
3737
- RAII for opening/closing files, groups, datasets, etc.
3838
- Compatible with the HDF5 C API.
3939
- A second API layer `H5Easy` provides a oneliner API for common, simple
40-
usecases.
40+
use cases.
4141

4242
### Feature support
4343
- create/read/write files, datasets, attributes, groups, dataspaces.
@@ -52,7 +52,7 @@ target_link_libraries(foo HighFive::HighFive)
5252
- etc... (see [ChangeLog](./CHANGELOG.md))
5353

5454
### Known flaws
55-
- HighFive is not thread-safe. At best it has the same limitations as the HDF5 library. However, HighFive objects modify their members without protecting these writes. Users have reported that HighFive is not thread-safe even when using the threadsafe HDF5 library, e.g., https://github.com/BlueBrain/HighFive/discussions/675.
55+
- HighFive is not thread-safe. At best, it has the same limitations as the HDF5 library. However, HighFive objects modify their members without protecting these writes. Users have reported that HighFive is not thread-safe even when using the threadsafe HDF5 library, e.g., https://github.com/BlueBrain/HighFive/discussions/675.
5656
- Eigen support in core HighFive was broken until v3.0. See https://github.com/BlueBrain/HighFive/issues/532. H5Easy was not
5757
affected.
5858
- The support of fixed length strings isn't ideal.
@@ -68,7 +68,7 @@ using namespace HighFive;
6868
std::string filename = "/tmp/new_file.h5";
6969

7070
{
71-
// We create an empty HDF55 file, by truncating an existing
71+
// We create an empty HDF5 file, by truncating an existing
7272
// file if required:
7373
File file(filename, File::Truncate);
7474

@@ -93,12 +93,12 @@ std::string filename = "/tmp/new_file.h5";
9393
**Note:** As of 2.8.0, one can use `highfive/highfive.hpp` to include
9494
everything HighFive. Prior to 2.8.0 one would include `highfive/H5File.hpp`.
9595

96-
**Note:** For advanced usecases the dataset can be created without immediately
96+
**Note:** For advanced use cases the dataset can be created without immediately
9797
writing to it. This is common in MPI-IO related patterns, or when growing a
9898
dataset over the course of a simulation.
9999

100100
### H5Easy
101-
For simple, common usecases the [highfive/H5Easy.hpp](include/highfive/H5Easy.hpp)
101+
For simple, common use cases the [highfive/H5Easy.hpp](include/highfive/H5Easy.hpp)
102102
interface provides single line solution. Here's the example from the
103103
introduction again:
104104
```cpp
@@ -123,8 +123,8 @@ See [easy_load_dump.cpp](src/examples/easy_load_dump.cpp) for more details.
123123
interchangeably.
124124

125125
### And Many More Examples!
126-
We strive to have one example per usecase or feature of HighFive,
127-
see [src/examples/](https://github.com/highfive-devs/highfive/blob/master/src/examples/)
126+
We strive to have one example per use case or feature of HighFive,
127+
see [src/examples/](https://github.com/highfive-devs/highfive/blob/main/src/examples/)
128128
for more examples.
129129

130130

@@ -155,15 +155,15 @@ target_link_libraries(foo HighFive::HighFive)
155155
```
156156

157157
**Note:** `add_subdirectory(third_party/HighFive)` will search and "link" HDF5
158-
but wont search or link any optional dependencies such as Boost.
158+
but won't search or link any optional dependencies such as Boost.
159159

160160
**Note:** The two targets `HighFive` and `HighFive::HighFive` are aliases. The
161161
former is older and works with v2, while the latter was introduced in v3,
162162
because CMake targets work more nicely if they contain `::`.
163163

164164
### Regular Installation of HighFive
165165

166-
Alternatively, HighFive can be install and "found" like regular software.
166+
Alternatively, HighFive can be installed and "found" like regular software.
167167
The project's `CMakeLists.txt` should add the following:
168168
```cmake
169169
find_package(HighFive REQUIRED)
@@ -217,7 +217,7 @@ HighFive integrates with the following libraries:
217217

218218
#### XTensor Header Location
219219
XTensor reorganized their headers in version 0.26. HighFive attempts to guess
220-
where the headers can be found. The guessing can be overridded by setting
220+
where the headers can be found. The guessing can be overridden by setting
221221
`HIGHFIVE_XTENSOR_HEADER_VERSION` to: `1` for finding `xtensor.hpp` in
222222
`<xtensor/xtensor.hpp>` and `2` for `<xtensor/containers/xtensor.hpp>`.
223223

@@ -269,7 +269,7 @@ Please consult its README for funding information by the Blue Brain Project or E
269269
One of the main contributors to
270270
[BlueBrain/HighFive](https://github.com/BlueBrain/HighFive) wanted to keep the
271271
project alive past the end of BBP. This repository was created to provide a
272-
seemless continuation of HighFive; and prevent fracturing or capturing of the
272+
seamless continuation of HighFive; and prevent fracturing or capturing of the
273273
project.
274274

275275
This repository is not supported by the Blue Brain Project or EPFL.

doc/Doxyfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -466,7 +466,7 @@ LOOKUP_CACHE_SIZE = 0
466466
# than 0 to get more control over the balance between CPU load and processing
467467
# speed. At this moment only the input processing can be done using multiple
468468
# threads. Since this is still an experimental feature the default is set to 1,
469-
# which efficively disables parallel processing. Please report any issues you
469+
# which effectively disables parallel processing. Please report any issues you
470470
# encounter. Generating dot graphs in parallel is controlled by the
471471
# DOT_NUM_THREADS setting.
472472
# Minimum value: 0, maximum value: 32, default value: 1.

doc/developer_guide.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Developer Guide
22
First clone the repository and remember the `--recursive`:
33
```bash
4-
git clone --recursive [email protected]:BlueBrain/HighFive.git
4+
git clone --recursive [email protected]:highfive-devs/HighFive.git
55
```
66
The instructions to recover if you forgot are:
77
```bash
@@ -149,7 +149,7 @@ arrays and mismatching sizes; and checking non-reallocation.
149149

150150
Read/Write cycles are implemented in two distinct checks. One for writing and
151151
another for reading. When checking writing we read with a "trusted"
152-
multi-dimensional array (a nested `std::vector`), and vice-versa when checking
152+
multi-dimensional array (a nested `std::vector`), and vice versa when checking
153153
reading. This matters because certain bugs, like writing a column major array
154154
as if it were row-major can't be caught if one reads it back into a
155155
column-major array.
@@ -227,7 +227,7 @@ brings their own copy of HighFive and uses it. The different vendoring
227227
strategies are:
228228

229229
- **find_package**: the standard way for finding dependencies in CMake. Usually
230-
the assumption is that HighFive was install properly, either systemwide or in
230+
the assumption is that HighFive was installed properly, either systemwide or in
231231
a specific subdirectory. HighFive is then found with `find_package` (or
232232
`find_dependency` when called from `*Config.cmake`).
233233

@@ -242,12 +242,12 @@ strategies are:
242242
works.
243243

244244
#### Integration Strategies
245-
These refer to downstream projects picking different HighFive targerts to
245+
These refer to downstream projects picking different HighFive targets to
246246
"link" with HighFive. There's four: two regular targets, a target that only
247247
adds `-I <dir>` and one that skips all HighFive CMake code.
248248

249249
#### Location Hints
250-
There are serveral ways of indicating where to find a package:
250+
There are several ways of indicating where to find a package:
251251

252252
- **CMAKE_PREFIX_PATH**: which adds a list of directories to the list of
253253
directories that are used as prefixes when searching for `HighFiveConfig.`
@@ -274,7 +274,7 @@ then their build breaks, complaining about missing HighFive targets (and it
274274
seems they can't "fix it up" on their end because then CMake complains that
275275
there's duplicate exported HighFive related targets).
276276

277-
The second way ommiting the missing `export()` can break downstream projects is
277+
The second way omitting the missing `export()` can break downstream projects is
278278
if they attempt to use HighFive's build directory (not install directory) as
279279
`HighFive_ROOT` (or `CMAKE_PREFIX_PATH`).
280280

@@ -297,7 +297,7 @@ symbols will exist, and they linker will pick one (arbitrarily); which is only
297297
safe if all definitions are identical.
298298

299299
### Test "Organization"
300-
The script to check everything is unwieldy. Here's a summary of what it attemps
300+
The script to check everything is unwieldy. Here's a summary of what it attempts
301301
to do.
302302

303303
There's three downstream project in play: `dependent_library` is a library that
@@ -314,7 +314,7 @@ The conceptually easy choices are:
314314

315315
- Libraries and applications that have dependencies that use HighFive should
316316
use `find_package` since it's the easiest way of injecting a common version
317-
of HighFive everwhere.
317+
of HighFive everywhere.
318318

319319
Since we can't (and don't want to) force our consumers to use `find_package`
320320
and ban vendoring, we have to test what happens when libraries vendor

doc/installation.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ Now, fingers crossed it'll compile and install:
124124

125125
Note that here we picked the installation path (or more precisely prefix) to be
126126
`../hdf5-v1.14.0`. You might want to install HDF5 somewhere else. This
127-
installation prefix is the also the path you need to give CMake so it's able to
127+
installation prefix is also the path you need to give CMake so it's able to
128128
find HDF5 later on.
129129

130130
### Confirming HDF5 Has Been Installed
@@ -200,7 +200,7 @@ enough.
200200
Just like before the steps are, clone, configure, compile (kinda a no-op),
201201
install. The detailed instructions would be
202202

203-
git clone --recursive https://github.com/BlueBrain/HighFive.git
203+
git clone --recursive https://github.com/highfive-devs/HighFive.git
204204
cd HighFive
205205
git checkout v2.8.0
206206

@@ -251,7 +251,7 @@ The required CMake commands are:
251251
[100%] Linking CXX executable dummy
252252
/usr/bin/c++ ... -o dummy -Wl,-rpath,${HDF5_ROOT}/lib ${HDF5_ROOT}/lib/libhdf5.so.310.0.0 ...
253253

254-
Pay attention to the semi-colon (not colon like the rest of Linux) used to
254+
Pay attention to the semicolon (not colon like the rest of Linux) used to
255255
separate directories in `CMAKE_PREFIX_PATH`. If this worked you should be set
256256
to either copy the instruction to your "real" project, or start developing the
257257
rest of your project.

0 commit comments

Comments
 (0)