Skip to content

Slim public API; remove properties#354

Merged
jsmariegaard merged 11 commits intomainfrom
remove-properties
Dec 20, 2023
Merged

Slim public API; remove properties#354
jsmariegaard merged 11 commits intomainfrom
remove-properties

Conversation

@jsmariegaard
Copy link
Copy Markdown
Member

@jsmariegaard jsmariegaard commented Dec 20, 2023

Removed properties:

  • Comparer
    • quantity_name
    • obs
    • mod
    • obs_name
    • _unit_text
  • TimeSeries
    • color (temporary disabled until we have better color handling)
    • plot.plot (no need for a ts.plot.plot() - it's ugly 🤢)

Apart from this lots of docstring improvements and fixing broken links in docs.

@jsmariegaard jsmariegaard marked this pull request as ready for review December 20, 2023 13:25
@ecomodeller
Copy link
Copy Markdown
Member

  • plot.plot (no need for a ts.plot.plot() - it's ugly 🤢)

Fixed already in #347 ... just saying

@jsmariegaard jsmariegaard merged commit 543821c into main Dec 20, 2023
@jsmariegaard jsmariegaard deleted the remove-properties branch December 20, 2023 14:57
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.

2 participants