Skip to content

Show number of YoY histogram points when detailed=True#324

Merged
mdeceglie merged 6 commits into
developmentfrom
n_slopes
May 18, 2022
Merged

Show number of YoY histogram points when detailed=True#324
mdeceglie merged 6 commits into
developmentfrom
n_slopes

Conversation

@kandersolar
Copy link
Copy Markdown
Member

@kandersolar kandersolar commented Apr 26, 2022

  • Code changes are covered by tests
  • Code changes have been evaluated for compatibility/integration with TrendAnalysis
  • [ ] New functions added to __init__.py
  • [ ] API.rst is up to date, along with other sphinx docs pages
  • [ ] Example notebooks are rerun and differences in results scrutinized
  • Updated changelog

See #298. I think detailed=True is not used in the example notebooks anywhere; worth using that somewhere to (1) highlight that it exists, and (2) see it in action for ourselves?

RTD failure is addressed by #325

Copy link
Copy Markdown
Collaborator

@mdeceglie mdeceglie left a comment

Choose a reason for hiding this comment

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

LGTM!

@mdeceglie mdeceglie merged commit 827361f into development May 18, 2022
@mdeceglie mdeceglie deleted the n_slopes branch May 18, 2022 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants