Skip to content

Conversation

@wxtim
Copy link
Member

@wxtim wxtim commented Feb 5, 2026

Follow on from #7204

Should satisfy the desires of any users who wish to open up the Raw data in Excel.

Check List

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • Applied any dependency changes to both setup.cfg (and conda-environment.yml if present).
  • Tests are not included, because this is almost entirely dependent on Pandas functionality.
  • Changelog entry included if this is a change that can affect users
  • Cylc-Doc pull request opened if required at cylc/cylc-doc/pull/XXXX.
  • If this is a bug fix, PR should be raised against the relevant ?.?.x branch.

oliver-sanders and others added 2 commits February 2, 2026 13:16
* Port onto latest pandas and matplotlib (bump pandas dep).
* Remove back-compat code.
* Add basic tests.
"to_<output>" methods.
Added "json" and "csv" to the list of raw output formats.
n.b: You can't use `to_excel` or `to_markdown` without
additional optional dependencies.
Also left scope in the interface for "csv-specific-char-delimited".
@wxtim wxtim force-pushed the report-timings.generic-pandas.DataFrame.to_type_interface branch from ffef099 to 6e04161 Compare February 5, 2026 09:53
@wxtim wxtim requested a review from oliver-sanders February 5, 2026 09:54
"""

buf.write('<html><head><style>%s</style></head><body>' % css)
buf.write('<!DOCTYPE html><html><head><style>%s</style></head><body>' % css)
Copy link
Member Author

Choose a reason for hiding this comment

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

Not having this showed as an warning when I looked at the HTML with dev tools on

'raw': format_raw,
'summary': format_summary,
'html': format_html,
'csv': partial(format_generic, format='csv'),
Copy link
Member Author

Choose a reason for hiding this comment

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

Sadly to_markdown and to_excel require extra dependencies, and to_html, though it works isn't very nice to look at and needs some CSS appended.

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