Skip to content

Commit

Permalink
fix the vulnerability, update 'requirements.txt', 'setup.cfg', add th…
Browse files Browse the repository at this point in the history
…e warning message in the method 'generate_report' of the class Report
  • Loading branch information
Hanna Imshenetska authored and Hanna Imshenetska committed Nov 25, 2024
1 parent f440914 commit c44ca87
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 2 deletions.
1 change: 1 addition & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ scipy==1.14.*
seaborn==0.13.*
setuptools==74.1.*
tensorflow==2.15.*
tornado==6.4.*
tqdm==4.66.3
Werkzeug==3.1.2
xlrd
Expand Down
3 changes: 2 additions & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ packages = find:
include_package_data = True
python_requires = >3.9, <3.12
install_requires =
aiohttp>=3.9.0
aiohttp>=3.10.11
attrs
avro
base32-crockford
Expand Down Expand Up @@ -58,6 +58,7 @@ install_requires =
seaborn==0.13.*
setuptools==74.1.*
tensorflow==2.15.*
tornado==6.4.*
tqdm==4.66.3
Werkzeug==3.1.2
xlrd
Expand Down
2 changes: 1 addition & 1 deletion src/syngen/VERSION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
0.9.52rc32
0.9.52rc34
3 changes: 3 additions & 0 deletions src/syngen/ml/reporters/reporters.py
Original file line number Diff line number Diff line change
Expand Up @@ -246,6 +246,9 @@ def generate_report(cls):
"""
grouped_reporters = cls._group_reporters()

if grouped_reporters:
logger.warning("The report(s) generation might be time consuming")

for table_name, reporters in grouped_reporters.items():
cls._start_mlflow_run(table_name)
delta = 0.25 / len(reporters)
Expand Down

0 comments on commit c44ca87

Please sign in to comment.