Beam analysis endpoint #98
Merged
Conversation
This file contains hidden or 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
added 2 commits
January 14, 2025 14:18
corke2013
force-pushed
the
beam-analysis-endpoint
branch
from
00b3ed1 to
9f986c0
Compare
corke2013
force-pushed
the
beam-analysis-endpoint
branch
from
7ccfc69 to
10f19b6
Compare
medhatphq
approved these changes
Jan 14, 2025
Contributor
medhatphq
left a comment
medhatphq
left a comment
There was a problem hiding this comment.
Looks great @corke2013
medhatphq
reviewed
Jan 14, 2025
* add analysis group endpoints * add analysis group import * python 3.8 compat * fix signature * use optional type hint --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com>
lpouvreau
approved these changes
Jan 15, 2025
* beam sink endpoint - allow json, csv and ndjson upload * expose beam endpoint in sdk * allow extra fields so that new fields are visible without an update to the SDK * remove undocumented endpoints * add features-api to_csv mixin * remove accidental print statement * fix typing for python 3.8 * update build url test * fix bug in features api endpoint when using iter_all * update feature api tests * update readme, add examples and missing param to beam update function * add beam endpoint tests (#102) * add beam endpoint tests * address comments - use correct fixtures --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com> * add comment regarding features-api * Pagination (#103) * add ArgKwargResult set for custom pagination * use custom pagination on all supported beam endpoints --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com> --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com>
corke2013
added a commit
that referenced
this pull request
Jan 20, 2025
* fix bug in to_json function * Beam analysis endpoint (#98) * add beam schema * add beam analysis endpoint * add BeamEndpoint import * update schema for python 3.8 * update schema for python 3.8 * update endpoints for python 3.8 * fix analysis endpoints and schema * add analysis group endpoints (#99) * add analysis group endpoints * add analysis group import * python 3.8 compat * fix signature * use optional type hint --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com> * Beam sink endpoint (#100) * beam sink endpoint - allow json, csv and ndjson upload * expose beam endpoint in sdk * allow extra fields so that new fields are visible without an update to the SDK * remove undocumented endpoints * add features-api to_csv mixin * remove accidental print statement * fix typing for python 3.8 * update build url test * fix bug in features api endpoint when using iter_all * update feature api tests * update readme, add examples and missing param to beam update function * add beam endpoint tests (#102) * add beam endpoint tests * address comments - use correct fixtures --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com> * add comment regarding features-api * Pagination (#103) * add ArgKwargResult set for custom pagination * use custom pagination on all supported beam endpoints --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com> --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com> --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.com> * fix bug in upload demand endpoint * use config dict on each model --------- Co-authored-by: Keegan Cordeiro <keegan.cordeiro@predicthq.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.
Add the Beam Analysis endpoint and relevant functions.