Skip to content

Commit

Permalink
Added cutout tests
Browse files Browse the repository at this point in the history
  • Loading branch information
nabobalis committed Jul 8, 2024
1 parent 66fadd4 commit 6667eff
Show file tree
Hide file tree
Showing 12 changed files with 3,105 additions and 924 deletions.
5 changes: 3 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ repos:
]
exclude: ".*(.fits|.fts|.fit|.txt|tca.*|extern.*|.rst|.md|docs/conf.py)$"
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: "v0.5.0"
rev: "v0.5.1"
hooks:
- id: ruff
args: ["--fix", "--unsafe-fixes"]
Expand All @@ -29,11 +29,12 @@ repos:
- id: check-case-conflict
- id: trailing-whitespace
- id: mixed-line-ending
args: ["--fix=lf"]
- id: end-of-file-fixer
- id: check-yaml
- id: debug-statements
- repo: https://github.com/crate-ci/typos
rev: v1.22.9
rev: v1.23.1
hooks:
- id: typos
- repo: https://github.com/pre-commit/mirrors-mypy
Expand Down
2 changes: 2 additions & 0 deletions changelog/215.bugfix.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Added tests with JSOC cutouts and found issues with the data type of the files.
Now `sunkit_image.enhance.mgn` will convert the data to float 32 internally for its operations.
14 changes: 7 additions & 7 deletions ruff.toml
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,11 @@ lint.select = [
"A",
"ARG",
"ASYNC",
"ASYNC1",
"B",
"BLE",
"C4",
# "C90",
"COM",
# "D",
"DTZ",
"E",
"EM",
Expand All @@ -27,20 +26,17 @@ lint.select = [
"F",
"FBT",
"FLY",
# "FURB",
"G",
"I",
"ICN",
"INP",
"INT",
"ISC",
"LOG",
# "N",
"NPY",
"PERF",
"PGH",
"PIE",
# "PL",
"PLE",
"PT",
"PTH",
Expand All @@ -49,19 +45,23 @@ lint.select = [
"RET",
"RSE",
"RUF",
# "S",
"SIM",
"SLF",
"SLOT",
"T10",
"T20",
"TCH",
"TID",
"TRIO",
"TRY",
"UP",
"W",
"YTT",
# "C90",
# "D",
# "FURB",
# "N",
# "PL",
# "S",
]
lint.extend-ignore = [
"E501", # Line too long
Expand Down
Loading

0 comments on commit 6667eff

Please sign in to comment.