Skip to content
This repository was archived by the owner on Jan 3, 2023. It is now read-only.

add scala notebooks & starting scripts. #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add scala notebooks & starting scripts. #39

wants to merge 1 commit into from

Conversation

Calculuser
Copy link

No description provided.

@megaSpoon
Copy link
Contributor

megaSpoon commented Jan 11, 2018

the same modifications are already applied to these notebooks as those in python language. Should I add the changes in this PR or just start a new PR to do it?

Also overall, no big errors to be fixed other than some dependency missing and dataset path not found exceptions. I will put more details to describe it later in the commits.

@megaSpoon
Copy link
Contributor

The loss curve of autoencoder notebook tutorial is not correct. I'm finding why this malfunction happens.

@Calculuser
Copy link
Author

Thanks for your update. @megaSpoon

In fact, I will start new PRs to finish Scala tutorials later according to your modifications in python version. So, I think you do not need to add changes in this PR.

If you find the reason why the loss curve is not correct, please contact me. Thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants