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

Refactor weight combination schemes #30

Closed
wants to merge 1 commit into from
Closed

Conversation

wehs7661
Copy link
Owner

@wehs7661 wehs7661 commented Nov 1, 2023

This PR aims to address issue #29, which includes the following tasks:

  • Removing the functionality relevant to rmse_cutoff
  • Removing the option final from the YAML parameter w_combine
  • Deprecating the schemes of multiple swaps.

For the reason why these functionalities are removed/deprecated, please refer to the issue.

@wehs7661 wehs7661 self-assigned this Nov 1, 2023
@wehs7661 wehs7661 added enhancement New feature or request documentation Improvements or additions to documentation labels Nov 1, 2023
@wehs7661
Copy link
Owner Author

wehs7661 commented Nov 1, 2023

Closing the PR since it might be easier to merge the master branch first.

@wehs7661 wehs7661 closed this Nov 1, 2023
@wehs7661 wehs7661 deleted the refactor_w_combine branch November 1, 2023 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant