Skip to content

Conversation

Innixma
Copy link
Contributor

@Innixma Innixma commented Oct 2, 2023

Issue #, if available:

Description of changes:

  • Add get_simulation_artifact for zeroshot HPO
  • Added s3 save/load support to save_json and load_json
  • Added s3fs to default dependencies as it is lightweight and is needed to access s3 files via pandas.
  • Added eval_metric column to leaderboard output
  • Added stopping_metric column to leaderboard output when extra_info=True.
  • Added score_format argument to leaderboard to add the option to return the error instead of the score.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Innixma Innixma added enhancement New feature or request module: tabular priority: 0 Maximum priority labels Oct 2, 2023
@Innixma Innixma added this to the 1.0 Release milestone Oct 2, 2023
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

Job PR-3555-e3257b1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-3555/e3257b1/index.html

"label", # The label column name
"""
aggregated_pred_proba = {}
aggregated_ground_truth = {}
Copy link
Collaborator

@shchur shchur Oct 12, 2023

Choose a reason for hiding this comment

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

Maybe with defaultdict we could get rid of the extra checks below?

from collections import defaultdict

def recursive_dd():
    return defaultdict(recursive_dd)

aggregated_pred_proba  = recursive_dd()
# we can now assign aggregated_pred_proba[task_name][fold][key][model] = value

Copy link
Collaborator

Choose a reason for hiding this comment

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

You can also optionally convert the output back to regular dicts with

def dd_to_dict(dd):
    dd = dict(dd)
    for k, v in dd.items():
        if isinstance(v, defaultdict):
            dd[k] = dd_to_dict(v)
    return dd

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point! Updated to use defaultdict

@Innixma Innixma force-pushed the add_zeroshot_hpo_simulation_fetcher branch 2 times, most recently from ff47f8b to 30d0820 Compare October 17, 2023 18:24
@Innixma Innixma force-pushed the add_zeroshot_hpo_simulation_fetcher branch 2 times, most recently from c816933 to 7099ac1 Compare November 8, 2023 19:50
@Innixma Innixma force-pushed the add_zeroshot_hpo_simulation_fetcher branch 2 times, most recently from 9a82ca2 to 882b200 Compare November 10, 2023 22:42
Copy link

Job PR-3555-9a82ca2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-3555/9a82ca2/index.html

Copy link

Job PR-3555-882b200 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-3555/882b200/index.html

@Innixma Innixma force-pushed the add_zeroshot_hpo_simulation_fetcher branch from cf7fd21 to 70e4d4c Compare November 13, 2023 03:19
@Innixma Innixma changed the title WIP: Add get_simulation_artifact Add get_simulation_artifact Nov 13, 2023
Copy link
Contributor

@prateekdesai04 prateekdesai04 left a comment

Choose a reason for hiding this comment

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

LGTM!

@Innixma Innixma merged commit f3028d6 into autogluon:master Nov 13, 2023
LennartPurucker pushed a commit to LennartPurucker/autogluon that referenced this pull request Jun 1, 2024
@Innixma Innixma deleted the add_zeroshot_hpo_simulation_fetcher branch April 16, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request module: tabular priority: 0 Maximum priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants