Skip to content

Commit

Permalink
Apply SAL-side changes for TF 2.11.1 upgrade (#153)
Browse files Browse the repository at this point in the history
  • Loading branch information
misberner committed May 9, 2023
1 parent 997be9e commit bd49ce8
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
with:
python-version: 3.9
- name: Install TF
run: pip install tensorflow==2.8
run: pip install tensorflow==2.11
- name: Install gretel-synthetics
run: pip install -e .[all,test] --use-deprecated=legacy-resolver
- name: Test
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/localwheel.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
- name: Setup Python
uses: actions/setup-python@v1
with:
python-version: 3.7
python-version: 3.9
- name: Build wheel
run: |
pip install wheel
Expand Down
6 changes: 4 additions & 2 deletions .github/workflows/unit-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,15 +7,17 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
python: [3.7,3.8,3.9]
python:
# These have to be strings, as, e.g., 3.10 and 3.1 are NOT the same.
- "3.9"
steps:
- uses: actions/checkout@v1
- name: Setup Python
uses: actions/setup-python@v1
with:
python-version: ${{ matrix.python }}
- name: Install Dependencies
run: pip install -r requirements.txt
run: pip install -r requirements.txt --use-deprecated=legacy-resolver
- name: Install gretel-synthetics
run: pip install -e .[all,test] --use-deprecated=legacy-resolver
- name: Lint
Expand Down
8 changes: 4 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,16 +37,16 @@ This section will guide you through installation of `gretel-synthetics` and depe
By default, we do not install certain core requirements, the following dependencies should be installed _external to the installation_
of `gretel-synthetics`, depending on which model(s) you plan to use.

- Tensorflow: Used by the LSTM model, we recommend version 2.8.x
- Torch: Used by Timeseries DGAN and ACTGAN (for ACTGAN, Torch is installed by SDV)
- Tensorflow: Used by the LSTM model, we recommend version 2.11.x
- Torch: Used by Timeseries DGAN and ACTGAN (for ACTGAN, Torch is installed by SDV), we recommend version 2.0
- SDV (Synthetic Data Vault): Used by ACTGAN, we recommend version 0.17.x

These dependencies can be installed by doing the following:

```
pip install tensorflow==2.8 # for LSTM
pip install tensorflow==2.11 # for LSTM
pip install sdv<0.18 # for ACTGAN
pip install torch==1.13.1 # for Timeseries DGAN
pip install torch==2.0 # for Timeseries DGAN
```

To install the actual `gretel-synthetics` package, first clone the repo and then...
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ def reqs(file, without=None):
long_description_content_type="text/markdown",
package_dir={"": "src"},
packages=find_packages("src"),
python_requires=">=3.7",
python_requires=">=3.9",
install_requires=base_reqs,
extras_require={"all": utils_reqs, "utils": utils_reqs, "test": test_reqs},
classifiers=[
Expand Down

0 comments on commit bd49ce8

Please sign in to comment.