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

logInputs endpoint #17

Merged
merged 10 commits into from
Oct 17, 2024
Merged

logInputs endpoint #17

merged 10 commits into from
Oct 17, 2024

Conversation

nojaf
Copy link
Collaborator

@nojaf nojaf commented Oct 10, 2024 •
edited
Loading

@nojaf nojaf linked an issue Oct 10, 2024 that may be closed by this pull request
Support LogInputs in mlflow-go #25
Closed
nojaf added 2 commits October 11, 2024 14:10
@nojaf
logInputs
e059f04
@nojaf
Add additional validation checks
e161a97 
Signed-off-by: nojaf <florian.verdonck@outlook.com>
nojaf added 3 commits October 14, 2024 14:52
@nojaf
Implement LogInputs
af00b5b 
Signed-off-by: nojaf <florian.verdonck@outlook.com>
@nojaf
Reuse receiver name
d09e73d 
Signed-off-by: nojaf <florian.verdonck@outlook.com>
@nojaf
Update ref to upstream fix
bb53032 
Signed-off-by: nojaf <florian.verdonck@outlook.com>
@nojaf nojaf mentioned this pull request Oct 14, 2024
Merged
39 tasks
@nojaf nojaf marked this pull request as ready for review October 14, 2024 15:58
nojaf added 2 commits October 16, 2024 06:36
@nojaf
Revert ref to master
501f0c8 
Signed-off-by: nojaf <florian.verdonck@outlook.com>
@nojaf
Merge branch 'main' into logInputs
8cd695e
@nojaf nojaf requested a review from dsuhinin October 16, 2024 06:51
dsuhinin
dsuhinin approved these changes Oct 16, 2024
View reviewed changes
pkg/tracking/store/sql/inputs.go Outdated Show resolved Hide resolved
pkg/tracking/store/sql/inputs.go Outdated Show resolved Hide resolved
pkg/tracking/store/sql/inputs.go Outdated Show resolved Hide resolved
nojaf added 2 commits October 17, 2024 07:45
@nojaf
Code review feedback
ac4d449 
Signed-off-by: nojaf <florian.verdonck@outlook.com>
@nojaf
Pin version for psycopg2-binary
4e7a787 
Signed-off-by: nojaf <florian.verdonck@outlook.com>
@nojaf
Copy link
Collaborator Author

nojaf commented Oct 17, 2024

See psycopg/psycopg2#1737 for the pinned psycopg2-binary

@nojaf nojaf merged commit f9fac9c into mlflow:main Oct 17, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dsuhinin dsuhinin dsuhinin 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.

Support LogInputs in mlflow-go
2 participants
@nojaf @dsuhinin