search

kjappelbaum / oximachinerunner

An easy API for using oximachine.
MIT License
7 stars 5 forks source link

make it Python 3.9 compatible #67

Open kjappelbaum opened 3 years ago

kjappelbaum commented 3 years ago

i do not remember what the issue was

kjappelbaum commented 3 years ago

i think the issue is a scikit-learn version pin

ltalirz commented 2 years ago

@xiaoqiepfl and @mpougin just stumbled over this. Any chance to relax this constraint?

kjappelbaum commented 2 years ago

not this week, but retraining is on my agenda (for various reasons) and then I'll bump the version.

kjappelbaum commented 2 years ago

is it urgent, i.e., can't they use an older Python atm?

mpougin commented 2 years ago

thanks for your fast reply @kjappelbaum. It's not urgent, we will downgrade python for now

kjappelbaum commented 2 years ago

Great, but please ping me mid November to actually do the retaining, also including data from the ICSD :D

On 25 Oct 2021, at 19:16, mpougin @.***> wrote:

thanks for your fast reply @kjappelbaum. It's not urgent, we will downgrade python for now

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.