lincc-frameworks / tape

[Deprecated] Package for working with LSST time series data
https://tape.readthedocs.io
MIT License
12 stars 3 forks source link

Adding Figures to Quickstart #403

Closed AndyTza closed 5 months ago

AndyTza commented 5 months ago

Change Description

The following pull request has included two new figures for the Quickstart tutorial (closes Issue 402). The first figure includes a simple scatter plot of the computed features (delta-time versus SF2). The added cell includes a simple method for fetching the ID of the source object with the highest SF2 value:

max_id = result_comp.iloc[result_comp['sf2'].argmax()].lc_id
lc = ens.source.compute().loc[int(max_id)]

and includes a simple matplotlib plotting routine for the multi-band light curve of that specific source.

Solution Description

No issues have been fixed.

Code Quality

Project-Specific Pull Request Checklists

Bug Fix Checklist

New Feature Checklist

Documentation Change Checklist

Build/CI Change Checklist

Other Change Checklist

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

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

github-actions[bot] commented 5 months ago
Before [86a0c45a] <v0.3.5> After [226a09db] Ratio Benchmark (Parameter)
44.5±0.1ms 45.4±0.1ms 1.02 benchmarks.time_batch
47.4±0.2ms 48.1±0.5ms 1.02 benchmarks.time_prune_sync_workflow

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.55%. Comparing base (86a0c45) to head (a40a6cf).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #403 +/- ## ======================================= Coverage 95.55% 95.55% ======================================= Files 25 25 Lines 1710 1710 ======================================= Hits 1634 1634 Misses 76 76 ```

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