Skip to content

fix #2061#2089

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

fix #2061#2089
Jonarw merged 2 commits into
oxyplot:developfrom
Jonarw:datetime

Conversation

@Jonarw

@Jonarw Jonarw commented Jul 20, 2024

Copy link
Copy Markdown
Member

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

@VisualMelon VisualMelon left a comment

Copy link
Copy Markdown
Contributor

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