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

feat(appunti): Aggiunta relazione esame Tecniche di Acquisizione Dati #76

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Fed19
Copy link

@Fed19 Fed19 commented Dec 11, 2023

No description provided.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Questo file a cosa serve ?
Penso lo possiamo rimuovere ?

@ncvescera ncvescera added the nuovo materiale Aggiunta di nuovi appunti/dispense per un esame label Dec 17, 2023
@ncvescera ncvescera changed the title aggiunta relazione esame su tecniche di acquisizione dati feat(appunti): Aggiunta relazione esame Tecniche di Acquisizione Dati Dec 17, 2023
@Fed19
Copy link
Author

Fed19 commented Dec 18, 2023 via email

Rimosso file inutile caricato per errore.
@ncvescera ncvescera added the ready La Pull Request è pronta per il merge label Dec 18, 2023
@CristianCosci
Copy link
Collaborator

@ncvescera Visto che non sono molte pagine, non si potrebbe riportare tutto in latex e utilizzare il nostro template?

@ncvescera
Copy link
Member

@ncvescera Visto che non sono molte pagine, non si potrebbe riportare tutto in latex e utilizzare il nostro template?

Dici di farlo direttamente da questa PR ?
Oppure effettuare il merge e farlo in un secondo momento, magari aprendo un'issue per segnalare la cosa ?

@CristianCosci
Copy link
Collaborator

@ncvescera Visto che non sono molte pagine, non si potrebbe riportare tutto in latex e utilizzare il nostro template?

Dici di farlo direttamente da questa PR ? Oppure effettuare il merge e farlo in un secondo momento, magari aprendo un'issue per segnalare la cosa ?

Ci sono già altri mille problemi e issue da risolvere nella roba della triennale. Rimandarlo sarebbe uguale e dire che non verrà mai fatto. Vista la ridotta quantità di roba, secondo me andrebbe fatto ora. Avere la PR aperta è l'unico modo per forzare questa modifica

@ncvescera
Copy link
Member

@ncvescera Visto che non sono molte pagine, non si potrebbe riportare tutto in latex e utilizzare il nostro template?

Dici di farlo direttamente da questa PR ? Oppure effettuare il merge e farlo in un secondo momento, magari aprendo un'issue per segnalare la cosa ?

Ci sono già altri mille problemi e issue da risolvere nella roba della triennale. Rimandarlo sarebbe uguale e dire che non verrà mai fatto. Vista la ridotta quantità di roba, secondo me andrebbe fatto ora. Avere la PR aperta è l'unico modo per forzare questa modifica

Perfetto, allora la lasciamo ancora aperta, faccio una review dicendo che va fatta la conversione in LaTex e rifiuto l'approvazione

@CristianCosci
Copy link
Collaborator

@ncvescera Visto che non sono molte pagine, non si potrebbe riportare tutto in latex e utilizzare il nostro template?

Dici di farlo direttamente da questa PR ? Oppure effettuare il merge e farlo in un secondo momento, magari aprendo un'issue per segnalare la cosa ?

Ci sono già altri mille problemi e issue da risolvere nella roba della triennale. Rimandarlo sarebbe uguale e dire che non verrà mai fatto. Vista la ridotta quantità di roba, secondo me andrebbe fatto ora. Avere la PR aperta è l'unico modo per forzare questa modifica

Perfetto, allora la lasciamo ancora aperta, faccio una review dicendo che va fatta la conversione in LaTex e rifiuto l'approvazione

Tempo permettendo, sarebbe utile una guida da mettere nella wiki su come riportare appunti ecc.. come questi in latex con il nostro template prima di una PR

Copy link
Member

@ncvescera ncvescera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO 📋

  • Riscrivere la relazione utilizzando il nostro template LaTex

@ncvescera
Copy link
Member

@ncvescera Visto che non sono molte pagine, non si potrebbe riportare tutto in latex e utilizzare il nostro template?

Dici di farlo direttamente da questa PR ? Oppure effettuare il merge e farlo in un secondo momento, magari aprendo un'issue per segnalare la cosa ?

Ci sono già altri mille problemi e issue da risolvere nella roba della triennale. Rimandarlo sarebbe uguale e dire che non verrà mai fatto. Vista la ridotta quantità di roba, secondo me andrebbe fatto ora. Avere la PR aperta è l'unico modo per forzare questa modifica

Perfetto, allora la lasciamo ancora aperta, faccio una review dicendo che va fatta la conversione in LaTex e rifiuto l'approvazione

Tempo permettendo, sarebbe utile una guida da mettere nella wiki su come riportare appunti ecc.. come questi in latex con il nostro template prima di una PR

Dici una specie di guida sul formato che gli appunti devono avere e più o meno come li vogliamo ?
Si, non sarebbe male. Dobbiamo ancora introdurre la wiki però e aggiornare il README magari aggiungendo una sezione dove segnaliamo la presenza della wiki, o magari solo un badge all'inizio del readme 🤔

@ncvescera ncvescera removed the ready La Pull Request è pronta per il merge label Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nuovo materiale Aggiunta di nuovi appunti/dispense per un esame
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants