Skip to content

Commit

Permalink
test: disable reuse tests (#884)
Browse files Browse the repository at this point in the history
πŸ‘‹ Thanks for submitting a Pull Request to EvaDB!

πŸ™Œ We want to make contributing to EvaDB as easy and transparent as
possible. Here are a few tips to get you started:

- πŸ” Search existing EvaDB
[PRs](https://github.com/georgia-tech-db/eva/pulls) to see if a similar
PR already exists.
- πŸ”— Link this PR to a EvaDB
[issue](https://github.com/georgia-tech-db/eva/issues) to help us
understand what bug fix or feature is being implemented.
- πŸ“ˆ Provide before and after profiling results to help us quantify the
improvement your PR provides (if applicable).

πŸ‘‰ Please see our βœ… [Contributing
Guide](https://evadb.readthedocs.io/en/stable/source/contribute/index.html)
for more details.

---------

Co-authored-by: Gaurav <[email protected]>
  • Loading branch information
jarulraj and gaurav274 authored Jun 22, 2023
1 parent 3451a08 commit 654546d
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 11 deletions.
4 changes: 4 additions & 0 deletions docs/_static/custom.css
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,10 @@ a:visited {
color: var(--orange);
}

article table.align-default {
display: inherit;
}

/* --------- TOC sidebar --------- */

div.sidebar-tree .toctree-l1>.reference {
Expand Down
5 changes: 0 additions & 5 deletions evadb/executor/create_udf_executor.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,6 @@ def __init__(self, db: EvaDBDatabase, node: CreateUDFPlan):
Path(self.config.get_value("core", "evadb_installation_dir")) / "udfs"
)

# https://stackoverflow.com/a/76322515
import os

os.environ["PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION"] = "python"

def handle_huggingface_udf(self):
"""Handle HuggingFace UDFs
Expand Down
5 changes: 0 additions & 5 deletions evadb/udfs/ocr_extractor.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,6 @@
class OCRExtractor(AbstractUDF, GPUCompatible):
@setup(cacheable=False, udf_type="FeatureExtraction", batchable=False)
def setup(self):
# https://stackoverflow.com/a/76322515
import os

os.environ["PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION"] = "python"

try_to_import_torch()
try_to_import_torchvision()
try_to_import_transformers()
Expand Down
8 changes: 7 additions & 1 deletion test/integration_tests/test_reuse.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
import os
import unittest
from pathlib import Path
from test.markers import duplicate_skip_marker, windows_skip_marker
from test.markers import duplicate_skip_marker, linux_skip_marker, windows_skip_marker
from test.util import (
get_evadb_for_testing,
get_logical_query_plan,
Expand Down Expand Up @@ -113,6 +113,7 @@ def test_reuse_partial(self):
batches, exec_times = self._reuse_experiment([select_query1, select_query2])
self._verify_reuse_correctness(select_query2, batches[1])

@linux_skip_marker
def test_reuse_in_with_multiple_occurrences(self):
select_query1 = """SELECT id, label FROM DETRAC JOIN
LATERAL HFObjectDetector(data) AS Obj(score, label, bbox) WHERE id < 10;"""
Expand All @@ -137,6 +138,7 @@ def test_reuse_in_with_multiple_occurrences(self):
reuse_batch = execute_query_fetch_all(self.evadb, select_query)
self._verify_reuse_correctness(select_query, reuse_batch)

@linux_skip_marker
def test_reuse_logical_project_with_duplicate_query(self):
project_query = (
"""SELECT id, HFObjectDetector(data).label FROM DETRAC WHERE id < 10;"""
Expand All @@ -146,6 +148,7 @@ def test_reuse_logical_project_with_duplicate_query(self):
# reuse should be faster than no reuse
self.assertGreater(exec_times[0], exec_times[1])

@linux_skip_marker
def test_reuse_with_udf_in_predicate(self):
select_query = """SELECT id FROM DETRAC WHERE ['car'] <@ HFObjectDetector(data).label AND id < 4"""

Expand All @@ -154,6 +157,7 @@ def test_reuse_with_udf_in_predicate(self):
# reuse should be faster than no reuse
self.assertGreater(exec_times[0], exec_times[1])

@linux_skip_marker
def test_reuse_across_different_predicate_using_same_udf(self):
query1 = """SELECT id FROM DETRAC WHERE ['car'] <@ HFObjectDetector(data).label AND id < 15"""

Expand All @@ -164,6 +168,7 @@ def test_reuse_across_different_predicate_using_same_udf(self):
# reuse should be faster than no reuse
self.assertGreater(exec_times[0], exec_times[1])

@linux_skip_marker
def test_reuse_filter_with_project(self):
project_query = """
SELECT id, Yolo(data).labels FROM DETRAC WHERE id < 5;"""
Expand All @@ -175,6 +180,7 @@ def test_reuse_filter_with_project(self):
# reuse should be faster than no reuse
self.assertGreater(exec_times[0], exec_times[1])

@linux_skip_marker
def test_reuse_in_extract_object(self):
select_query = """
SELECT id, T.iids, T.bboxes, T.scores, T.labels
Expand Down
File renamed without changes.

0 comments on commit 654546d

Please sign in to comment.