Skip to content

Branch for coadd + bugfixes + bump python versions - #82

Merged
BrunoSanchez merged 18 commits into
masterfrom
coaddJoblib
Nov 24, 2025
Merged

BrunoSanchez merged 18 commits into
masterfrom
coaddJoblib

Conversation

@BrunoSanchez

Copy link
Copy Markdown
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
Copy Markdown
Collaborator Author

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

Comment thread .github/workflows/testing.yml Outdated
os: [ubuntu-latest]
tox_env: [py37, py38, py39, py310]
include:
- tox_env: py39

Copy link
Copy Markdown
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.

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

Copy link
Copy Markdown
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

Comment thread pyproject.toml Outdated
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
"Programming Language :: Python :: 3.9",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

drop 3.9

Comment thread setup.py Outdated
"Programming Language :: Python :: 3",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
"Programming Language :: Python :: 3.9",

Copy link
Copy Markdown
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
Copy Markdown
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