-
Notifications
You must be signed in to change notification settings - Fork 8
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
update bioformats to 7.1.0 #68
Changes from all commits
456acaa
ee25678
13cf3e7
ac92c2a
85732bd
9a8263c
edab223
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
[flake8] | ||
docstring-convention=google | ||
max-line-length=88 | ||
exclude=noxfile.py,docs/*,tests/*,examples/*,.nox/*,build/*,bfio/OmeXml.py,setup.py | ||
exclude=noxfile.py,docs/*,tests/*,examples/*,.nox/*,build/* | ||
ignore=D105, W503, D100, D104, AZ100 | ||
extend-ignore = E203 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,4 +20,4 @@ jobs: | |
- uses: psf/black@stable | ||
with: | ||
options: "--check --color --verbose" | ||
src: "./bfio" | ||
src: "./src" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
__pycache__ | ||
.vscode | ||
.venv | ||
venv | ||
docs/build | ||
dist | ||
build | ||
|
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,123 @@ | ||
[build-system] | ||
# Run dependencies are also considered as build dependencies | ||
# due to cascading import situation in src/bfio/__init__.py | ||
requires = [ | ||
"setuptools", | ||
"wheel", | ||
"imagecodecs>=2021.2.26", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Upto you, but I would probably add a comment here saying that the runtime dependencies have been added as build-time dependencies, simply due to the fact that src/bfio/init.py exports some aliases, which causes cascading imports which effectively requires adding all runtime dependencies at build-time. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. will do that! |
||
"numpy", | ||
"ome-types>=0.4.2", | ||
"zarr>=2.6.1", | ||
"scyjava", | ||
"jpype1", | ||
"tifffile>=2022.8.12" | ||
] | ||
build-backend = "setuptools.build_meta" | ||
|
||
|
||
[project] | ||
name = "bfio" | ||
dynamic = ["version"] | ||
dependencies = [ | ||
"imagecodecs>=2021.2.26", | ||
"numpy", | ||
"ome-types>=0.4.2", | ||
"zarr>=2.6.1", | ||
"scyjava", | ||
"jpype1", | ||
"tifffile>=2022.8.12" | ||
] | ||
|
||
description = "Simple reading and writing classes for tiled tiffs using Bioformats." | ||
readme = "README.md" | ||
license = {file = "LICENSE"} | ||
|
||
authors = [ | ||
{name = "Nick Schaub", email = "nick.schaub@nih.gov"}, | ||
{name = "Sameeul Bashir Samee", email = "sameeul.samee@axleinfo.com"}, | ||
] | ||
|
||
maintainers = [ | ||
{name = "Sameeul Bashir Samee", email = "sameeul.samee@axleinfo.com"}, | ||
] | ||
|
||
classifiers = [ | ||
"Development Status :: 5 - Production/Stable", | ||
"Programming Language :: Python :: 3", | ||
"License :: OSI Approved :: MIT License", | ||
"Operating System :: OS Independent", | ||
] | ||
|
||
requires-python = ">=3.8" | ||
|
||
|
||
[project.optional-dependencies] | ||
dev = [ | ||
"ome_zarr", | ||
"requests>=2.26.0" | ||
] | ||
|
||
|
||
[project.urls] | ||
Documentation = "https://bfio.readthedocs.io/en/latest/" | ||
Repository = "https://github.com/PolusAI/bfio" | ||
|
||
|
||
[tool.setuptools.dynamic] | ||
version = {attr = "bfio.__version__"} | ||
|
||
|
||
[tool.setuptools] | ||
package-dir = {"" = "src"} | ||
include-package-data = true | ||
|
||
|
||
[tool.setuptools.packages.find] | ||
where = ["src"] | ||
namespaces = false | ||
|
||
|
||
[tool.setuptools.package-data] | ||
"*" = [ | ||
"VERSION" | ||
] | ||
|
||
|
||
[tool.bumpversion] | ||
current_version = "2.3.3" | ||
commit = true | ||
tag = true | ||
commit_args = "--no-verify" | ||
parse = "(?P<major>\\d+)\\.(?P<minor>\\d+)\\.(?P<patch>\\d+)(\\-(?P<release>[a-z]+)(?P<build>\\d+))?" | ||
serialize = [ | ||
"{major}.{minor}.{patch}-{release}{build}", | ||
"{major}.{minor}.{patch}" | ||
] | ||
|
||
[tool.bumpversion.parts.release] | ||
optional_value = "_" | ||
first_value = "dev" | ||
values = [ | ||
"dev", | ||
"_" | ||
] | ||
|
||
[[tool.bumpversion.files]] | ||
filename = "src/bfio/VERSION" | ||
search = "{current_version}" | ||
replace = "{new_version}" | ||
|
||
[[tool.bumpversion.files]] | ||
filename = "requirements/requirements-base.txt" | ||
search = "bfio[all]=={current_version}" | ||
replace = "bfio[all]=={new_version}" | ||
|
||
[[tool.bumpversion.files]] | ||
filename = "README.md" | ||
search = "{current_version}" | ||
replace = "{new_version}" | ||
|
||
|
||
[tool.codespell] | ||
skip = "*.svg,*.html" | ||
ignore-words-list = "utput" |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
# -*- coding: utf-8 -*- | ||
from __future__ import absolute_import, unicode_literals | ||
|
||
import logging | ||
import pathlib | ||
|
||
JAR_VERSION = None | ||
|
||
logging.basicConfig( | ||
format="%(asctime)s - %(name)-8s - %(levelname)-8s - %(message)s", | ||
datefmt="%d-%b-%y %H:%M:%S", | ||
) | ||
logger = logging.getLogger("bfio.init") | ||
|
||
log_level = logging.WARNING | ||
|
||
try: | ||
with open(pathlib.Path(__file__).parent.joinpath("VERSION"), "r") as fh: | ||
__version__ = fh.read() | ||
except FileNotFoundError: | ||
logger.info( | ||
"Could not find VERSION. " | ||
+ "This is likely due to using a local/cloned version of bfio." | ||
) | ||
__version__ = "0.0.0" | ||
logger.info("VERSION = {}".format(__version__)) | ||
|
||
|
||
from .bfio import BioReader, BioWriter, start # NOQA: F401, E402 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checking that you intended to remove the "." If this is correct, then LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
intentional change.