-
Notifications
You must be signed in to change notification settings - Fork 0
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
teach-logarithm #2
Comments
Szia Bálint! Egyelőre nem foglalkoztam vele sokat, de a következőket látom
A README.md-ben szívesen olvasnék bővebb szöveges kifejtést is! |
Köszi Iván! :)
Az adatok nem közzétehetők, viszont feltettem őket a
Jogos, javítva. Tudom
Kiírja a
Jó, jó... :) Egyébként jól érted.
Sztem jó. :) Én is most vettem rá magam a
Így osztottam be: 68-69. sima grafikon + 71-75. log grafikon + 77-78. jelmagyarázat.
Hm.. gondolkodom rajta.
:) Plusz dolog: kitaláltam ezt a |
|
Nem a pycharm az egyetlen, ami tudja ezt, de biztos van CLI is, csak utána kell nézni...
De minek írja ki? Kell valamire? A grafikon ha jól értem közvetlenül a df-ekből jön létre?
OK, értem!
Nem tudom, hogy van-e erre bevett jelölés, a pep8 semmit nem mond róla. Nekem mindenesetre ez nagyon a shebang-re hasonlít (nem pro vagy kontra mondom, csak ez van). Plusz egy dolog: Ha még csinosítani akarod, akkor a doc string-ekkel lenne érdemes foglalkozni szerintem! Ebben a formában (plot - plot, load - load, sample - sample) nem túl sokatmondóak. |
Igen, igen. :)
Az van, hogy az első 100 nevet mindig közzéteszik, de az egészet nem. Rejtély, hogy miért. Egyszer megvette tőlük az intézet az akkor aktuális adatokat, és mivel pénzért adták, ezért tippelem, hogy nem örülnének, ha közzétennénk. |
Hát nem annyira lényeg, a kiinduló giga adathalmaztól a
Jogos, persze. :) Kösz a megjegyzéseket! |
OK, értem. Azt hittem, a nytud automatikusan tudja ezeket az adatokat, csak valami rossz beidegződés miatt "titkolja".
A main() nem túl fontos szerintem, az csak a parancssori hívhatóságot biztosítja. A lényeg a tényleges függvények dokumentációja (a main-t nem is feltétlenül kell külön függvénybe írni, mehet közvetlenül is az |
Ezzel nagyon nem értek egyet, mert akkor global scope-ba megy az egész és sok galibát fog okozni, a belülről látható változók, amik random külső változókkal azonos nevűek. Javasolnám a main() függvényt, amit az
Ezeket jó így utólag, véletlenül megtudni. :D
Tipp: GDPR, személyiségi jogok? A Zipf görbe miatt az egyedi neveknél viszonylag jól ki lehet következtetni, kiről van szó. talán ezért vágnak.
Nekem is egyből ez jutott eszembe. -> Ezért NEKEM nem tetszik.
Sőt! Túl sok ilyen van és azokat be kell állítgatni, az a baj. A Pycharm, "meg csak működik". A VSCode-ban nézegettem a beköthető programokat, de fél percnél tovább tartott és nem mentem mélyebbre.
A seaborn-t szokták még szeretni, mert egyszerűbb/pythonikusabb mint a matplotlib. A matplotlib inkább a MATLAB-ra akar hajazni, amit ha valaki nem ismer vagy nem szeret, annak bonyolult. A kódról: SZVSZ a semmitmondó általános függvénynevek a) összeakadnak a különféle library-kban ha refaktorálunk vagy keresünk ezért kerülendők, b) helyett kifejtős függvényneveket használnék, ha nem annyira általános a függvény. A docstring és kommentekkel szemben a hívásnál látszik, hogy mit csinál a függvény és nem kell lemenni a definícióig a megértéshez. A A többit Iván már mondta. Csak így tovább! :) |
Stimmel. Engem nem érint a dolog, mert meg szoktam írni a def fun(x):
print(f'fun: x = {x}')
if __name__ == '__main__':
x = 5
fun(x) Mivel az def fun():
print(f'fun: x = {x}')
if __name__ == '__main__':
x = 5
fun() Szóval @dlazesz -nek igaza van, a |
A Pycharm ilyenkor aláhúzza a belső példányt (ha jól be van állítva), hogy jelezze, hogy most épp "felüldefiniálsz" egy globálist, amit nem biztos, hogy így tervezel.
Köszi! Szent igaz! :) |
A feleségem logaritmust tanít a suliban, neki készítettem:
https://github.com/sassbalint/teach-logarithm/blob/main/scripts/log_y_axis.py
Próbáltam szépre csinálni.
Kíváncsi vagyok, hogy láttok-e benne említésre méltó megjegyeznivalót. :)
The text was updated successfully, but these errors were encountered: