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

Update packaging #85

Open
IAlibay opened this issue Feb 9, 2024 · 7 comments
Open

Update packaging #85

IAlibay opened this issue Feb 9, 2024 · 7 comments

Comments

@IAlibay
Copy link
Contributor

IAlibay commented Feb 9, 2024

Is your feature request related to a problem?

Follow up from previous issues - a few nice to haves to packaging especially as we start looking at supporting NAGL downstream.

Describe the solution you'd like

Couple of quick thoughts, there's probably more:

  • Upgrading versioneer to be py3.12 compatible (or making it clear that it isn't compatible, or switching to an alternate versioning alternative)
  • Switching to a pyproject.toml (just a nice to have)
  • Bumping up the minimum supported Python version? (cf indicates 3.9+ and setup.py indicates 3.8 if I remember correctly)
  • Support for pydantic 2.0? (downstream is planning to switch soon)
@mattwthompson mattwthompson mentioned this issue Feb 9, 2024
6 tasks
@mattwthompson
Copy link
Member

  • I started Update versioneer #86 which may or may not get things spinning on 3.12, who knows
  • I don't use pyproject.toml myself and seem to have generally missed the boat on where it must be used.
  • The last release is 3.9+, at least on the conda side of things. I can't release OpenFF tools on PyPI right now so there might be something different going on in that space.
  • Pydantic v1/v2 (runtime) compatibility should already be out in the world (Pydantic v1 and v2 compatibility #46 / version 0.3.3)

@mattwthompson
Copy link
Member

@IAlibay
Copy link
Contributor Author

IAlibay commented Feb 9, 2024

Thanks @mattwthompson !

I should clarify, I was mostly just reading setup.py here - I have absolutely no idea what is possible or not.

Re: pyproject.toml - I woud have been happy to help out if necessary but I'm out of office for a bit starting today so I probably won't have time to catch up, maybe @dwhswenson can lend a hand if necessary with POSE powers?

@dwhswenson
Copy link
Member

Yeah, in principle, bringing things up to current recommended practice is within POSE scope. Although TBH, I'm going to have to learn how to do the pyproject.toml transition as well, since I've always been a setup.cfg person, myself.

I assume this isn't an urgent task? A broader plan to transition OpenFF projects to pyproject.toml would be a good topic for the first OpenFF/POSE devs meeting, which should get scheduled for late Feb/early March.

@mattwthompson
Copy link
Member

mattwthompson commented Feb 9, 2024

I assume this isn't an urgent task?

This repo is Lily's domain but in general, for our infrastructure I don't see any urgency. (I never know how to properly use commas and in this sentence I can see four places a comma might go and maybe half of the use/don't use combinations look valid to my eye. I'm glad I went into science, where important writing is proofread by people who know better.)

When we get to it, I'd like to learn more about the tradeoffs - until I can ship our ecosystem on PyPI I'm unaware of the benefits

@lilyminium
Copy link
Collaborator

FWIW this repo started off with a mostly-functional pyproject.toml, so it should be in the history somewhere and can be revived! I removed it due to #28 and haven't revisited it yet, but can do if/when it becomes a priority.

@lilyminium lilyminium mentioned this issue Feb 12, 2024
4 tasks
@lilyminium
Copy link
Collaborator

There appears to be no DGL on 3.12?

There's no OpenEye for 3.12 currently either, but I've added tests for non-DGL, RDKit-only installations in #87.

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

No branches or pull requests

4 participants