Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Conversation

@janpipek
Copy link
Member

@janpipek janpipek commented Jan 10, 2020

Nahrazuje #23 s aplikací tamějších komentářů.

Důvod: Chceme větev uvnitř PyDataCZ repa, aby se dalo checkoutovat bez přidání remotes.

TODO:

  • Přidat nějaké úkoly?
  • Zapracovat komentáře, které mám externě
  • Přidat YAMLy

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

@janpipek janpipek changed the title WIP: [Materiál Praha] EDA 2 [Materiál Praha] EDA 2 Jan 10, 2020
@janpipek janpipek mentioned this pull request Jan 10, 2020
@janpipek
Copy link
Member Author

Poznámka: nějak se mi nezobrazuje plotly.express v Jupyter Labu. Je potřeba něco doinstalovat?

@@ -0,0 +1,10723 @@
{
Copy link

@kokes kokes Jan 17, 2020

Choose a reason for hiding this comment

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

Budem tu chtít poznamenat, že alternativou je ještě df.copy(), nebo to si necháme na jindy?

(Byla teď moc pěkná přednáška Vincenta Warmerdama, kde ukazoval pandas pipeliny a na začátku udělal vždy copy, aby měl proces bez side effectů).


Reply via ReviewNB

Copy link
Member Author

Choose a reason for hiding this comment

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

"Mimochodem, kopii tabulky můžeš kdykoliv vytvořit metodou copy - to se hodí třeba při psaní funkcí, kde se vstupní tabulka z různých důvodů upravuje."

Copy link

Choose a reason for hiding this comment

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

jj, něco takovýho

@@ -0,0 +1,10723 @@
{
Copy link

@kokes kokes Jan 17, 2020

Choose a reason for hiding this comment

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

Je obvious, že se to propíše do všech řádků? Asi jo... jen já to teda chápal hlavně díky tomu, že to vim, tak přemýšlím, jestli to tam napsat explicitně.


Reply via ReviewNB

Copy link
Member Author

Choose a reason for hiding this comment

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

Explicit is better than implicit.

Dávám to tam.

Copy link

Choose a reason for hiding this comment

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

👍

@@ -0,0 +1,10723 @@
{
Copy link

@kokes kokes Jan 17, 2020

Choose a reason for hiding this comment

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

Proč?


Reply via ReviewNB

Copy link
Member Author

Choose a reason for hiding this comment

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

Tak jsem to vymazal :-)

Copy link

Choose a reason for hiding this comment

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

No ne, třeba to má důvod, to si klidně obhaj :)

Copy link
Member Author

Choose a reason for hiding this comment

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

Se mi už nechtělo ;-)

@@ -0,0 +1,10723 @@
{
Copy link

Choose a reason for hiding this comment

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

Asi bych lidi nezatěžoval binárníma formátama. Myslim že pro začátečníky je s nima víc problémů než užitku. A pro jejich datasety asi problém s výkonem nebude. CSV je v pohodě, navíc to bude jednoduše shareovatelné.


Reply via ReviewNB

Copy link
Member Author

Choose a reason for hiding this comment

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

"Excel ani CSV nejsou formáty pro ukládání velikých dat zcela vhodné (jako alternativy se nabízí třeba feather nebo parquet), pro naše účely (malé soubory, čitelný textový formát) ale budou CSV postačovat."

Copy link

Choose a reason for hiding this comment

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

jop, super

@review-notebook-app
Copy link

review-notebook-app bot commented Jan 18, 2020

View / edit / reply to this conversation on ReviewNB (backstory for this conversation format).

kokes commented on 2020-01-18T18:20:53Z
----------------------------------------------------------------

místo split/rsplit radši používám partition/rpartition, protože to má fixní počet výstupů

nečekám změnu v materiálu, jen tě nudguju k jinejm str metodám pro vlastní použití


janpipek commented on 2020-01-20T11:40:57Z
----------------------------------------------------------------

Díky.

@review-notebook-app
Copy link

review-notebook-app bot commented Jan 18, 2020

View / edit / reply to this conversation on ReviewNB (backstory for this conversation format).

kokes commented on 2020-01-18T18:20:54Z
----------------------------------------------------------------

jak jsme se dohodli s tím query? Používá se to? budem to používat my?

navíc tady je ten filtr takový trochu tajemný - query "is_eu"? Jako chápu co to dělá, ale je to takové kryptické.


janpipek commented on 2020-01-20T11:44:35Z
----------------------------------------------------------------

Já bych query používal. Ale tohle zkusím dekryptizovat.

@review-notebook-app
Copy link

review-notebook-app bot commented Jan 18, 2020

View / edit / reply to this conversation on ReviewNB (backstory for this conversation format).

kokes commented on 2020-01-18T18:20:55Z
----------------------------------------------------------------

viz diskuse na slacku - změnit na logx/logy, což je čitelnější


janpipek commented on 2020-01-20T11:44:12Z
----------------------------------------------------------------

To je úplně, ale úplně super, protože to umožňuje v této lekci (až na appendix) vůbec matplotlib neřešit. Díky za postřeh.

@janpipek janpipek requested a review from coobas January 21, 2020 12:50
@janpipek janpipek merged commit 11e3583 into master Jan 22, 2020
@janpipek janpipek deleted the prague-eda2 branch March 6, 2020 18:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants