-
Notifications
You must be signed in to change notification settings - Fork 282
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
Replace pyke package with other solutions for netcdf loading #3415
Comments
Detail : state of play on possible logic programming packages.
Best of others seem to be logpy(or 'kanren' ?) and pyswip
|
@pp-mo Guilty as charged. I've been meaning to purge I ended up taking the pure Python route, but still building on the It would be very healthy of us to purge ourselves of the Nudge @trexfeathers 😉 |
That said, I'd even float the concept of making Indeed I'd even go as far as saying that we should seriously consider how we trim the fat of |
Couldn't help myself, see #3416 |
At the very least, we do need to be very sure of any performance impact. |
That's surely a different issue though ?? On the subject, I'm not convinced that a standalone 'cf' will be useful without the rules too. |
PyKE takes a HyKE... go for it @pp-mo 👍 |
Is this closed by #4198? |
Yes. |
I think Pyke is near-obsolete, at best static with no development since 2010 as fas as I can tell
This is a bit worrying, to say the least.
We could replace it with another Python logic programming solution, but they all seem in a similar state IMHO (check it out).
Instead, I reckon that the actual rules operation is really quite simple and not hard to rewrite in ordinary Python. Which would certainly make the rules easier to maintain.
BTW, I don't think we are really making ideal usage of the logic programming concept in the loader anyway : A rules-type language would be a really good solution to assigning netcdf variables to specific CF categories. This is currently handled by the
iris.fileformats.cf
wher theCfReader
using theidentify
methods in the various subclasses ofCFVariable
.Either that is a missed opportunity or, possibly, it just shows that pure Python solutions are actually perfectly adequate !
The text was updated successfully, but these errors were encountered: