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

Add a function to move a local config dict and all of it's assets to google cloud storage #47

Open
nbren12 opened this issue Jan 17, 2020 · 2 comments

Comments

@nbren12
Copy link
Contributor

nbren12 commented Jan 17, 2020

For the one-step workflows, @oliverwm1 wrote some code which prepares a local configuration dictionary with a mix of local and remote assets. The initial conditions are on GCS, but the diag_table and some other things are local. It would be nice to refactor the manual file-moving code in that workflow to a function in this package. I imagine something like this interface

def upload_config_to_url(config, url):
    """Upload a config dict and all of it's assets to the url
    """

The main difficulty is what the locations of the assets should be. Maybe, url should point to a directory, which will contain the assets and the configuration yaml.

@nbren12
Copy link
Contributor Author

nbren12 commented Jan 17, 2020

@oliverwm1
Copy link

I would suggest

def upload_config_to_url(config, url) -> config:

where the returned config dict has all of its local paths updated to the appropriate remote urls.

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

No branches or pull requests

2 participants