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

int: Pin sqlalchemy to 1.x.x versions #3496

Merged
merged 1 commit into from
Aug 3, 2023
Merged

int: Pin sqlalchemy to 1.x.x versions #3496

merged 1 commit into from
Aug 3, 2023

Conversation

ksbrar
Copy link
Collaborator

@ksbrar ksbrar commented Aug 3, 2023

Was experimenting with my ludwig installation, and I came across this error while trying to run a basic test:

Created a temporary directory at /tmp/tmphl05r644
Writing /tmp/tmphl05r644/_remote_module_non_scriptable.py
[2023-08-02 23:23:56,329] [INFO] [real_accelerator.py:133:get_accelerator] Setting ds_accelerator to cuda (auto detect)
ImportError while loading conftest '/home/ksbrar/work/ludwig-2/tests/conftest.py'.
tests/conftest.py:35: in <module>
    from ludwig.hyperopt.run import hyperopt
ludwig/hyperopt/run.py:11: in <module>
    from ludwig.api import LudwigModel
ludwig/api.py:40: in <module>
    from ludwig.backend import Backend, initialize_backend, provision_preprocessing_workers
ludwig/backend/__init__.py:22: in <module>
    from ludwig.backend.base import Backend, LocalBackend
ludwig/backend/base.py:32: in <module>
    from ludwig.data.cache.manager import CacheManager
ludwig/data/cache/manager.py:8: in <module>
    from ludwig.data.dataset.base import DatasetManager
ludwig/data/dataset/base.py:21: in <module>
    from ludwig.distributed import DistributedStrategy
ludwig/distributed/__init__.py:3: in <module>
    from ludwig.distributed.base import DistributedStrategy, LocalStrategy
ludwig/distributed/base.py:9: in <module>
    from ludwig.modules.optimization_modules import create_optimizer
ludwig/modules/optimization_modules.py:21: in <module>
    from ludwig.utils.torch_utils import LudwigModule
ludwig/utils/torch_utils.py:14: in <module>
    from ludwig.utils.strings_utils import SpecialSymbol
ludwig/utils/strings_utils.py:33: in <module>
    from ludwig.utils.tokenizers import get_tokenizer_from_registry
ludwig/utils/tokenizers.py:26: in <module>
    from ludwig.utils.hf_utils import load_pretrained_hf_tokenizer
ludwig/utils/hf_utils.py:7: in <module>
    from transformers import AutoConfig, AutoTokenizer, LlamaConfig, PreTrainedModel
../../.venvwrapper_virtualenvs/ludwig-2/lib/python3.8/site-packages/transformers/utils/import_utils.py:1089: in __getattr__
    module = self._get_module(self._class_to_module[name])
../../.venvwrapper_virtualenvs/ludwig-2/lib/python3.8/site-packages/transformers/utils/import_utils.py:1101: in _get_module
    raise RuntimeError(
E   RuntimeError: Failed to import transformers.modeling_utils because of the following error (look up to see its traceback):
E   Aim v3.17.5 does not support sqlalchemy v2.0.0. Please check the following issue for further updates: https://github.com/aimhubio/aim/issues/2514

It seems the aim package from aimstack only supports sqlalchemy 1.x.x, and there is an open issue in their repo to add support (aimhubio/aim#2514)

@github-actions
Copy link

github-actions bot commented Aug 3, 2023

Unit Test Results

  6 files  ±  0    6 suites  ±0   1h 6m 5s ⏱️ - 1h 30m 5s
34 tests ±  0  29 ✔️ +  1    5 💤 ±0  0  - 1 
88 runs  +10  72 ✔️ +10  16 💤 +1  0  - 1 

Results for commit c5bc92c. ± Comparison against base commit a2d3f6d.

@ksbrar ksbrar merged commit 1152133 into master Aug 3, 2023
16 checks passed
@ksbrar ksbrar deleted the ksbrar-patch-1 branch August 3, 2023 15:49
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

Successfully merging this pull request may close these issues.

None yet

3 participants