Assertion Error after an Upgrade to XCode Version 14.2 and Charts Master
- Dominant language
- Swift
- Stars
- 28k
- Forks
- 6k
- PR merge metrics
- No merged PRs in 30d
Description
> ℹ Please fill out this template when filing an issue.
> All lines beginning with an ℹ symbol instruct you with what info we expect. You can delete those lines once you've filled in the info.
>
> Per our [*CONTRIBUTING guidelines](https://github.com/jjatie/Charts/blob/master/CONTRIBUTING.md), we use GitHub for
> bugs and feature requests, not general support. Other issues should be opened on Stack Overflow with the tag [`ios-charts`.](https://stackoverflow.com/questions/tagged/ios-charts)
>
> Please remove this line and everything above it before submitting.
* [x] I've read, understood, and done my best to follow the [*CONTRIBUTING guidelines](https://github.com/jjatie/Charts/blob/master/CONTRIBUTING.md).
## What did you do?
Upgraded XCode to Version 14.2 from 13 and Charts to branch->Master from older version 3.6
## What did you expect to happen?
I expected my app to run without any error and show chart as it was doing before these upgrades
## What happened instead?
XCode project was built successfully after I made some code changes as per #4860 (and made few other minor changes like changing addDataSet to append)
However, after the build, the simulator gets stuck and XCode throws following error after launching
Assertion failed: removeAll(keepingCapacity:) not supported for CombinedData
Error at :
Script: ChartData.swift
function: removeAll
LOC: assert(!(self is CombinedChartData), "\(#function) not supported for CombinedData")
## Charts Environment
**Charts version/Branch/Commit Number: Branch Master
**Xcode version: 14.2
**Swift version: Swift 5
**Platform(s) running Charts: XCode Simulator iPhone 14 pro (iOS 16.2)
**macOS version running Xcode: Ventura version 13.1 (22C65)
## Demo Project
ℹ Please link to or upload a project we can download that reproduces the issue.
Contributor guide
Research direction
Start with ChartData.swift and the removeAll assertion for CombinedChartData, then review the code changes required by #4860 and the reported addDataSet-to-append changes. Reproduce the failure using Charts master with Xcode 14.2, Swift 5, and the iPhone 14 Pro simulator; done means the simulator launches without this assertion and the chart displays as before.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- swift
- Domain
- data-visualization
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100