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

Run scripts in CI #19

Merged
merged 14 commits into from
Apr 30, 2023
Merged

Run scripts in CI #19

merged 14 commits into from
Apr 30, 2023

Conversation

janosh
Copy link
Owner

@janosh janosh commented Apr 29, 2023

Closes #16.

Add .github/workflows/run-scripts.yml to test a few scripts run error-free in CI.

@janosh janosh temporarily deployed to github-pages April 30, 2023 01:57 — with GitHub Actions Inactive
@janosh
Copy link
Owner Author

janosh commented Apr 30, 2023

I think #17 and #18 should already be fixed unless I made a dumb mistake in #15. Leaving them open for now awaiting @pbenner's feedback. This meanwhile fixes #16.

@janosh janosh merged commit 0da443f into main Apr 30, 2023
@janosh janosh deleted the run-scripts-in-ci branch April 30, 2023 02:04
janosh added a commit that referenced this pull request Jun 20, 2023
* pyproject add [tool.setuptools.package-data] ../data/figshare/* (closes #16)

pyproject add optional-dependencies fetch-data
add url to install aviary from git

* add .github/workflows/run-scripts.yml to test a few scripts run error-free in CI

* make aviary optional in data/wbm/fetch_process_wbm_dataset.py

* add dep pymatviz

* run-scripts.yml continue-on-error: true

* i meant fail-fast: false

* editable install in run-scripts.yml

* install extra deps pymatviz[export-figs] to get kaleido

* add TMP_FIGS dir

looks like data/wbm/fetch_process_wbm_dataset.py might auto-cancelled due to exceeding the download limit or sth like that

just getting Error: The operation was canceled.
so let's test scripts/compile_metrics.py instead

* compile_metrics.py make dataframe_image optional

* run-scripts provide WANDB_API_KEY

* compile_metrics.py fix missing column access

* editable install in gh-pages.yml

* fix vite build ReferenceError: latest_figshare_release is not defined
janosh added a commit that referenced this pull request Jun 20, 2023
* pyproject add [tool.setuptools.package-data] ../data/figshare/* (closes #16)

pyproject add optional-dependencies fetch-data
add url to install aviary from git

* add .github/workflows/run-scripts.yml to test a few scripts run error-free in CI

* make aviary optional in data/wbm/fetch_process_wbm_dataset.py

* add dep pymatviz

* run-scripts.yml continue-on-error: true

* i meant fail-fast: false

* editable install in run-scripts.yml

* install extra deps pymatviz[export-figs] to get kaleido

* add TMP_FIGS dir

looks like data/wbm/fetch_process_wbm_dataset.py might auto-cancelled due to exceeding the download limit or sth like that

just getting Error: The operation was canceled.
so let's test scripts/compile_metrics.py instead

* compile_metrics.py make dataframe_image optional

* run-scripts provide WANDB_API_KEY

* compile_metrics.py fix missing column access

* editable install in gh-pages.yml

* fix vite build ReferenceError: latest_figshare_release is not defined
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

Successfully merging this pull request may close these issues.

Figshare json missing
1 participant