Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Serving/batch serving feature #1158

Merged
merged 145 commits into from
Nov 23, 2023
Merged

Serving/batch serving feature #1158

merged 145 commits into from
Nov 23, 2023

Conversation

RiddhiBagadiaa
Copy link
Contributor

Description

Implement batch feature serving
Add backend unit tests in offline_test.go
Add behave tests for batch feature serving in Snowflake

Type of change

Does this correspond to an open issue?

Select type(s) of change

  • Bug fix (non-breaking change which fixes an issue)
  • [-] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist:

  • [-] I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • [-] I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have fixed any merge conflicts

RiddhiBagadiaa and others added 30 commits September 25, 2023 15:33
@RiddhiBagadiaa
add batch serving framework to offline.go and implement batch serving…
ad6d6dd 
… in sql
@simba-git @RiddhiBagadiaa
Docs rewrite (#984)
22dcb6d
@simba-git @RiddhiBagadiaa
Update gitbooks yaml (#985)
1b7e131
@simba-git @RiddhiBagadiaa
Update some of the docs titles (#986)
3fc56bc
@epps @sdreyer
@ahmadnazeri @aolfat @RiddhiBagadiaa
Filepath Refactor (#978)
3f0892a 
Co-authored-by: sdreyer <sterling@featureform.com>
Co-authored-by: ahmadnazeri <42309296+ahmadnazeri@users.noreply.github.com>
Co-authored-by: Ali Olfat <ali@featureform.com>
@ahmadnazeri @RiddhiBagadiaa
Bugfix: adding python versions to docker (#990)
392123f
@anthonylasso @RiddhiBagadiaa
Format python transformation code in entity view (#994)
a2b7226
@anthonylasso @RiddhiBagadiaa
Link Update (#998)
0b215d1
@ahmadnazeri @RiddhiBagadiaa
Bugfix: postgres backend default disable ssl (#999)
8849387
@anthonylasso @RiddhiBagadiaa
BugFix: Link to correct source variant (#1000)
066c0d7
@sdreyer @RiddhiBagadiaa
Refined the Get docs page (#1001)
4ea827e
@ahmadnazeri @joshcolts18 @RiddhiBagadiaa
[Mintlify Docs] Add Initial Content and Configuration w/ resolved con…
b9e4645 
…flicts (#1006)

Co-authored-by: joshcolts18 <123998500+joshcolts18@users.noreply.github.com>
@sdreyer @RiddhiBagadiaa
Logging Updates (#995)
0c0b56c
@sdreyer @RiddhiBagadiaa
Fixed Quickstart Link in Docs (#1008)
507c030
@ahmadnazeri @RiddhiBagadiaa
Bugfix: Returning meaningful error message when Dill doesn't work (#993)
bcf12ee
@sdreyer @RiddhiBagadiaa
Behave Tests for Credentials (#996)
d567d03
@sdreyer @RiddhiBagadiaa
Clears Pinecone indexes after tests (#1009)
d4aa737
@ahmadnazeri @aolfat @RiddhiBagadiaa
Bugfix: removing bucket_path and using bucket_name + validating file …
d7694f1 
…prefix (#991)

Co-authored-by: Ali Olfat <ali@featureform.com>
@aolfat @RiddhiBagadiaa
Empty dataframe variant uses random (#1002)
551e0ad
@ahmadnazeri @RiddhiBagadiaa
Bugfix: using validate for register_file instead of validate_file_sch…
18a229f 
…eme (#1012)
@sdreyer @RiddhiBagadiaa
Additional Log Updates (#1013)
748c0b3
@aolfat @RiddhiBagadiaa
Bugfix: Add a validate for hosts (#1004)
8f0f931
@aolfat @RiddhiBagadiaa
Bugfix: Submap doesn't check for empty array value for etcd lookup (#997
a0838e6 
)
@anthonylasso @RiddhiBagadiaa
Display search result count (#1010)
b02d7b1
@anthonylasso @RiddhiBagadiaa
Handle missing default variant issue on dash (#1014)
122c185
@anthonylasso @RiddhiBagadiaa
Fix date conversion (#1007)
9bfd1e1
@sdreyer @RiddhiBagadiaa
Fixed typo in dashboard function (#1021)
bc05f95
@sdreyer @RiddhiBagadiaa
Changed localmode for batch storage to file (#979)
d699dd8
@ahmadnazeri @RiddhiBagadiaa
Bugfix: sorting values on training set for merge_feature_into_ts (#1025)
593d613
@aolfat @RiddhiBagadiaa
Bugfix: Ensure each transformation function argument has an input (#1018
5186748 
)
@RiddhiBagadiaa RiddhiBagadiaa merged commit 276cf1e into main Nov 23, 2023
42 checks passed
@RiddhiBagadiaa RiddhiBagadiaa deleted the serving/batch-serving-feature branch November 23, 2023 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DLT1412 DLT1412 DLT1412 left review comments

@sdreyer sdreyer sdreyer left review comments

@ahmadnazeri ahmadnazeri ahmadnazeri approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet