8000 public site update for ch1, 2 polish by trevorcampbell · Pull Request #72 · UBC-DSCI/introduction-to-datascience-python · GitHub
[go: up one dir, main page]

Skip to content

public site update for ch1, 2 polish #72

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

Merged
merged 7 commits into from
Dec 21, 2022
Merged

public site update for ch1, 2 polish #72

merged 7 commits into from
Dec 21, 2022

Conversation

trevorcampbell
Copy link
Contributor

will merge immediately

trevorcampbell and others added 7 commits December 19, 2022 00:04
* empty commit for WIP PR

* ignore vim swap files

* python and pandas fix; code tags around df[]

* fix 'Chapter X' typesetting

* an python fix

* fix broken url to data in this repo

* fixed capitalization in pandas team in refs.bib

* more bugfix in pandas dev team refs.ib

* pd alias; removing print stmts in initial naming part; fixed naming syntax

* single vs double quotes; property -> operation; df[] to just []

* fixed chapter typesetting to be even better

* added note about wrong fig 1.2

* added remarks about square brackets and dots

* dataframe -> data frame

* documented all missing figures

* height of bars to lengths of bars; fixed exercises title and url

* added explanation of ranges

* polish and simplification of altair intro; fixed multi-line expression text

* fixed putting it together code block; removed docstring help

* editing learning objs for overhaul of loc/[]/iloc in ch1

* get rid of prepended . on function/method names. some minor wording polish

* fixed loc and [] in chapter 1

* putting all together new loc and []

* altair name typesetting consistency

* general -> powerful in notebox

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* remove unnecessary second multiline example

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* build html executable

* switched [] to be for rows or cols, and loc[] to be for both

* minor wording polish

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* minor spacing fix

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* minor wording polish

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* minor spacing fix

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>
* empty commit for WIP PR

* hide ignore warnings cell; import pandas move down; remove read_table from objectives; .csv formatting consistency

* fixed canadian language data with metainfo

* fixed tsv/no cols data in ch2

* working on polishing reading

* polished ch2 up to db section

* commented out web scraping and api for first round; issue opened

* commented out learning objs for scraping/api

* built_html executable

* done new ibis database section

* Fixed learning objectives sqlalch -> ibis; minor sqlalch bugfixes

* ibis "modern"

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* nicer formatting of readcsv with args

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* minor polish on wording

* fix header in skiprows example

* nicer formatting on ibis connect args

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>

* Removed warnings filter--unneeded

* index false in to_csv

Co-authored-by: Joel Ostblom <joelostblom@users.noreply.github.com>
@trevorcampbell trevorcampbell merged commit 5b14f0a into production Dec 21, 2022
@netlify
Copy link
netlify bot commented Dec 21, 2022

Deploy Preview for mellifluous-pie-f3e391 failed.

Name Link
🔨 Latest commit 553aaa5
🔍 Latest deploy log https://app.netlify.com/sites/mellifluous-pie-f3e391/deploys/63a3399a6fb4a8000881278c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0