-
Notifications
You must be signed in to change notification settings - Fork 5
Add more documentation and notebooks #106
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
Conversation
|
@sinnec I made a quick review. |
|
@NickNtamp Weird because I used pip to install it. Do you believe we could combine the SDK documentation with the notebook, because their content is almost identical? |
As it is the 1st iteration, seems fine for me. |
|
@stavrostheocharis Well, the code exists in the "Using Whitebox" section and the descriptions provided are more detailed than some comments. It's almost identical (at least the one I made). I don't reject that thought of just referencing the notebooks, also given the fact that they look kinda ugly in the docs. Not all markdown can be applied on them. I made a sample one if you ewant to check out and we can decide on what we want to do and continue with that. I personally prefer a folder with notebooks that's dettached from the docs. Your thoughts @NickNtamp @momegas ? |
|
See the sections "Using Whitebox" and the mispelled "Notebooks/Usaging Whitebox" under the tutorial section. They tell the same story but one looks uglier! |
As it is a first iteration i like it @sinnec. The only bizarre thing is that, ok...we do this for what reason? I mean we create a model, load a training and inference dataset and then what? I think that we have to give something at the end (e.g. use the UI, go to page http... etc.) |
NickNtamp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very good work @sinnec . Some links are missing, especially in metrics etc but I can fill them, working in the other PR for the regression models. Ok from my side.
|
I know that there are missing links. After the doc's restructure I didn't know where they should point and since I don't know if they will work, I omitted them. 😊 |


Edit: This PR brings the following changes:
I'll add more markdown text, but how does it seem as a first iteration?