Skip to content

Commit

Permalink
Merge pull request #25 from mughetto/main
Browse files Browse the repository at this point in the history
Updating local build reqs
  • Loading branch information
dgeleta authored Aug 16, 2023
2 parents 6247eb4 + b1a3d21 commit 1c1c1d0
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 9 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,11 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-python@v1
with:
python-version: 3.8.12
python-version: 3.9.17
- uses: s-weigand/setup-conda@v1
with:
activate-conda: true
python-version: 3.8
python-version: 3.9
- run: conda --version
- run: which python
- name: Run installation.
Expand Down
5 changes: 5 additions & 0 deletions docs/requirements_1.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,8 @@ markupsafe==2.0.1
nbconvert
sphinxcontrib-pseudocode
Jinja2==3.0.0
dataclasses-json
networkx
pandas-profiling
plotly
great_expectations
4 changes: 2 additions & 2 deletions onto_merger/data/data_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@ def load_specified_tables(self, table_names: List[str]) -> List[NamedTable]:
def load_analysis_report_table_as_dict(self,
section_name: str,
table_name: str,
rename_columns: dict = None) -> List[dict]:
rename_columns: Union[None, dict[str, str]] = None) -> List[dict]:
"""Load a CSV table as a list of dictionaries.
:param section_name: The name of the report section (prefix of the file).
Expand Down Expand Up @@ -254,7 +254,7 @@ def save_table(
logger.info(f"Saving table '{f'{table.name}.csv'}' with {len(table.dataframe):,d} " + f"row(s) to {file_path}.")
table.dataframe.to_csv(file_path, index=False)

def save_tables(self, tables: List[NamedTable], process_directory: str = None) -> None:
def save_tables(self, tables: List[NamedTable], process_directory: Union[None, str] = None) -> None:
"""Save a list of named tables Pandas dataframe part as CSVs.
:param tables: The tables to be saved.
Expand Down
4 changes: 1 addition & 3 deletions readthedocs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,8 @@ formats:
- htmlzip

python:
version: "3.8"
system_packages: "True"
version: "3.9"
install:
- requirements: docs/requirements_1.txt
- method: setuptools
path: .
system_packages: true
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
pandas==1.3.5
pandas-profiling==3.1.0
pandas-profiling==3.6.6
dataclasses-json==0.5.7
great_expectations==0.15.2
jinja2==3.0.3
Expand Down
3 changes: 2 additions & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ long_description_content_type = text/markdown
ignore =
E203,
W503,
BLK100
BLK100,
B023
exclude =
.tox,
.git,
Expand Down

0 comments on commit 1c1c1d0

Please sign in to comment.