• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

terrapower / armi / 5978350895
91%
master: 87%

Build:
Build:
LAST BUILD BRANCH: main
DEFAULT BRANCH: master
Ran 25 Aug 2023 05:07PM UTC
Jobs 1
Files 273
Run time 9s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

25 Aug 2023 04:44PM UTC coverage: 88.063% (+0.03%) from 88.033%
5978350895

push

github

web-flow
Refactor armi.reactor.grids to wider submodule; Grids as ABC (#1373)

* Move grids.py to own module armi.reactor.grids

Goal is to introduce separate files for each grid type, but still
make those importable from armi.reactor.grids

* Add armi.reactor.grids.constants

Intended to share some constants and type hints across the
`armi.reactor.grids` submodule. Intentionally sparse.

Items will be imported into `armi.reactor.grids` to help with
back compatibility.

* Add armi.reactor.grids.locations

Provides abstract `LocationBase` as the interface defining base class.
Concrete subclasses `IndexLocation`, `MultiIndexLocation`, and
`CoordinateLocation` are also defined.

There is little to no implementation differences in this change. Only
that `LocationBase` is a proper abstract base class, so subclasses that
don't override the necessary abstract methods will need to change. Some
other key changes

- `LocationBase.__eq__` returns `NotImplemented` if the RHS argument
  is not a tuple nor a `LocationBase`

The four main classes defined are imported into `armi.reactor.grids`
to maintain back compatibility.

* Move reactor/tests/test_grids to grids/tests

* Add armi.reactor.grids.grid for Grid base class

Imported into `armi.reactor.grids` so still back compatible

* Add armi.reactor.grids.cartesian for CartesianGrid

* Add armi.reactor.grids.hexagonal

* Add armi.reactor.grids.thetarz

* Remove six.cPickle -> pickle in test_grids.py

* Add armi.reactor.grids.axial with AxialGrid class

Replaces grids.axialUnitGrid

* Move duplicated grids.addingIsValid to grids.locations

* Refactor Grid to be ABC, intermediate StructuredGrid

Heavier lift, but the classes defined in `armi.reactor.grids`
follow this inheritance pattern

- `Grid(abc.ABC)` :: abstract interface for _a_ grid, or things that reside in
  space
  - `StructuredGrid(Grid)` :: still abstract, but for things that have
    some structure, known... (continued)

752 of 752 new or added lines in 9 files covered. (100.0%)

26854 of 30494 relevant lines covered (88.06%)

0.88 hits per line

Jobs
ID Job ID Ran Files Coverage
1 5978350895.1 25 Aug 2023 05:07PM UTC 0
88.06
GitHub Action Run
Source Files on build 5978350895
Detailed source file information is not available for this build.
  • Back to Repo
  • 5e76197d on github
  • Prev Build on main (#5967976592)
  • Next Build on main (#5994904557)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc