37b-4-tiphunting #278
37b-4-tiphunting #278
Conversation
|
Σχετικά με το stack overflow: αν κάνεις compile σε windows σκάει πολύ πιο εύκολα το stack λόγω περισσότερων αποθηκευμένων δεδομένων. Σε linux ίσως να μην υπάρχει θέμα. Θα κάνω δοκιμή σε linux αργότερα. |
makis-modulus
left a comment
makis-modulus
left a comment
There was a problem hiding this comment.
Thanks! I took care of most of the comments.
Θα βάλω σχόλια στον κώδικα της optimal λύσης σε επόμενο commit.
marsenis
left a comment
marsenis
left a comment
There was a problem hiding this comment.
Major changes:
- Έκανα όλα τα vectors global σε όλα τα subtasks. Thanks for the suggestion, Δημήτρη! Όντως βοηθάει με το stack usage, αλλά ακόμα χάνω 4 test cases λόγω stack overflow αν δεν ανεβάσω το όριο σε 16 MB.
- Άλλαξα ονόματα των μεταβλητών
src, dstστον κώδικα σεL, Rώστε να ταιριάζει με το κείμενο. - Στον κώδικα πλεον χρησιμοποιώ παντού το convention
parent[root] = root, σε αντίθεση με πριν όπου σε κάποια σημεία χρησιμοποιούσαparent[root] = -1.
Μένει μόνο να βάλω code snippets και code links στο κείμενο. Θα το κάνω αργότερα.
| for (long i = 0; i < n-1; ++i) { | ||
| long u, v, w; | ||
| scanf("%li%li%li", &u, &v, &w); | ||
| assert(1 <= u && u <= n); |
There was a problem hiding this comment.
Ισχύει. Τα είχα μόνο για debugging βασικά. Τα βγάζω.
Dim131
left a comment
Dim131
left a comment
There was a problem hiding this comment.
Μερικά μικρά σχόλια. Κατά τα άλλα νομίζω είναι έτοιμο!
Μερικές εικόνες μπορεί να ήταν ωραίο να είχαμε, αλλά δεν πειράζει αν δεν έχεις χρόνο
| passes_only = [2, 3, 4, 5], | ||
| lang = "c++", | ||
| ), | ||
| SOLUTION( |
There was a problem hiding this comment.
Fixed.
Fyi, έκανα copy-paste από το TASK του samepizzas. Έχουν ξεμείνει κι εκεί tabs.
kbokis
left a comment
kbokis
left a comment
There was a problem hiding this comment.
Μερικά typos κυρίως. Ωραία!
4 participants
Lower priority TODOs (probably not in this PR
loopμε τη λέξηcycleγιατί κατάλαβα ότι "loop" είναι λίγο άστοχο (θυμίζει self-loop), και μοιάζει πολύ με τη λέξη "root" που επίσης χρησιμοποιώ σε παρόμοιο context. Πείτε μου κι εσείς τη γνώμη σας.