Minor docs cleanups for TAPE

lincc-frameworks / tape

[Deprecated] Package for working with LSST time series data

https://tape.readthedocs.io

MIT License

12 stars 3 forks source link

Minor docs cleanups for TAPE #414

Closed wilsonbb closed 5 months ago

wilsonbb commented 5 months ago

A series of minor docs cleanups addressing small changes proposed in https://github.com/lincc-frameworks/tape/issues/372

The following changes are made:

When loading a time series object in what was formerly this section, add a sentence such as ``Note that this loads the data from all available bands’’
Changed Installing in the Quickstart to be an executable shell command to pip install TAPE
Demonstrate setting a random seed in Ensemble.smaple
Improves the plot in in From the ensemble
[x] My PR includes a link to the issue that I am addressing

review-notebook-app[bot] commented 5 months ago

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

github-actions[bot] commented 5 months ago

Before [19ff5ad2]	After [7292fe78]	Ratio	Benchmark (Parameter)
34.1±0.3ms	34.7±0.6ms	1.02	benchmarks.time_prune_sync_workflow
33.1±0.3ms	33.5±0.6ms	1.01	benchmarks.time_batch

Click here to view all benchmarks.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.77%. Comparing base (19ff5ad) to head (bad18b6).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #414 +/- ## ======================================= Coverage 95.77% 95.77% ======================================= Files 25 25 Lines 1751 1751 ======================================= Hits 1677 1677 Misses 74 74 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.