Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mechanical application of black. #220

Merged
merged 3 commits into from
Feb 13, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/code_style.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
- name: Analyze code with mypy
run: |
mypy ./src ./tests --ignore-missing-imports
mypy ./src --ignore-missing-imports
- name: Analyze code for black formatting
run: |
black --check .
Expand Down
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ dependencies = [
# On a mac, install optional dependencies with `pip install '.[dev]'` (include the single quotes)
[project.optional-dependencies]
dev = [
"black",
"mypy", # Used for static type checking of files
"pre-commit", # Used to run checks before finalizing a git commit
"pylint", # Used for static linting of files
Expand Down
1 change: 0 additions & 1 deletion src/hipscat_import/catalog/run_import.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
The actual logic of the map reduce is in the `map_reduce.py` file.
"""


import hipscat.io.write_metadata as io
import numpy as np
from hipscat import pixel_math
Expand Down
1 change: 1 addition & 0 deletions src/hipscat_import/margin_cache/__init__.py
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
"""All modules for generating margin caches."""

from .margin_cache import generate_margin_cache
1 change: 1 addition & 0 deletions src/hipscat_import/pipeline.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
"""Flow control and pipeline entry points."""

import smtplib
from email.message import EmailMessage

Expand Down
1 change: 0 additions & 1 deletion tests/hipscat_import/catalog/test_argument_validation.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
"""Tests of argument validation"""


import pytest

from hipscat_import.catalog.arguments import ImportArguments, check_healpix_order_range
Expand Down
1 change: 1 addition & 0 deletions tests/hipscat_import/catalog/test_file_readers.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
"""Test dataframe-generating file readers"""

import os

import hipscat.io.write_metadata as io
Expand Down
1 change: 0 additions & 1 deletion tests/hipscat_import/catalog/test_run_round_trip.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
regression the test case is exercising.
"""


import glob
import os

Expand Down
1 change: 0 additions & 1 deletion tests/hipscat_import/index/test_index_argument.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
"""Tests of argument validation"""


import pytest

from hipscat_import.index.arguments import IndexArguments
Expand Down
4 changes: 2 additions & 2 deletions tests/hipscat_import/index/test_index_map_reduce.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
"""Tests of map reduce operations"""


import os

import numpy as np
Expand Down Expand Up @@ -117,6 +116,7 @@ def test_create_index_source(
assert len(data_frame) == 17161
assert np.logical_and(data_frame["Norder"] >= 0, data_frame["Norder"] <= 2).all()


@pytest.mark.dask
def test_create_index_with_divisions(
small_sky_source_catalog,
Expand All @@ -126,7 +126,7 @@ def test_create_index_with_divisions(
"""Create an index catalog for the large(r) source catalog, passing
some divisions hints. This should partition the final output according to
the `compute_partition_size` and not the provided divisions."""
divisions = np.arange(start = 70_000, stop = 87_161, step = 5_000)
divisions = np.arange(start=70_000, stop=87_161, step=5_000)
divisions = np.append(divisions, 87_161).tolist()

args = IndexArguments(
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
"""Tests of margin cache generation arguments"""


import pytest

from hipscat_import.margin_cache.margin_cache_arguments import MarginCacheArguments
Expand Down
1 change: 1 addition & 0 deletions tests/hipscat_import/margin_cache/test_margin_cache.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
"""Tests of map reduce operations"""

import numpy as np
import numpy.testing as npt
import pandas as pd
Expand Down
1 change: 1 addition & 0 deletions tests/hipscat_import/soap/test_run_soap.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
"""Test full execution of SOAP."""

import pytest
from hipscat.catalog.association_catalog.association_catalog import AssociationCatalog

Expand Down
1 change: 0 additions & 1 deletion tests/hipscat_import/test_runtime_arguments.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
"""Tests of argument validation"""


import os

import pytest
Expand Down
Loading