Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Live Chart pokazuje teraz cały kształt wykresu. #5

Merged
merged 2 commits into from
Apr 17, 2024

Conversation

nifiz
Copy link
Collaborator

@nifiz nifiz commented Apr 16, 2024

Dodałem klasę DataScaler, ze statyczną funkcją ScaleData(DataTable dt, int TargetDatapoints=250).

Skaluje ona dowolnie duży DataTable dt do nowego DataTable o ilości rzędów = TargetDatapoints.
Dzięki temu cały kształt wykresu jest widoczny w oknie aplikacji.

Póki co tracimy do TargetDatapoints-1 punktów oryginalnych danych (trzeba uwzględnić resztę po podziale danych), kod jest do refaktoryzacji/ulepszenia ale ma swoją zasadniczą funkcjonalność.

TargetDatapoints powinien być równy atrybutowi _visibleElements klasy ViewModel z RealTimeChart.cs, aby wykres się w oknie nie powtarzał, a także żeby było go widać w całości.

@nifiz nifiz requested review from JN0122, puechua and A-Janowski April 17, 2024 15:17
Copy link
Owner

@JN0122 JN0122 left a comment

Choose a reason for hiding this comment

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

Super, przyda się, żeby te dane lepiej zaprezentować, ale to proponuje później dodać jakiś select w interfejsie, z którego użytkwonik może zmienić skale wykresu.

@JN0122
Copy link
Owner

JN0122 commented Apr 17, 2024

Jeszcze jakbyś merge z tym branchem zrobił i konflikty rozwiązał było by super

@nifiz nifiz merged commit b8b96d7 into realtimechart Apr 17, 2024
@nifiz nifiz deleted the rtc_skalowanie_wykresu branch April 17, 2024 21: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.

3 participants