Skip to content

Series' describe_series fixes, method standardization across classes#19

Merged
d-giles merged 4 commits intomainfrom
16-dataseries-time
Jul 11, 2025
Merged

Series' describe_series fixes, method standardization across classes#19
d-giles merged 4 commits intomainfrom
16-dataseries-time

Conversation

@d-giles
Copy link
Copy Markdown
Collaborator

@d-giles d-giles commented Jul 11, 2025

Fixes #16 and #17

This PR addresses a couple issues with how Series' were initiating, causing the issues mentioned above, as well as some standardization of class workflows and documentation. The issues stemmed from an inconsistency in how Series' were initialized compared with Frames and Cubes.

@github-actions
Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/lkdata
  datacube.py
  dataframe.py
  dataseries.py 83, 130-134, 256, 281, 284
  mixins.py
Project Total  

This report was generated by python-coverage-comment-action

@d-giles d-giles merged commit 1ea9357 into main Jul 11, 2025
5 checks passed
@d-giles d-giles deleted the 16-dataseries-time branch July 11, 2025 18:10
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.

DataSeries not defining time objects properly

1 participant