Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update and correct docs #180

Merged
merged 56 commits into from
Oct 30, 2024
Merged

Update and correct docs #180

merged 56 commits into from
Oct 30, 2024

Conversation

griff-rees
Copy link
Collaborator

@griff-rees griff-rees commented Oct 29, 2024 •
edited
Loading

  • Refactor navigation
  • Remove files no longer in use
  • Deprecate previous documentation
  • Fix broken links and references
  • Add day/night theme

This includes and supersedes #174

spool and others added 30 commits September 26, 2024 23:15
@spool
feat: add rough version of join_xr_time_series
98025a3
@spool
feat: rename and improve join_xr_time_series -> `join_xr_time_serie…
66aaa29 
…s_var`
@spool
feat: finish join_xr_time_series_var to summarise xarray time series
488139a
@spool
feat: add annual_group_xr_time_series plotting
34939d8
@spool
doc: add quarto doc for cpm interporation
0463ae8
@sgreenbury
Fix extension
859484f
@sgreenbury @griff-rees
Edits while running quarto notebook
abf227e 
Co-authored-by: griff-rees <[email protected]>
@spool
doc: add jupyter caching and rename to cpm_projection.qmd
3522ee4
@spool
doc: fix import error in cpm_projection.qmd
0c7ab94
@spool
fix: ensure join_xr_time_series results are sorted chronologically
effe46f
@spool
feat: add XarrayTimeSeriesCalcManager to aggregate summary calculat…
9744466 
…ions
@spool
fix: XarrayTimeSeriesCalcManager PathLike attributes -> Path
df50b31
@spool
feat: add range_len for easing progress information
e00b984
@griff-rees
fix: cases where XarrayTimeSeriesCalcManager.sub_path is None
46923a9
@griff-rees
fix: XarrayTimeSeriesCalcManager save_path -> sub_path typo
0b4a602
@spool
doc: fix and add plots to cpm_projection.qmd
aedea44
@spool
Merge remote-tracking branch 'origin/plot-cpm-time-series' into plot-…
7e2c48b 
…cpm-time-series
@spool
doc: add . marks for a plot in cpm_projection.qmd
82ada4e
@spool
chore: pre-commit run --all-files
10f4980
@spool
fix: add jupyter-cache for quarto builds
a6aa5fb
@spool
doc: add group plots in cpm_projection.qmd
f28040c
@spool
doc: refactor cpm_projection.qmd for remote hosted data
7b7fa98
@spool
ci: add pip install for docs rendering
3ea428e
@spool
ci: fix local pip install for docs rendering
61b9689
@spool
ci: add gdal for docs render
25f0f20
@spool
ci: target gdal=3.8.4 for doc render
378b271
@spool
doc: refactor cpm_projection.qmd for remote hosted data
f897a84
@sgreenbury
Notebook with plots measuring variable diffs
a3c7101
@sgreenbury
Fix plots
56a3f4d
@sgreenbury
Additional debugging plot
128a5e3
@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@griff-rees griff-rees self-assigned this Oct 29, 2024
spool and others added 9 commits October 29, 2024 11:27
@spool
doc: fix quarto links in cpm_projection.qmd
e347444
@andrewphilipsmith @griff-rees
Updated to docs section of root Readme.md dodc
216619f 
Co-Authored-By: griff-rees <[email protected]>
@spool
Merge remote-tracking branch 'origin/fix-docs' into fix-docs
0e197da
@spool
doc: remove broken table of contents and update README.md
1147c06
@andrewphilipsmith @sgreenbury
add warning about install methods
03accd3 
Co-Authored-By: Sam Greenbury <[email protected]>
@andrewphilipsmith @sgreenbury @griff-rees
Update navigation
952fd4f 
Co-Authored-By: Sam Greenbury <[email protected]>
Co-Authored-By: griff-rees <[email protected]>
@andrewphilipsmith @sgreenbury @griff-rees
Add subheadings for doc types
782d724 
Co-Authored-By: Sam Greenbury <[email protected]>
Co-Authored-By: griff-rees <[email protected]>
@andrewphilipsmith @griff-rees
doc: deprecate old pipeline guidance
2f8c2cc 
Co-Authored-By: griff-rees <[email protected]>
@andrewphilipsmith @griff-rees
doc: reorders sections within the navigation
45d7a4d 
adds an Appendix header and moves more detailed exploration and older content here.

Co-Authored-By: griff-rees <[email protected]>
andrewphilipsmith
andrewphilipsmith approved these changes Oct 30, 2024
View reviewed changes
Copy link
Collaborator

@andrewphilipsmith andrewphilipsmith left a comment

Choose a reason for hiding this comment

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

Reviewed in in-person session

@griff-rees griff-rees added the documentation Improvements or additions to documentation label Oct 30, 2024
@griff-rees griff-rees linked an issue Oct 30, 2024 that may be closed by this pull request
Add tool for plotting projection time series #173
Closed
@griff-rees griff-rees marked this pull request as ready for review October 30, 2024 17:01
@griff-rees griff-rees merged commit 7d7a675 into main Oct 30, 2024
3 checks passed
@griff-rees griff-rees deleted the fix-docs branch October 30, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrewphilipsmith andrewphilipsmith andrewphilipsmith approved these changes

Assignees

@griff-rees griff-rees

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
Create and release UK BC datasets
Development

Successfully merging this pull request may close these issues.

Add tool for plotting projection time series
4 participants
@griff-rees @andrewphilipsmith @spool @sgreenbury