Skip to content

Commit

Permalink
Merge pull request #268 from dtsuzuku-ibm/add-build-language
Browse files Browse the repository at this point in the history
add build-language job to build-images workflow
  • Loading branch information
daw3rd authored Jun 13, 2024
2 parents 04d3e1a + eff9148 commit 19dccd2
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 2 deletions.
10 changes: 10 additions & 0 deletions .github/workflows/build-images.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,16 @@ jobs:
run: |
make -C data-processing-lib DOCKER=docker image
make -C transforms/code DOCKER=docker test-image
build-language:
runs-on: ubuntu-latest
timeout-minutes: 30
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Build and Test Language Transforms
run: |
make -C data-processing-lib DOCKER=docker image
make -C transforms/language DOCKER=docker test-image
build-universal:
runs-on: ubuntu-latest
timeout-minutes: 30
Expand Down
2 changes: 1 addition & 1 deletion transforms/language/lang_id/python/src/lang_id_local.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,6 @@
table = data_access.get_table(os.path.join(input_folder, "test_01.parquet"))
print(f"input table: {table}")
# Transform the table
table_list, metadata = transform.transform(table)
table_list, metadata = transform.transform(table[0])
print(f"\noutput table: {table_list}")
print(f"output metadata : {metadata}")
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ def __init__(self, config: dict[str, Any]):
)
self.column_name = config.get(content_column_name_key)

def transform(self, table: pa.Table) -> tuple[list[pa.Table], dict[str, Any]]:
def transform(self, table: pa.Table, file_name: str = None) -> tuple[list[pa.Table], dict[str, Any]]:
"""
Put Transform-specific to convert one Table to 0 or more tables. It also returns
a dictionary of execution statistics - arbitrary dictionary
Expand Down

0 comments on commit 19dccd2

Please sign in to comment.