yoeo / guesslang

Detect the programming language of a source code
https://guesslang.readthedocs.io
MIT License
773 stars 110 forks source link

update to tensorflow 2.8.0 #61

Open nschloe opened 2 years ago

nschloe commented 2 years ago

Your install_requires is taken from requirements.txt where the version number is fixed. This is usually not a good idea because you prevent Python users from updating their installation when guesslang is installed. (Remember Python doesn't install packages locally, per package, but globally.)

A better solution is to provide a version range, usually between major versions, e.g.,

tensorflow >=2, <3

There are quite a few other things that could be improved in an around metadata handling in guesslang.

jimmydevine commented 1 year ago

I just cloned this repo, removed the 2.5.0 requirement from the tensorflow requirement and guesslang installed and seems to operate fine with tensorflow 2.10.0 installed.

nsthorat commented 6 months ago

+1, this is causing issues with typing-extensions conflicting. Would be awesome to just update this and publish a new package!

garrettmaring commented 4 months ago
# from tensorflow.estimator import ModeKeys, Estimator
# from tensorflow.python.training.tracking.tracking import AutoTrackable
from tensorflow.python.trackable.autotrackable import AutoTrackable
from tensorflow_estimator.python.estimator.estimator import Estimator, ModeKeys

after upgrading