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

the zero baseline model #38

Open
wants to merge 1 commit into
base: production
Choose a base branch
from
Open

the zero baseline model #38

wants to merge 1 commit into from

Conversation

lujzi05
Copy link

@lujzi05 lujzi05 commented Jun 23, 2024

The zero baseline model: assigns zeros.

  • no sweep run
  • evaluation for testing and calibration
  • forecasting
    It works with data from ./data/raw directory.

@@ -0,0 +1,27 @@
def get_swep_config():
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small typo (sweep)


elif args.run_type == 'forecasting':

execute_model_tasks(config = config, project = project, train = False, eval = False, forecast=True)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

train is not necessarily False. We might also train the model based on the forecasting partition

meta_config = {
"name": "hazel_rabbit",
"algorithm": "zero baseline",
"target(S)": ["ln_sb_best", "ln_ns_best", "ln_os_best", "ln_sb_best_binarized", "ln_ns_best_binarized", "ln_os_best_binarized"],

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just out of curiosity: is there a reason for the capital S in "target(S)"?

Copy link

@angelicalmcgowan angelicalmcgowan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a question out of curiosity – no other comments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new model New model
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants