Janelia l5 inter rater - Githubissues

BlueBrain / morphoclass

Neuronal morphology preparation and classification using Machine Learning.

https://morphoclass.readthedocs.io

Apache License 2.0

8 stars 4 forks source link

Janelia l5 inter rater #77

Closed FrancescoCasalegno closed 2 years ago

FrancescoCasalegno commented 2 years ago

Fixes #76.

Description

Update pages on using_cnn and using_perslay.
In both pages, the following sections are updated to reflect the current code API:
1. data loading
2. model training
3. model evaluation

How to test?

Follow the instructions in using_cnn.rst and using_perslay.rst.

Checklist

[x] This PR refers to an issue from the issue tracker. (if it is not the case, please create an issue first).
[x] Type annotations added. (if a function is added or modified)
[ ] All CI tests pass.

FrancescoCasalegno commented 2 years ago

Hi @YannRoussel, I tried to run the code and now everything should work. But before I merge this PR and close your issue (#76) could you please also try to run the new code in the docs and confirm that it works for you as well?

After cloning this repo, just do a

git pull

followed by

git checkout janelia-l5-inter-rater

and then open the two pages of the docs under docs/source/sections/:

using_perslay.rst
using_cnn.rst

Thanks in advance :)

FrancescoCasalegno commented 2 years ago

Hi @YannRoussel could you please confirm if you are happy with the changes, so I can merge this PR and close #76 ?

Thanks!

YannRoussel commented 2 years ago

Ho @FrancescoCasalegno Yes, thank you it works fine, and sorry for the delay, I was on leave.

FrancescoCasalegno commented 2 years ago

This PR is from the wrong branch!

Closing and opening #78 instead.