-
Notifications
You must be signed in to change notification settings - Fork 24
Update for PyPI release #18
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
Changes from 9 commits
0b322e4
8678477
52b1b73
cb6c125
cc5d2ea
bb7a036
cb97a6a
57c5201
fd10df2
d483fb1
66342fb
b8e5c38
94ab3e7
0197bb2
c439667
69b1d8e
71a9ca9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| build | ||
| twine |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,10 +6,10 @@ numpy<2 | |
| wandb | ||
| tensorly | ||
| tensorly-torch | ||
| xarray | ||
| xarray>=2023.2.0 | ||
|
||
| dacite | ||
| torch | ||
| torch-harmonics@git+https://github.com/NVIDIA/torch-harmonics.git@8826246cacf6c37b600cdd63fde210815ba238fd | ||
| torch-harmonics==0.6.2 | ||
| zarr | ||
| gcsfs | ||
| s3fs | ||
|
|
||
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,39 @@ | ||
| [build-system] | ||
| requires = ["setuptools", "wheel"] | ||
| build-backend = "setuptools.build_meta" | ||
|
|
||
| [project] | ||
| name = "fme" | ||
| version = "2024.10.5" | ||
|
||
| description = "Train and evaluate weather/climate model emulators" | ||
| readme = "README.md" | ||
| requires-python = ">=3.9" | ||
| license = {file = "LICENSE"} | ||
|
Contributor
Author
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. I had to move the @mcgibbon doesn't recall if there was a requirement that
Contributor
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. This does fundamentally change our repo so that it is now the repo of a single python package, as opposed to being a repo containing a single python package (but which could contain more). Probably worth discussing in our weekly fme sync.
Collaborator
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. I would lean to maintaining the current repo structure. Maybe we could duplicate the LICENSE within |
||
| authors = [ | ||
| {name = "Ai2 Climate Modeling", email = "[email protected]"} | ||
| ] | ||
| keywords = ["weather", "climate", "machine learning", "emulation"] | ||
| classifiers = [ | ||
| "Intended Audience :: Science/Research", | ||
| "Development Status :: 4 - Beta", | ||
| "License :: OSI Approved :: Apache Software License", | ||
| "Programming Language :: Python :: 3", | ||
| "Topic :: Scientific/Engineering :: Artificial Intelligence" | ||
| ] | ||
| dynamic = ["dependencies", "optional-dependencies"] | ||
| [project.urls] | ||
| Homepage = "https://github.com/ai2cm/ace" | ||
| Documentation = "https://ai2-climate-emulator.readthedocs.io/" | ||
|
|
||
| [tool.setuptools.dynamic] | ||
| dependencies = { file = "fme/requirements.txt" } | ||
| optional-dependencies.dev = { file = "fme/dev-requirements.txt" } | ||
| optional-dependencies.docs = { file = "fme/docs/requirements.txt" } | ||
| optional-dependencies.deploy = { file = "fme/deploy-requirements.txt" } | ||
|
|
||
| [tool.setuptools.packages.find] | ||
| where = ["fme"] | ||
| include = ["fme"] | ||
|
|
||
| [tool.setuptools.exclude-package-data] | ||
| fme = ["scripts", "fme/docs"] | ||
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.
User needs an API key to push, so it's safe to include in make target (i.e., can't accidentally push without access, and pushing same version fails)