-
Notifications
You must be signed in to change notification settings - Fork 262
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
internal: updating udf catalog from decorators #603
Merged
suryatejreddy
merged 47 commits into
georgia-tech-db:decorators_1
from
suryatejreddy:suryatej/extending_decorators
Mar 10, 2023
Merged
internal: updating udf catalog from decorators #603
suryatejreddy
merged 47 commits into
georgia-tech-db:decorators_1
from
suryatejreddy:suryatej/extending_decorators
Mar 10, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…nto suryatej/extending_decorators
…nto suryatej/extending_decorators
…nto suryatej/extending_decorators
…ejreddy/eva into suryatej/extending_decorators
…nto suryatej/extending_decorators
…into suryatej/extending_decorators
suryatejreddy
changed the title
internal: loading from decorators to catalog
internal: updating udf catalog from decorators
Mar 10, 2023
suryatejreddy
added a commit
that referenced
this pull request
Mar 12, 2023
gaurav274
added a commit
that referenced
this pull request
Mar 16, 2023
* created the udf service class. * created the decorators for udfs. * removed old decorators implementation files * new implementation file for decorators * new Eva Argumet type files. * new exception type for input mismatches. * renamed folder name * removed the test_udf_service file. * created new data types for eva * created the decorators for udfs * created a sample udf using the decorators * updates to decorators * changed the yolo decorators implementation * fixed linting errors. * fixed linting errors. * added the new pandas dataframe class * updates to decorators * updated the create udf parser format. * Added documentation. * removed the type parameter in create udf command. * added udf type to setup decorator. * added the tags to function signature. * updated the decorators to do dynamic conversion. moved the exception to the errors file. * added tests for udf decorators * added unit tests for udf decorators. made some fixes to code * removed the main fn for test * linting changes * fixed some errors in tests. linting changes. * updates to sql query in test * refactoring files and updated paths. * updated tests and data types classes. * linting changes. renamed the exception * removed debugging statements updated the file paths. * changed inputs to list of inputs * linting changes * fixed the decorators tests. linting changes. * internal: updating udf catalog from decorators (#603) * created a new class for arguments * updated the EvaArgument class for the PandasDataframe * created the udf service class. * created the decorators for udfs. * removed old decorators implementation files * new implementation file for decorators * new Eva Argumet type files. * new exception type for input mismatches. * renamed folder name * removed the test_udf_service file. * created new data types for eva * created the decorators for udfs * created a sample udf using the decorators * updates to decorators * changed the yolo decorators implementation * fixed linting errors. * fixed linting errors. * added the new pandas dataframe class * updates to decorators * updated the create udf parser format. * Added documentation. * removed the type parameter in create udf command. * added udf type to setup decorator. * added the tags to function signature. * updated the decorators to do dynamic conversion. moved the exception to the errors file. * added tests for udf decorators * added unit tests for udf decorators. made some fixes to code * removed the main fn for test * linting changes * fixed some errors in tests. linting changes. * updates to sql query in test * refactoring files and updated paths. * updated tests and data types classes. * linting changes. renamed the exception * removed debugging statements updated the file paths. * changed inputs to list of inputs * linting changes * fixed the decorators tests. linting changes. * internal: updating udf catalog from decorators (#603) * created a new class for arguments * updated the EvaArgument class for the PandasDataframe * address review commits one * lint * lint lint * refactor names * more refactor * refactor * lint * bug fix * lint * remove type checking code * remove arg name * adds integration test case * lint * adds test case for output as well * lint * adds test case to improve coverage * fixes test to improve coverage * remove error * adds modules * lint * updates docstrings * lint * review comments 1 * lint * lint lint * bug fix * bug fix * adds graceful handling for pandas datafram * lint * adds tests for decorators * lint * lint * adds test cases for descriptors * lint * changes to abstract * should increase coverage * lint * simplify * purge input_format * lint * updates asl url * update link in notebook * fix url in notebook * Update 05_asl_action_recognition.ipynb * reset file --------- Co-authored-by: Suryatej Reddy Vyalla <suryatej16102@iiitd.ac.in> Co-authored-by: Gaurav Tarlok Kakkar <gaurav21776@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR pulls is meant to be merged into #601
Description
Adds functionality for parsing decorators and updating the catalog.
Approach
Create
UdfIOCatalogEntry
objects by parsing decorators. The decorator input and outputs are built of a genericEVAArgument
class which has a similar structure toUdfIOCatalogEntry
.Testing
Adds testcases.