Skip to content

Commit

Permalink
Reduce the number of ignored words for Codespell (#654)
Browse files Browse the repository at this point in the history
* Deal with "entrie"

* "filname" does not seem to appear in the current code.

* "parm" only appears in .dat files.

* "pres" only appears in .dat and .html files.

* "WAN" does not appear in the current code.

* "ans" only appears as "Ans" in library names.

* Rename "dout" to "d_out"

* Fix availale-> available in getting_started_with_dpf_server.rst

* Fix accross->across in 04-basic-load-file.py

* Add containter to ignored_words until fixed

* Adding periods to error messages modified.

* Fix typos in test_lsdyna.py

* One must use lowercase for ignore_words.txt

* Retrigger pipelines

* Remove continue-on-error for Codespell style check. It should now pass.
  • Loading branch information
PProfizi authored Dec 13, 2022
1 parent 4dd8b6a commit b6b0e15
Show file tree
Hide file tree
Showing 9 changed files with 291 additions and 287 deletions.
1 change: 0 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,6 @@ jobs:
- name: "Codespell"
run: |
make codespell
continue-on-error: true
- name: "flake8"
run: |
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/ci_release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ jobs:
- name: "Codespell"
run: |
make codespell
continue-on-error: true
- name: "flake8"
run: |
Expand Down
4 changes: 2 additions & 2 deletions docs/source/user_guide/getting_started_with_dpf_server.rst
Original file line number Diff line number Diff line change
Expand Up @@ -139,8 +139,8 @@ The following Ansys licensing increments currently provide rights to use DPF Ser

- "preppost" available in Ansys Mechanical Enterprise PrepPost product
- "meba" available in ANSYS Mechanical Enterprise Solver product
- "mech_2" availale in ANSYS Mechanical Premium product
- "mech_1" availale in ANSYS Mechanical Pro product
- "mech_2" available in ANSYS Mechanical Premium product
- "mech_1" available in ANSYS Mechanical Pro product
- "ansys" available in ANSYS Mechanical Enterprise product
- "dynapp" available in ANSYS LS-DYNA PrepPost product
- "vmotion" available in Ansys Motion product
Expand Down
2 changes: 1 addition & 1 deletion examples/04-file-IO/04-basic-load-file.py
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@
# ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# Note that the absolute error is bigger where the displacements are
# bigger, at the tip of the geometry.
# Instead, the relative error is similar accross the geometry since we
# Instead, the relative error is similar across the geometry since we
# are dividing by the displacements ``fc_out``.
# Both plots show errors that can be understood as zero due to machine precision
# (1e-12 mm for the absolute error and 1e-5% for the relative error).
Expand Down
7 changes: 1 addition & 6 deletions ignore_words.txt
Original file line number Diff line number Diff line change
@@ -1,9 +1,4 @@
parm
pres
WAN
filname
ans
entrie
dout
containter
mane
systyem
5 changes: 3 additions & 2 deletions src/ansys/dpf/core/collection.py
Original file line number Diff line number Diff line change
Expand Up @@ -327,9 +327,10 @@ def __getitem__(self, index):
index = self_len + index

if not self_len:
raise IndexError("This collection contains no items")
raise IndexError("This collection contains no items.")
if index >= self_len:
raise IndexError(f"This collection contains only {self_len} entrie(s)")
word = "entry" if self_len == 1 else "entries"
raise IndexError(f"This collection contains only {self_len} {word}.")

return self._get_entries(index)

Expand Down
Loading

0 comments on commit b6b0e15

Please sign in to comment.