Skip to content

refresh main with changes after 0.2.0 - 0.2.1 #804

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 30 commits into from
Sep 30, 2022
Merged

refresh main with changes after 0.2.0 - 0.2.1 #804

merged 30 commits into from
Sep 30, 2022

Conversation

lionsardesai
Copy link
Contributor

Description

Catch up main branch with bugfixes and changes preparing for release. This also introduces a condensed github action for syncing notebooks across different folders.

Type of change

N/A

lionsardesai and others added 30 commits September 28, 2022 15:28
* main:
  add tagging for demo notebooks (#793)
* Move intro text to pipeline's TOC section

* Reorganize RTD pages related to artifact storage

* Add a new section on using existing artifacts

* Update index.rst

minor change

* Update index.rst

minor tweaks

* Update s3.rst

minor tweak

* Update s3.rst

@yoonspark let me know if this modification makes sense

* Fix typos; remove duplacate doc

* For Postgres and S3, make clearer distinction between storing artifact values vs. metadata

* Add phrasing suggested by MMA

Co-authored-by: Moustafa AbdelBaky <[email protected]>
* Copy demos repo stories
* Exclude self-hosting-lineapy notebooks from CI
* Use zip artifact store file
* Skip cell due to bugs in pandas
Only happen in python 3.10, so not going to force updating pandas version in requirements.
Related pandas-dev/pandas#41935

Co-authored-by: Humble bot servant <[email protected]>
* Fix mkdir demos error and remove NBVAL_prefix in demos notebooks
* Exclude notebooks in demos from CI
* Modify a comment
* Add tutorials to docs

Co-authored-by: Humble bot servant <[email protected]>
…and docs folder to sync together and commit only once (might need to merge into main for this to work)
Copy link
Contributor

@yoonspark yoonspark left a comment

Choose a reason for hiding this comment

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

Looks good to me, checked that the new GH action does what is intended.

@lionsardesai lionsardesai merged commit ddae428 into main Sep 30, 2022
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.

6 participants