Skip to content

Commit

Permalink
Update dependencies. Fix clippy lints.
Browse files Browse the repository at this point in the history
The ERFA C library is no longer needed, so all mentions of it are removed.

rubbl_casatables now uses thiserror instead of failure, meaning we no longer
need to unwrap its errors. Also update how DUT1 info can be accessed from a MS.

hifitime methods replaced a heap of "as_" and "in_" methods with "to_" (my
fault!). Also use its new Duration::from_seconds (and similar) methods.

atty is no longer considered sound, so use is_terminal instead. (Unfortunately
atty is still being used by other dependencies, but this change is still worth
doing.)

Stop importing std::ops::{Deref, DerefMut} for things that can be dereferenced
directly (e.g. `beam.deref()` is now `&*beam`, when `beam` is a `Box<dyn
Beam>`).
  • Loading branch information
cjordan committed Sep 23, 2023
1 parent 4021a47 commit 048a76c
Show file tree
Hide file tree
Showing 73 changed files with 1,595 additions and 1,383 deletions.
1 change: 0 additions & 1 deletion .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ jobs:
git \
lcov \
libcfitsio-dev \
liberfa-dev \
libhdf5-dev \
libfreetype-dev \
libexpat1-dev \
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/releases.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,19 +55,19 @@ jobs:
curl https://heasarc.gsfc.nasa.gov/FTP/software/fitsio/c/License.txt -o LICENSE-cfitsio
cp .github/workflows/releases-readme.md README.md
cargo build --profile production --locked --no-default-features --features=hdf5-static,erfa-static,cfitsio-static
cargo build --profile production --locked --no-default-features --features=hdf5-static,cfitsio-static
mv target/production/hyperdrive .
tar -acvf mwa_hyperdrive-$(git describe --tags)-Linux-x86-64-v3.tar.gz \
LICENSE COPYING-hdf5 LICENSE-erfa LICENSE-cfitsio README.md \
hyperdrive
cargo build --profile production --locked --no-default-features --features=hdf5-static,erfa-static,cfitsio-static,cuda
cargo build --profile production --locked --no-default-features --features=hdf5-static,cfitsio-static,cuda
mv target/production/hyperdrive .
tar -acvf mwa_hyperdrive-$(git describe --tags)-Linux-x86-64-v3-CUDA-double.tar.gz \
LICENSE COPYING-hdf5 LICENSE-erfa LICENSE-cfitsio LICENSE-NVIDIA README.md \
hyperdrive
cargo build --profile production --locked --no-default-features --features=hdf5-static,erfa-static,cfitsio-static,cuda-single
cargo build --profile production --locked --no-default-features --features=hdf5-static,cfitsio-static,cuda-single
mv target/production/hyperdrive .
tar -acvf mwa_hyperdrive-$(git describe --tags)-Linux-x86-64-v3-CUDA-single.tar.gz \
LICENSE COPYING-hdf5 LICENSE-erfa LICENSE-cfitsio LICENSE-NVIDIA README.md \
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/run-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ jobs:
git \
lcov \
libcfitsio-dev \
liberfa-dev \
libhdf5-dev \
libfreetype-dev \
libexpat1-dev \
Expand Down
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ Versioning](https://semver.org/spec/v2.0.0.html).
- More CUDA benchmarks for the modelling code.

### Fixed
- When raw MWA data is missing gpubox files in what is otherwise a contiguous
spectral window, it is no longer treated as a "picket fence" situation.
- Flux densities were not being correctly estimated for curved-power-law
components with GPU code.

Expand Down
Loading

0 comments on commit 048a76c

Please sign in to comment.