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

Conversation

@janpipek
Copy link
Member

Druhá lekce Pražské EDA:

Měla by se skládat ze dvou notebooků.

Tento, delší, pojednává o manipulaci s DF, typech, aritmetice, filtrování a řazení.
Druhý, kratší, bude o základech vizualizace a objeví se co nevidět v tomto pull requestu.

Prosím o kritiku.

@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.

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

@AndrejSvitek AndrejSvitek Dec 26, 2019

Choose a reason for hiding this comment

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

Asi by som to trocha preformuloval, aby bolo jasnejší zámer - nie že to nie je praktické, len sa to využíva skôr v špecifických prípadoch (ak to teda zámer bol :-D).

Napáda mi nasledovné:

  1. Nechať len "Pro hodnoty nového sloupce lze použít i jednu skalární hodnotu:"
  2. Nechať to, čo v prípade 1, len na koniec pridať "v praxi sa s touto potrebou nestretneme tak často" alebo niečo podobné.

Reply via ReviewNB

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

@AndrejSvitek AndrejSvitek Dec 26, 2019

Choose a reason for hiding this comment

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

Ako dobrá pomôcka pre začiatočníkov mi príde , že:

  • axis=0 sa tiež dá napísať ako axis='index' alebo axis='rows'.
  • axis=1 ako axis='columns'.

Dokumentácia tvrdí, že funguje pre všetky pandas funkcie s týmto argumentom, ale našiel som nejakú, kde to nešlo (už si ale nespomeniem).

A keďže sme v pandas, tak df.mean(axis='rows') vracia priemer za jednotlivé stĺpce a df.mean(axis='columns') za jednotlivé riadky. Axis tu špecifikuje, po ktorej ose sa priemer počíta - logicky :-D.


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.

Today I learnt ;-)

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

@AndrejSvitek AndrejSvitek Dec 26, 2019

Choose a reason for hiding this comment

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

ROFL!


Reply via ReviewNB

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

@AndrejSvitek AndrejSvitek Dec 26, 2019

Choose a reason for hiding this comment

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

"s rychlostí řádově rychlejší" -> "s rychlostí řádově vyšší"


Reply via ReviewNB

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

@coobas coobas Dec 28, 2019

Choose a reason for hiding this comment

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

To bychom ale měli spíš nedoporučovat


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.

Pentagram ti nestačí?

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

@coobas coobas Dec 28, 2019

Choose a reason for hiding this comment

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

# Místo set_index dát výš - je myslím trochu matoucí jestli to neptří k sort_index


Reply via ReviewNB

Copy link

coobas commented Dec 28, 2019

Za mě hodně povedené, v angličtině by to klidně mohlo doplnit oficiální Pandas docs :) Bude to náročné na čas, řekl bych že při výuce bude potřeba něco schovat na samostudium. Pokud se to zvládne, tak dostatek materiálu na doma je určitě super!

@janpipek janpipek changed the title WIP: EDA 2 - typy a operace EDA 2 - typy a operace Jan 1, 2020
@janpipek
Copy link
Member Author

janpipek commented Jan 1, 2020

Co ještě přidat?

  • rename? (používá se v EDA 3)

@coobas
Copy link

coobas commented Jan 1, 2020

* `rename`? (používá se v EDA 3)

používá, ale asi není nutné vysvětlit tady

janpipek added a commit that referenced this pull request Jan 10, 2020
@janpipek janpipek mentioned this pull request Jan 10, 2020
3 tasks
@janpipek
Copy link
Member Author

Zavírám ve prospěch #36

@janpipek janpipek closed this Jan 10, 2020
janpipek added a commit that referenced this pull request Jan 18, 2020
janpipek added a commit that referenced this pull request Jan 22, 2020
janpipek added a commit that referenced this pull request Jan 22, 2020
* EDA2: before applying comments

* Zapracování komentářů  z #23

* Zapracování poznámek k typům

* Zapracování poznámek k vizualizacím

* info.yml pro EDA2

* Drobné příklady pro EDA2 (první část)

* Vizualizace - úkoly

* info.yml pro vizualizace

* Přidání EDA 2 do definice runu

* Fix: špatně cesta k vizualizace

* Několik zapracovaných změn

* Zapracovány Ondřejovy poznámky

* EDA 2: odkazy na metody a stahování ve vizualizacích

* Zapracování Ondřejových poznámek k vizualizaci + few small fixes

* Update lesson titles
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