Skip to content

make docstring test work in all environments#896

Merged
bpkroth merged 2 commits into
microsoft:mainfrom
motus:sergiym/test/config_persistence_doc
Dec 7, 2024
Merged

make docstring test work in all environments#896
bpkroth merged 2 commits into
microsoft:mainfrom
motus:sergiym/test/config_persistence_doc

Conversation

@motus

@motus motus commented Dec 7, 2024

Copy link
Copy Markdown
Member

Pull Request

Title

Make inline unit tests that display Pandas DataFrame work in all environments, regardless of terminal width.


Description

Inline tests in docstrings sometimes fail if the terminal width does not match the expected one and the pandas DataFrame got displayed differently.


Type of Change

  • 🛠️ Bug fix
  • 📝 Documentation update
  • 🧪 Tests

Testing

Unit tests should pass now


@motus motus added bug Something isn't working ready for review Ready for review tests Add or fix unit tests mlos-bench labels Dec 7, 2024
@motus motus self-assigned this Dec 7, 2024
@motus motus requested a review from a team as a code owner December 7, 2024 00:38
@bpkroth bpkroth merged commit 25bf9a0 into microsoft:main Dec 7, 2024
@motus motus deleted the sergiym/test/config_persistence_doc branch December 9, 2024 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working mlos-bench ready for review Ready for review tests Add or fix unit tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix flaky pydoctest issue with pandas printing

2 participants