fix #2061 by Jonarw · Pull Request #2089 · oxyplot/oxyplot · GitHub
Skip to content

fix #2061#2089

Merged
Jonarw merged 2 commits intooxyplot:developfrom
Jonarw:datetime
Jul 20, 2024
Merged

fix #2061#2089
Jonarw merged 2 commits intooxyplot:developfrom
Jonarw:datetime

Conversation

@Jonarw
Copy link
Copy Markdown
Member

@Jonarw Jonarw commented Jul 20, 2024

Fixes #2061 .

Checklist

  • I have included examples or tests
  • I have updated the change log
  • I am listed in the CONTRIBUTORS file
  • I have cleaned up the commit history (use rebase and squash)

Changes proposed in this pull request:

  • Add DateTimeAxis.DateTimePrecision property, which is used to round DateTime values used by the axis. Default value is 1ms, so this should restore the behavior from .NET 6 and before.
  • Add an example showcasing the issue
  • Mark DateTimeAxis.ToDateTime as obsolete to make people aware of the issue

@oxyplot/admins

Copy link
Copy Markdown
Contributor

@VisualMelon VisualMelon left a comment

Choose a reason for hiding this comment

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

Thanks for picking this up!

Comment thread Source/OxyPlot/Axes/DateTimeAxis.cs Outdated
@Jonarw Jonarw merged commit 4c372ac into oxyplot:develop Jul 20, 2024
@Jonarw Jonarw deleted the datetime branch July 21, 2024 08:24
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.

DateTimeAxis.ToDateTime doesn't behave as intended in .NET 8

2 participants