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.

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

Milano #7

Open
wants to merge 19 commits into
base: milano
Choose a base branch
from
Open

Milano #7

wants to merge 19 commits into from

Conversation

luigi-calori
Copy link

Abbiamo aggiunto un notebook per la parte di matplotlib e Bokeh presentata a Milano

Per farlo abbiamo fatto un fork, dateci un' occhiata e se è accettabile, metteelo insieme

Grazie
Luigi

@pdonorio
Copy link
Contributor

Ciao Luigi,
un paio di proposte per darci delle regole comuni per il futuro
(che se concodate conviene segnare nel README principale del repo)

  • Usare solo la lingua inglese, nei commenti, nei nomi, nelle pull request etc. - dalla prossima direi ;)
  • Il Readme della cartella pydata potrebbe riportare le sessioni e i relativi notebook
  • Potremmo dare i nomi ai notebook con prefissi tipo SessionXX_YY_NAME.ipynb (e.g. Session01_02_numpy.ipynb) in modo da mostrare i file già in ordine nella pagina principale
  • Spostare le cose che non sono notebook in altre sottocartelle, e.g. scripts, data, images e così via

Pingo @gfiameni al riguardo così che queste decisioni le prendiamo una volta per tutte adesso.

A fine "lavoro" possiamo spostare tutto il necessario su master, in maniera ufficiale.

Ultima nota: non ho accesso di push al tuo branch e quindi io adesso non potrei farci modifiche, dovresti farle tutte tu se le concordassimo. L'alternativa sarebbe fare questo merge e modificare di concerto a seguito.

Paolo

@luigi-calori
Copy link
Author

Scusa... è la prima volta che uso github, quindi non so bene come funziona.

On 26/06/2015 12:19, Paolo D. wrote:

Ciao Luigi,
un paio di proposte per darci delle regole comuni per il futuro
(che se concodate conviene segnare nel README principale del repo)

  • Usare solo la lingua inglese, nei commenti, nei nomi, nelle pull
    request etc. - dalla prossima direi ;)

Concordo pienamente, non pensavo che la pull request entrasse tra i
commit e quindi la ho trattata come una mail

  • Il Readme della cartella |pydata| potrebbe riportare le sessioni e
    i relativi notebook

Non mi pare che ci sia un Readme nella cartella pydata, mentre ce ne è
uno, che ho usato per prova nella cartella principale

  • Potremmo dare i nomi ai notebook con prefissi tipo
    SessionXX_YY_NAME.ipynb (e.g. Session01_02_numpy.ipynb) in modo da
    mostrare i file già in ordine nella pagina principale

Ok, provo rinominarle

  • Spostare le cose che non sono /notebook/ in altre sottocartelle,
    e.g. scripts, data, images e così via

concordo, probabilmente ci sono cose che forse non dovrebbero nemmeno
esserci, sono script di gestione ma non volevo dimenticarmi: puoi fare
un pull selettivo con git o devi accettare tutto?

Pingo @gfiameni https://github.com/gfiameni al riguardo così che
queste decisioni le prendiamo una volta per tutte adesso.

A fine "lavoro" possiamo spostare tutto il necessario su master, in
maniera ufficiale.

Ultima nota: non ho accesso di push al tuo branch e quindi io adesso
non potrei farci modifiche, dovresti farle tutte tu se le
concordassimo. L'alternativa sarebbe fare questo merge e modificare di
concerto a seguito.

Io non ho nessun problema a darti i diritti di push, se solo imparo come
si fa ;-)
Se vuoi provo a ripulire le cose per come ho capito e poi ti faccio un
nuovo pull request.

Magari sarebbe opportuno versionare anche le ricette per le immagini docker.

Paolo


Reply to this email directly or view it on GitHub
#7 (comment).

Luigi Calori
SuperComputing Applications and Innovation Department
CINECA - via Magnanelli, 6/3, 40033 Casalecchio di Reno (Bologna) - ITALY
Tel: +39 051 6171509 Fax: +39 051 6132198
hpc.cineca.it

@pdonorio
Copy link
Contributor

pdonorio commented Jul 1, 2015

Il giorno ven 26 giu 2015 alle ore 15:14 Luigi Calori <
[email protected]> ha scritto:

Scusa... è la prima volta che uso github, quindi non so bene come funziona.

Ma che scusa, figurati!

  • Il Readme della cartella |pydata| potrebbe riportare le sessioni e
    i relativi notebook

Non mi pare che ci sia un Readme nella cartella pydata, mentre ce ne è
uno, che ho usato per prova nella cartella principale

Si, per questo se vogliamo tenerlo possiamo aggiungere quei dati, per
esempio. Nessun probelma al riguardo.

puoi fare

un pull selettivo con git o devi accettare tutto?

La seconda. La prassi è che il richiedente corregga la pull request come
richiesto fino a essere utilizzabile.

Io non ho nessun problema a darti i diritti di push, se solo imparo come
si fa ;-)
Se vuoi provo a ripulire le cose per come ho capito e poi ti faccio un
nuovo pull request.

Se hai modo tu è meglio, io sono preso da impegni familiari (bimbo in
arrivo ;D )

Magari sarebbe opportuno versionare anche le ricette per le immagini
docker.

Vedi il branch "docker" che sto usando:
https://github.com/cineca-scai/lectures/tree/docker/docker/notebook

L'ho collegato al docker hub di cineca:
https://registry.hub.docker.com/repos/cineca/

Paolo

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

Successfully merging this pull request may close these issues.

2 participants