Skip to content

Fix: connect 02 and 03 #11

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

Merged
merged 3 commits into from
Nov 7, 2022
Merged

Fix: connect 02 and 03 #11

merged 3 commits into from
Nov 7, 2022

Conversation

niklas-palm
Copy link
Collaborator

Infers the S3 model path from the HPO job, and stores it under a known prefix.
Uses the above prefix in the evaluation.

@niklas-palm niklas-palm added the enhancement New feature or request label Nov 4, 2022
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@absynthe absynthe merged commit 9f0b787 into main Nov 7, 2022
firdousi-begum added a commit that referenced this pull request Nov 7, 2022
commit e6684e2
Merge: 9f0b787 5f17132
Author: Anamaria Todor <[email protected]>
Date:   Mon Nov 7 08:52:54 2022 +0100

    Merge pull request #12 from absynthe/refactor/04pipelines

    refactor: unused vars, language and import changes

commit 9f0b787
Merge: 75985c7 388b6b6
Author: Anamaria Todor <[email protected]>
Date:   Mon Nov 7 08:52:33 2022 +0100

    Merge pull request #11 from absynthe/fix/connect-02-03

    Fix: connect 02 and 03

commit 5f17132
Merge: ada1a8a 8bac4c3
Author: niklas-palm <[email protected]>
Date:   Sun Nov 6 21:58:22 2022 +0100

    Merge pull request #13 from absynthe/refactor/update-model-fetching

    refactor: Update model fetching

commit 8bac4c3
Author: Niklas Palm <[email protected]>
Date:   Sun Nov 6 20:57:29 2022 +0000

    Remove output

commit 6aac58d
Author: Niklas Palm <[email protected]>
Date:   Sun Nov 6 20:54:02 2022 +0000

    Fetch model from known prefix

commit ada1a8a
Author: Niklas Palm <[email protected]>
Date:   Sun Nov 6 20:36:35 2022 +0000

    Remove cleanup step

commit f559fab
Author: Niklas Palm <[email protected]>
Date:   Sun Nov 6 20:34:43 2022 +0000

    Remove unsed processor and fetch JsonGet from functions package

commit 8567136
Author: Niklas Palm <[email protected]>
Date:   Sun Nov 6 20:30:11 2022 +0000

    Remove model approval status

commit a31a0f7
Author: Niklas Palm <[email protected]>
Date:   Sun Nov 6 19:31:12 2022 +0000

    Remove approaval status parameter

commit 388b6b6
Author: Niklas Palm <[email protected]>
Date:   Sun Nov 6 19:18:18 2022 +0000

    Lower number of max training jobs

commit b4aba82
Merge: f14b6c7 75985c7
Author: Niklas Palm <[email protected]>
Date:   Fri Nov 4 13:46:11 2022 +0000

    Merge branch 'main' into fix/connect-02-03

commit f14b6c7
Author: Niklas Palm <[email protected]>
Date:   Fri Nov 4 13:45:07 2022 +0000

    Connect hpo and evaluation
@niklas-palm niklas-palm deleted the fix/connect-02-03 branch November 7, 2022 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants