Skip to content

handle negative microseconds with %f format - #13495

Merged
bryevdv merged 2 commits into
bokeh:branch-3.4from
muendlein:13067_fix_negative_microseconds_ticks_for_%f
Oct 31, 2023
Merged

handle negative microseconds with %f format#13495
bryevdv merged 2 commits into
bokeh:branch-3.4from
muendlein:13067_fix_negative_microseconds_ticks_for_%f

Conversation

@muendlein

@muendlein muendlein commented Oct 29, 2023

Copy link
Copy Markdown
Contributor

All pull requests must have an associated issue in the issue tracker. If there
isn't one, please go open an issue describing the defect, deficiency or desired
feature. You can read more about our issue and PR processes in the
wiki.

Note: The main bug of #13067 is within the external timezone library

@codecov

codecov Bot commented Oct 29, 2023

Copy link
Copy Markdown

Codecov Report

Merging #13495 (65f0a47) into branch-3.4 (362677d) will decrease coverage by 0.05%.
Report is 24 commits behind head on branch-3.4.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           branch-3.4   #13495      +/-   ##
==============================================
- Coverage       92.67%   92.62%   -0.05%     
==============================================
  Files             320      321       +1     
  Lines           20362    20383      +21     
==============================================
+ Hits            18870    18880      +10     
- Misses           1492     1503      +11     

@mattpap

mattpap commented Oct 29, 2023

Copy link
Copy Markdown
Contributor

Note: The main bug of #13067 is within the external timezone library

Which btw. is not maintained anymore, so it's up to us to fix any underlying problems, or switch to another library.

@bryevdv

bryevdv commented Oct 31, 2023

Copy link
Copy Markdown
Member

or switch to another library.

Probably the best solution, but this will have to do for now

@bryevdv
bryevdv merged commit d620157 into bokeh:branch-3.4 Oct 31, 2023
@bryevdv bryevdv mentioned this pull request Oct 31, 2023
bryevdv pushed a commit that referenced this pull request Oct 31, 2023
* fix %f formatting for pre epoch

* add test

---------

Co-authored-by: maxmuendlein <max.muendlein@stud-mail.uni-wuerzburg.de>
@bryevdv bryevdv modified the milestones: 3.4, 3.3.1 Oct 31, 2023
bryevdv pushed a commit that referenced this pull request Nov 1, 2023
* fix %f formatting for pre epoch

* add test

---------

Co-authored-by: maxmuendlein <max.muendlein@stud-mail.uni-wuerzburg.de>
Chiemezuo pushed a commit to Chiemezuo/bokeh that referenced this pull request Aug 27, 2024
* fix %f formatting for pre epoch

* add test

---------

Co-authored-by: maxmuendlein <max.muendlein@stud-mail.uni-wuerzburg.de>
@github-actions

Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants