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.

Sign up for GitHub

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

Jump to bottom

Fix multi-layer optimization and broken links #7

Open

KelvinYang0320 wants to merge 2 commits into maestro-project:master from KelvinYang0320:all_layers

KelvinYang0320 commented Sep 4, 2022

We cannot save a proper result_c.csv/plt for multi-layer optimization when we skip --singlelayer because of the following reasons:

Users might not be aware of --num_layer since it isn't listed in the README.md (Basic Usage). Also, the default value of it is 2 which means optimizing the first two layers.
These lines overwrite the existing csv and plt file, so we can only save the last single-layer result.

I address these problems and fix some broken links in the README.md in this pull request.

KelvinYang0320 added 2 commits

September 4, 2022 21:16


          Fix multi-layer optimization

9c0d7df


          Fix broken links

eeb6f73

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet