collab-uniba / pynblint

Pynblint is a linter for Python Jupyter notebooks.
MIT License
40 stars 2 forks source link

Added bottom md ratio function and example in the notebook #27

Closed RomitoVincenzo closed 3 years ago

RomitoVincenzo commented 3 years ago

@louieQ Rimango abbastanza dubbioso sulla scelta di questa implementazione, ritengo che la soluzione con la percentuale di notebook ci consentisse di ricavare più informazioni, mi spiego meglio. Magari le ultime 4 celle possono essere anche contenere markdown, ma in un notebook molto grande la loro percentuale di md risulterà infinitesima e non abbiamo nulla a cui rapportarla, mentre nella scorsa implementazione riuscivamo effettivamente a confrontare la percentuale di markdown separando il notebook in parti uguali e verificare effettivamente se l'ultima sezione di notebook avesse una percentuale di markdown minore rispetto alle altre sezioni. Abbiamo una percentuale di markdown nelle ultime tot celle, ipotizziamo che questo tot celle siano il 10% del notebook , non possiamo arrivare a nessuna conclusione se non abbiamo altre 9 percentuali di markdown per ciascun 10% del notebook. Penso che dovremmo o ritornare alla precedente implementazione, o pensarne una nuova. Dimmi se mi sto perdendo qualcosa.

Ad ogni modo la funzione attuale, così come ci siamo detto è stata creata, è funzionante, e ho inserito il risultato nel dataframe del notebook