-
-
Notifications
You must be signed in to change notification settings - Fork 6k
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
The line chart enlarged is crash #2797
Comments
I need full steps to reproduce, especially prefer ChartDemo. You should provide more info for a crash. Like input/crash log, not just this vague message |
I'm experiencing a crash in the same function. Unfortunately, I can't reproduce it, but it's crashing in our beta-released version. I'm attaching the call-stack. The crash is not frequent but on the same dataset, it always happens. The crash happens in viewDidLoad, so the graph is not even appeared. ps. maybe helpful, the granularity is disabled for the axis. I think the "roundToNextSignificant", maybe because of a negative number. I've checked the implementation, and also tried mocking to cause a crash, but without success. |
@PeterKovacs89 It may not be the same root cause for crashing. but for yours, a wild guess is your data sets is empty but try to calculate something. If you are using crashlytics, you can add some log/var to print out the values when crashing. |
I could reproduce the crash on an empty dataset, thank you for your help! <3 |
what's the cause then? A bug of ours or misusage? if latter, we can close this. |
I know nothing about the original issue, my issue was a kind of misusage. Btw I think the framework should not crash when it receives empty data, so I think it should be fixed in any case. |
We will try our best not to crash. But some crashes warns us what's wrong. Catching the crash and throw a log on your side will make us unaware of anything wrong. |
The new data will crash when the line graph is enlarged and then reassigned. If the assignment data is the same as before, it won't crash.
charts crash code:
For help,thanks.
The text was updated successfully, but these errors were encountered: