claimed-framework / claimed-framework/mlx_deprecated

Address caveats katalog update script

Open
#222 1 comment 0 reactions 1 assignee Claimed by @JiaxuanYang View on GitHub
help wanted
Dominant language
Python
Stars
220
Forks
54
PR merge metrics
No merged PRs in 30d

Description

We need to address (some of) the current caveats in the catalog update script:

- [ ] move script from /hack to /tools/python
- [ ] add developer documentation
- [ ] clone the `katalog` repo into a temp directory instead of relying on the user to have done it prior to running the script
- [ ] not all of the YAML files in the `katalog` repo may want to be included in the catalog upload, so alert the user to all the changes instead of relying on `git diff`
- [ ] read the `catalog_upload.json` file prior to regenerating it to capture assets that live outside the `katalog` repo and re-add them later

_Originally posted by @ckadner in https://github.com/machine-learning-exchange/mlx/issues/200#issuecomment-926195710_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.