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.

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

BF-ML : Hyperparameter Handling #789

Merged
merged 14 commits into from
Feb 26, 2024
Merged

Conversation

laxmikantbaheti
Copy link
Contributor

Description

Background

Checklists:

  • Read the CONTRIBUTION guide (required).
  • Update the history on the source code (required).

@laxmikantbaheti laxmikantbaheti added the BF Basic Functions/Infrastructure label Jul 24, 2023
@laxmikantbaheti laxmikantbaheti self-assigned this Jul 24, 2023
@laxmikantbaheti laxmikantbaheti linked an issue Jul 24, 2023 that may be closed by this pull request
3 tasks
@laxmikantbaheti
Copy link
Contributor Author

Hi everyone, sorry for the multiple pull requests, I was a bit confused, since the actions weren't running. I realized the actions run only on pull the pull requests compared to main. I will keep this one.

@laxmikantbaheti laxmikantbaheti changed the title BF:ML - Hyperparameter Handling BF-ML : Hyperparameter Handling Jul 24, 2023
@detlefarend
Copy link
Member

@laxmikantbaheti could you please check the current state of this activity? The related branch is outdated and needs to be updated from main. Can we merge to main afterwards?

@detlefarend
Copy link
Member

@laxmikantbaheti branch is updated from main...

@laxmikantbaheti
Copy link
Contributor Author

Hi @detlefarend , so here the story was to improve the basics of hp tuning. And then in subsequent affected layers, I.e. MLP in SL. So the basics are already implemented. This branch can be safely merged. We can later think of MLP in the sl branch.

Shall I do this?

@detlefarend
Copy link
Member

Hi @laxmikantbaheti, thanks for the update. Yes, please merge to main.

@detlefarend detlefarend marked this pull request as ready for review February 26, 2024 11:55
@laxmikantbaheti laxmikantbaheti merged commit 6343e10 into main Feb 26, 2024
9 checks passed
@laxmikantbaheti laxmikantbaheti deleted the bf/ml/hyperparameter_handling branch February 27, 2024 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BF Basic Functions/Infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BF-ML: Extensions for Hyperparameter Updates
2 participants