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.

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

Omit TOC due to github feature change(We don't need TOC now.) #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Omit TOC due to github feature change(We don't need TOC now.) #3

wants to merge 4 commits into from

Conversation

kyoro1
Copy link
Contributor

@kyoro1 kyoro1 commented Oct 2, 2021

I found other TOC description in the repository, and omit them.
The reason why we don't need it is shown here

@kyoro1
Copy link
Contributor Author

kyoro1 commented Oct 2, 2021

Hi @WillieAhlers1 , please check & merge them, if it doesn't have any issues.

@kyoro1
Copy link
Contributor Author

kyoro1 commented Oct 2, 2021

The 2nd content is related to the issue.

@kyoro1
Copy link
Contributor Author

kyoro1 commented Oct 3, 2021

In executing train_n_classifier.py under /notebooks folder without Azure ML workspace, the process hadnot completed.
At least, it continued 1-2 hours.

!python ../src/train_n_classifier.py \
    --dataset-name "../docs/data/Two_class.csv"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kyoro1