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

[AutoScheduler] Re-organize logs files for tutorials #6768

Merged
merged 2 commits into from
Oct 27, 2020

Conversation

merrymercy
Copy link
Member

@merrymercy merrymercy commented Oct 27, 2020

The previous PR saves the demo log files in the top-level folder tutorials/ and saves the generated log files inside the tutorials/logs/ folder.

But the demo logs files are only used to prevent CI errors, so they should be hidden.

In this PR, we move the demo log files to tutorials/ci_logs/ to hide them from users.
We then save the generated log files to the top-level tutorials/, which is the same folder of the scripts.

@merrymercy merrymercy requested a review from comaniac October 27, 2020 03:18
@merrymercy merrymercy changed the title [AutoScheduler Re-organize logs files for tutorials [AutoScheduler] Re-organize logs files for tutorials Oct 27, 2020
@comaniac comaniac self-assigned this Oct 27, 2020
Copy link
Contributor

@comaniac comaniac left a comment

Choose a reason for hiding this comment

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

LGTM. Make sense to me.

Copy link
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

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

LGTM

@merrymercy merrymercy merged commit 453d30c into apache:main Oct 27, 2020
@merrymercy merrymercy deleted the pr-ci-tutorial branch October 27, 2020 14:05
zhiics pushed a commit to zhiics/tvm that referenced this pull request Oct 28, 2020
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Oct 29, 2020
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Dec 2, 2020
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Dec 4, 2020
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Dec 4, 2020
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.

3 participants