Skip to content

Conversation

@BrunoSanchez
Copy link
Collaborator

This PR addresses several issues:

  • Add joblib for coadd
  • Create docs for coadd
  • Fix bugs in infer mask from data
  • Fix deprecated python version
  • Fix bugs on deprecated attrs API
  • Logging and minor updates

@BrunoSanchez
Copy link
Collaborator Author

This addresses issues #81, #79 and #77.

os: [ubuntu-latest]
tox_env: [py37, py38, py39, py310]
include:
- tox_env: py39
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

python 3.9 is already EOL, you can probably go up to 3.14 now.

pyproject.toml Outdated
[tool.black]
line-length = 79
target-version = ["py38"]
target-version = ["py39", "py310", "py311", "py312"]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here, 3.9 is end of life, and you can add 3.13 & 3.14

pyproject.toml Outdated
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
"Programming Language :: Python :: 3.9",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

drop 3.9

setup.py Outdated
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
"Programming Language :: Python :: 3.9",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you even need a setup.py? seems redundant if you already have a pyproject.toml, but in any case, you can drop 3.9 here too.

@leliel12
Copy link
Collaborator

it's merge ready for me

@BrunoSanchez BrunoSanchez merged commit 64de7ab into master Nov 24, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants